mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-12-04 02:57:38 +00:00
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
This commit is contained in:
parent
666ef60528
commit
a3d777a1b7
@ -785,6 +785,8 @@ sub filterEmailGroup ($$$) {
|
||||
|
||||
push(@filteredList,$person);
|
||||
|
||||
my $detectedOn = 0;
|
||||
|
||||
foreach my $attribute (@emailAttributes) {
|
||||
|
||||
my $matchName = 'email' . $emailGroup . $attribute;
|
||||
@ -795,14 +797,23 @@ sub filterEmailGroup ($$$) {
|
||||
next;
|
||||
}
|
||||
|
||||
if ( $flagName eq $matchName && $flagValue ne 'on') {
|
||||
pop(@filteredList);
|
||||
if ($flagName eq $matchName){
|
||||
if ($flagValue eq 'on') {
|
||||
$detectedOn = 1;
|
||||
}
|
||||
}
|
||||
|
||||
} # for each userFlag
|
||||
|
||||
} # 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
|
||||
# group.
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user