Comment 1 for bug 475433

Revision history for this message
Björn Tillenius (bjornt) wrote : Re: [Bug 475433] [NEW] oops leaving milestone code_name empty

On Thu, Nov 05, 2009 at 02:12:36PM -0000, Curtis Hovey wrote:
> This was not a problem last release when I edited milestones. The
> StrippedTextLine.set() method was added to by Tim's 2009-10-2 bmp-
> inline-commit-message branch. It either needs to return early, or
> NoneableTextLine should test for None before calling super.

Don't forget to consider the case where the field is already set to some
non-None value, and you try to set it to None.

As a side note, we should replace the default TextLine widget with a
widget that always strips the input, so that we don't need to do this in
the field itself.