Closed
Bug 179321
Opened 21 years ago
Closed 21 years ago
cannot clear status whiteboard entirely
Categories
(Bugzilla :: Creating/Changing Bugs, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: levon, Assigned: bbaetz)
References
Details
Attachments
(1 file)
731 bytes,
patch
|
bugreport
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•21 years ago
|
||
I'll look at this in a sec
Assignee | ||
Comment 2•21 years ago
|
||
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...
Assignee | ||
Comment 3•21 years ago
|
||
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
Assignee | ||
Updated•21 years ago
|
Attachment #105737 -
Flags: review?
Assignee | ||
Comment 4•21 years ago
|
||
Comment on attachment 105737 [details] [diff] [review] fix myk, we probably want this to make the next bmo pull
Comment 5•21 years ago
|
||
Comment on attachment 105737 [details] [diff] [review] fix 2xr=joel
Attachment #105737 -
Flags: review?
Attachment #105737 -
Flags: review+
Assignee | ||
Comment 6•21 years ago
|
||
Comment on attachment 105737 [details] [diff] [review] fix dave, a=, please?
Attachment #105737 -
Flags: review+ → review?
Comment 7•21 years ago
|
||
a=justdave
Assignee | ||
Comment 8•21 years ago
|
||
checked in to cvs; will appear on bmo when myk updates it. The strange flag state is due to bug 179334.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•21 years ago
|
Attachment #105737 -
Flags: review?
Updated•11 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•