diff options
| author | myk%mozilla.org <> | 2002-11-18 08:33:54 +0100 |
|---|---|---|
| committer | myk%mozilla.org <> | 2002-11-18 08:33:54 +0100 |
| commit | 78d40afc44198a333659c12fd20af620ea40a43a (patch) | |
| tree | c880d76c7b345791560caef7b9c11755d507d583 | |
| parent | 8edc955f0637f4e8c1cde136319a03e0060f0819 (diff) | |
| download | bugzilla-78d40afc44198a333659c12fd20af620ea40a43a.tar.gz bugzilla-78d40afc44198a333659c12fd20af620ea40a43a.tar.xz | |
Fix for bug 180632: corrects reference flag->is_requesteeble to flag->type->is_requesteeble
r=bbaetz
a=myk
| -rw-r--r-- | Bugzilla/Flag.pm | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/Bugzilla/Flag.pm b/Bugzilla/Flag.pm index d601a115b..0a404eff4 100644 --- a/Bugzilla/Flag.pm +++ b/Bugzilla/Flag.pm @@ -323,10 +323,11 @@ sub modify { # being requested, otherwise we'll get false positives and think # the user changed the flag when they didn't. next if - $status eq $flag->{'status'} # the flag's status hasn't changed, and - && (!$flag->{'is_requesteeble'} # the flag isn't specifically requestable, or - || $status ne "?" # the flag isn't being requested, or - || ($flag->{'requestee'} # the requestee hasn't changed + $status eq $flag->{'status'} # the flag's status hasn't changed, and: + && (!$flag->{'type'}->{'is_requesteeble'} + # the flag isn't specifically requestable + || $status ne "?" # or the flag isn't being requested + || ($flag->{'requestee'} # or the requestee hasn't changed && ($requestee_email eq $flag->{'requestee'}->{'email'}))); # Since the status is validated, we know it's safe, but it's still |