summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--Bugzilla/BugMail.pm13
-rw-r--r--template/en/default/email/newchangedmail.txt.tmpl2
2 files changed, 7 insertions, 8 deletions
diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm
index 539a481ed..d29ffaf1e 100644
--- a/Bugzilla/BugMail.pm
+++ b/Bugzilla/BugMail.pm
@@ -500,7 +500,7 @@ sub Send {
\@diffparts,
$comments{$lang},
$anyprivate,
- $start,
+ ! $start,
$id,
exists $watching{$user_id} ?
$watching{$user_id} : undef);
@@ -522,8 +522,8 @@ sub Send {
}
sub sendMail {
- my ($user, $hlRef, $relRef, $valueRef, $dmhRef, $fdRef,
- $diffRef, $newcomments, $anyprivate, $start,
+ my ($user, $hlRef, $relRef, $valueRef, $dmhRef, $fdRef,
+ $diffRef, $newcomments, $anyprivate, $isnew,
$id, $watchingRef) = @_;
my %values = %$valueRef;
@@ -590,8 +590,6 @@ sub sendMail {
return 0;
}
- my $isnew = !$start;
-
# If an attachment was created, then add an URL. (Note: the 'g'lobal
# replace should work with comments with multiple attachments.)
@@ -629,13 +627,14 @@ sub sendMail {
my $threadingmarker;
if ($isnew) {
$threadingmarker = "Message-ID: <bug-$id-" . $user->id . "$sitespec>";
- } else {
+ }
+ else {
$threadingmarker = "In-Reply-To: <bug-$id-" . $user->id . "$sitespec>";
}
my $vars = {
- neworchanged => $isnew ? 'New: ' : '',
+ isnew => $isnew,
to => $user->email,
bugid => $id,
alias => Bugzilla->params->{'usebugaliases'} ? $values{'alias'} : "",
diff --git a/template/en/default/email/newchangedmail.txt.tmpl b/template/en/default/email/newchangedmail.txt.tmpl
index 93d002d2a..7c0e30a27 100644
--- a/template/en/default/email/newchangedmail.txt.tmpl
+++ b/template/en/default/email/newchangedmail.txt.tmpl
@@ -21,7 +21,7 @@
[% PROCESS "global/variables.none.tmpl" %]
From: [% Param('mailfrom') %]
To: [% to %]
-Subject: [[% terms.Bug %] [%+ bugid %]] [% neworchanged %][%+ summary %]
+Subject: [[% terms.Bug %] [%+ bugid %]] [% 'New: ' IF isnew %][%+ summary %]
X-Bugzilla-Reason: [% reasonsheader %]
X-Bugzilla-Type: newchanged
X-Bugzilla-Watch-Reason: [% reasonswatchheader %]