Commit 9ad9385a authored by jocuri%softhome.net's avatar jocuri%softhome.net

Patch for bug 212017: Should be able to change a bug if it has a flag requested…

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.
parent 038e3b0e
...@@ -166,7 +166,9 @@ sub validate { ...@@ -166,7 +166,9 @@ sub validate {
{ id => $id, status => $status }); { id => $id, status => $status });
# Make sure the user didn't request the flag unless it's requestable. # 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", ThrowCodeError("flag_status_invalid",
{ id => $id, status => $status }); { id => $id, status => $status });
} }
......
...@@ -103,7 +103,7 @@ ...@@ -103,7 +103,7 @@
[% IF type.is_active %] [% IF type.is_active %]
<option value="+" [% "selected" IF flag.status == "+" %]>+</option> <option value="+" [% "selected" IF flag.status == "+" %]>+</option>
<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> <option value="?" [% "selected" IF flag.status == "?" %]>?</option>
[% END %] [% END %]
[% ELSE %] [% ELSE %]
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment