From 23f1508f31cbcf7c59f82020c4782abf1582fabc Mon Sep 17 00:00:00 2001 From: "olav%bkor.dhs.org" <> Date: Sun, 7 Jan 2007 19:17:59 +0000 Subject: Bug 366187: Bugmail should consider 'Any field not mentioned above changes' even when a 'known' field changes Patch by Olav Vitters r=LpSolit a=myk --- Bugzilla/User.pm | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) (limited to 'Bugzilla/User.pm') diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm index 02c6c4db9..c6fd0b524 100644 --- a/Bugzilla/User.pm +++ b/Bugzilla/User.pm @@ -1351,11 +1351,7 @@ sub wants_bug_mail { } else { # Catch-all for any change not caught by a more specific event - # XXX: Temporary Compatibility Change 2 of 2: - # This code is disabled, and replaced with the code a few lines - # below, in order to make the behaviour more like the original, - # which only added this event if _all_ changes were of "other" type. - # $events{+EVT_OTHER} = 1; + $events{+EVT_OTHER} = 1; } # If the user is in a particular role and the value of that role @@ -1396,12 +1392,6 @@ sub wants_bug_mail { my @event_list = keys %events; - # XXX Temporary Compatibility Change 2 of 2: - # See above comment. - if (!scalar(@event_list)) { - @event_list = (EVT_OTHER); - } - my $wants_mail = $self->wants_mail(\@event_list, $relationship); # The negative events are handled separately - they can't be incorporated -- cgit v1.2.3-24-g4f1b