From c9806c3558e6a37c748488520d0c83e471809104 Mon Sep 17 00:00:00 2001 From: "mkanat%bugzilla.org" <> Date: Wed, 19 Aug 2009 04:44:52 +0000 Subject: [PATCH] Bug 73330: Remove extra newlines in bugmail Patch by Max Kanat-Alexander r=LpSolit, a=LpSolit --- Bugzilla/BugMail.pm | 4 +++- template/en/default/email/newchangedmail.txt.tmpl | 2 ++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm index 9a1b2a8a25..ec4bf30dae 100644 --- a/Bugzilla/BugMail.pm +++ b/Bugzilla/BugMail.pm @@ -352,7 +352,7 @@ sub Send { if ($deptext) { my $diffpart = {}; - $diffpart->{'text'} = "\n" . trim("\n\n" . $deptext); + $diffpart->{'text'} = "\n" . trim($deptext); push(@diffparts, $diffpart); } } @@ -595,6 +595,8 @@ sub sendMail { } my $diffs = $difftext; + # Remove extra newlines. + $diffs =~ s/^\n+//s; $diffs =~ s/\n+$//s; if ($isnew) { my $head = ""; foreach my $f (@headerlist) { diff --git a/template/en/default/email/newchangedmail.txt.tmpl b/template/en/default/email/newchangedmail.txt.tmpl index 494c12060f..b90c15b1d2 100644 --- a/template/en/default/email/newchangedmail.txt.tmpl +++ b/template/en/default/email/newchangedmail.txt.tmpl @@ -41,8 +41,10 @@ X-Bugzilla-Changed-Fields: [% changedfields %] [%+ threadingmarker %] [%+ urlbase %]show_bug.cgi?id=[% bugid %] +[%- IF diffs %] [%+ diffs %] +[% END -%] [% FOREACH comment = new_comments %] --- Comment #[% comment.count %] from [% comment.author.identity %] [%+ comment.time FILTER time %] --- -- 2.47.2