179321 - cannot clear status whiteboard entirely

Status: RESOLVED FIXED

(Bugzilla :: Creating/Changing Bugs, defect)

Description

[John Levon]

go to http://bugzilla.mozilla.org/show_bug.cgi?id=179294

and attempt to entirely clear the Status Whiteboard, and
commit. The change seems to be accepted, but if you revisit
the bug, the status whiteboard still has the old value.

You /can/ change the value (that's where "test" came from)
but clearing it entirely does not work.

Comment 1

[Bradley Baetz (:bbaetz)]

I'll look at this in a sec

Comment 2

[Bradley Baetz (:bbaetz)]

This is gerv's $::FORM{dontchange} stuff - show_bug doesn't set it, so |'' eq
undef|, and we don't change it. Patch coming

I still think that taht was a silly change, and that we should have used
checkboxes instead...

Comment 3

[Bradley Baetz (:bbaetz)]

Attachment: fix

The other places which check this blindly are comparing agiast stuff like
product and component, which can never be empty. Someone should double check
that, though

Comment 4

[Bradley Baetz (:bbaetz)]

Comment on attachment 105737 [details] [diff] [review]
fix

myk, we probably want this to make the next bmo pull

Comment 5

[Joel Peshkin]

Comment on attachment 105737 [details] [diff] [review]
fix

2xr=joel

Comment 6

[Bradley Baetz (:bbaetz)]

Comment on attachment 105737 [details] [diff] [review]
fix

dave, a=, please?

Comment 7

[Dave Miller [:justdave]]

a=justdave

Comment 8

[Bradley Baetz (:bbaetz)]

checked in to cvs; will appear on bmo when myk updates it.

The strange flag state is due to bug 179334.