Commit 2bdc68f5 authored by cyeh%bluemartini.com's avatar cyeh%bluemartini.com

fix for 71798: bugmail not sent when moving from cc to assigned fields

checked in patch based on patch submitted by jake@acutex.net
parent 2c793d31
...@@ -657,7 +657,6 @@ sub getEmailAttributes ($@) { ...@@ -657,7 +657,6 @@ sub getEmailAttributes ($@) {
my ($commentField,@fieldDiffs) = @_; my ($commentField,@fieldDiffs) = @_;
my (@flags,@uniqueFlags,%alreadySeen) = (); my (@flags,@uniqueFlags,%alreadySeen) = ();
my $Status = 0;
foreach my $ref (@fieldDiffs) { foreach my $ref (@fieldDiffs) {
my ($who, $fieldName, $when, $old, $new) = (@$ref); my ($who, $fieldName, $when, $old, $new) = (@$ref);
...@@ -671,7 +670,6 @@ sub getEmailAttributes ($@) { ...@@ -671,7 +670,6 @@ sub getEmailAttributes ($@) {
if ($new eq 'RESOLVED' || $new eq 'VERIFIED') { if ($new eq 'RESOLVED' || $new eq 'VERIFIED') {
push (@flags, 'Resolved'); push (@flags, 'Resolved');
} }
$Status = 1;
} }
elsif ( $fieldName eq 'Severity' || $fieldName eq 'Status' || elsif ( $fieldName eq 'Severity' || $fieldName eq 'Status' ||
$fieldName eq 'Priority' ) { $fieldName eq 'Priority' ) {
...@@ -686,7 +684,7 @@ sub getEmailAttributes ($@) { ...@@ -686,7 +684,7 @@ sub getEmailAttributes ($@) {
if ( $commentField =~ /Created an attachment \(/ ) { if ( $commentField =~ /Created an attachment \(/ ) {
push (@flags, 'Attachments'); push (@flags, 'Attachments');
} }
elsif ( $commentField ne '' && !($Status)) { elsif ( ($commentField ne '') && (scalar(@flags) == 1) && ($flags[0] eq 'Resolved')) {
push (@flags, 'Comments'); push (@flags, 'Comments');
} }
......
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