'Add New Milestone' not working

[Brad Anderson <brad@…>]

I pulled Revision 940 to my local and was unable to create a new milestone. The problem was that the 'id' variable in Milestone.py was not being set to a default of -1 by the dictionary's get() method.

I'm a total Python newbie, so someone needs to check if the patch is a) in the right place, and b) the proper Python way. I don't think removing the hidden field name="id" on the html form is the way, because at times, it's filled with the milestone name for other modes.

[brad@…]

patch file

[Brad Anderson <brad@…>]

Oops, forgot some particulars:

Gentoo Linux (2.6.6 kernel) Python 2.3.3 mod_python

anything else?

[Brad Anderson <brad@…>]

I was getting an error with the patch, something like

TypeError: len() of unsized object

So I changed the check to:

if id == "":
   id = -1;

[brad@…]

patch file (Try #2)

[jonas]

test

[jonas]

test2

[cmlenz]

Fixed in [941].