Skip to content

bugzilla
bugzilla
Commit 78d40afc authored by myk%mozilla.org's avatar myk%mozilla.org
Browse files
Options

Fix for bug 180632: corrects reference flag->is_requesteeble to flag->type->is_requesteeble

r=bbaetz a=myk
parent 8edc955f
Show whitespace changes
Inline Side-by-side
Showing with 5 additions and 4 deletions
Bugzilla/Flag.pm
View file @ 78d40afc
...@@ -323,10 +323,11 @@ sub modify { ...@@ -323,10 +323,11 @@ sub modify {
# being requested, otherwise we'll get false positives and think # being requested, otherwise we'll get false positives and think
# the user changed the flag when they didn't. # the user changed the flag when they didn't.
next if next if
$status eq $flag->{'status'} # the flag's status hasn't changed, and $status eq $flag->{'status'} # the flag's status hasn't changed, and:
&& (!$flag->{'is_requesteeble'} # the flag isn't specifically requestable, or && (!$flag->{'type'}->{'is_requesteeble'}
|| $status ne "?" # the flag isn't being requested, or # the flag isn't specifically requestable
|| ($flag->{'requestee'} # the requestee hasn't changed || $status ne "?" # or the flag isn't being requested
|| ($flag->{'requestee'} # or the requestee hasn't changed
&& ($requestee_email eq $flag->{'requestee'}->{'email'}))); && ($requestee_email eq $flag->{'requestee'}->{'email'})));
# Since the status is validated, we know it's safe, but it's still # Since the status is validated, we know it's safe, but it's still
......
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