Closed Bug 365443 Opened 18 years ago Closed 18 years ago

Bugmail for new bugs without flags contains two unneeded lines

Categories

(Bugzilla :: Email Notifications, defect)

2.23.3
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: reed, Assigned: reed)

Details

(Keywords: regression, ue)

Attachments

(1 file)

Bug 365067 fixed an issue with flags set on initial bug creation not being included in the bugmail, but it also caused an annoying problem. Now, all new bug bugmail that does not contain initial flags has two unneeded lines (\n\n) in it. The extra lines should only be added if the bug has flags to show.
Attached patch patch - v1Splinter Review
Only add the "\n\n" if there is $difftext to display.
Assignee: email-notifications → reed
Status: NEW → ASSIGNED
Attachment #250825 - Flags: review?(LpSolit)
Comment on attachment 250825 [details] [diff] [review]
patch - v1

r=LpSolit
Attachment #250825 - Flags: review?(LpSolit) → review+
Flags: approval+
Checking in Bugzilla/BugMail.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/BugMail.pm,v  <--  BugMail.pm
new revision: 1.102; previous revision: 1.101
done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: