From fca47162ff41ff44e06448ff49fb833972a00e60 Mon Sep 17 00:00:00 2001 From: "myk%mozilla.org" Date: Sun, 17 Nov 2002 23:33:54 +0000 Subject: [PATCH] Fix for bug 180632: corrects reference flag->is_requesteeble to flag->type->is_requesteeble r=bbaetz a=myk --- webtools/bugzilla/Bugzilla/Flag.pm | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/webtools/bugzilla/Bugzilla/Flag.pm b/webtools/bugzilla/Bugzilla/Flag.pm index d601a115b6c1..0a404eff48de 100644 --- a/webtools/bugzilla/Bugzilla/Flag.pm +++ b/webtools/bugzilla/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