diff options
| author | jocuri%softhome.net <> | 2004-11-03 06:59:39 +0000 |
|---|---|---|
| committer | jocuri%softhome.net <> | 2004-11-03 06:59:39 +0000 |
| commit | 9ad9385a463557b40163f890b06a882679962b29 (patch) | |
| tree | 4cafaae739845f3de28204bc320910cd59469d70 | |
| parent | 038e3b0e82dde2dec9a426cbbdf4e615d5f2a308 (diff) | |
| download | bugs-9ad9385a463557b40163f890b06a882679962b29.tar bugs-9ad9385a463557b40163f890b06a882679962b29.tar.gz bugs-9ad9385a463557b40163f890b06a882679962b29.tar.bz2 bugs-9ad9385a463557b40163f890b06a882679962b29.tar.xz bugs-9ad9385a463557b40163f890b06a882679962b29.zip | |
Patch for bug 212017: Should be able to change a bug if it has a flag requested that is no longer requestable; patch by Frédéric Buclin <LpSolit@netscape.net>; r=myk, vladd, a=myk.
| -rw-r--r-- | Bugzilla/Flag.pm | 4 | ||||
| -rw-r--r-- | template/en/default/flag/list.html.tmpl | 2 |
2 files changed, 4 insertions, 2 deletions
diff --git a/Bugzilla/Flag.pm b/Bugzilla/Flag.pm index aca271261..a58bc7e3a 100644 --- a/Bugzilla/Flag.pm +++ b/Bugzilla/Flag.pm @@ -166,7 +166,9 @@ sub validate { { id => $id, status => $status }); # Make sure the user didn't request the flag unless it's requestable. - if ($status eq '?' && !$flag->{type}->{is_requestable}) { + # If the flag was requested before it became unrequestable, leave it as is. + if ($status eq '?' && $flag->{status} ne '?' && + !$flag->{type}->{is_requestable}) { ThrowCodeError("flag_status_invalid", { id => $id, status => $status }); } diff --git a/template/en/default/flag/list.html.tmpl b/template/en/default/flag/list.html.tmpl index 75e61d774..c93515842 100644 --- a/template/en/default/flag/list.html.tmpl +++ b/template/en/default/flag/list.html.tmpl @@ -103,7 +103,7 @@ [% IF type.is_active %] <option value="+" [% "selected" IF flag.status == "+" %]>+</option> <option value="-" [% "selected" IF flag.status == "-" %]>-</option> - [% IF type.is_requestable %] + [% IF type.is_requestable || flag.status == "?" %] <option value="?" [% "selected" IF flag.status == "?" %]>?</option> [% END %] [% ELSE %] |