Commit 771e38ec authored by dmose%mozilla.org's avatar dmose%mozilla.org

Fixed bug 71600: mail would be sent about a given bug iff all flags for stuff…

Fixed bug 71600: mail would be sent about a given bug iff all flags for stuff changed in a given group were set. Should have sent mail if any flags for stuff changed in a given group were set. r=dave@intrec.com,endico@mozilla.org
parent 41b4d957
...@@ -785,6 +785,8 @@ sub filterEmailGroup ($$$) { ...@@ -785,6 +785,8 @@ sub filterEmailGroup ($$$) {
push(@filteredList,$person); push(@filteredList,$person);
my $detectedOn = 0;
foreach my $attribute (@emailAttributes) { foreach my $attribute (@emailAttributes) {
my $matchName = 'email' . $emailGroup . $attribute; my $matchName = 'email' . $emailGroup . $attribute;
...@@ -795,14 +797,23 @@ sub filterEmailGroup ($$$) { ...@@ -795,14 +797,23 @@ sub filterEmailGroup ($$$) {
next; next;
} }
if ( $flagName eq $matchName && $flagValue ne 'on') { if ($flagName eq $matchName){
pop(@filteredList); if ($flagValue eq 'on') {
$detectedOn = 1;
}
} }
} # for each userFlag } # for each userFlag
} # for each email attribute } # for each email attribute
# if the current flag hasn't been detected on at least once,
# this person gets filtered from this group.
#
if (! $detectedOn) {
pop(@filteredList);
}
# check to see if the person was removed from this email # check to see if the person was removed from this email
# group. # group.
......
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