new bug notification shouldn't include blank fields

RESOLVED FIXED in Bugzilla 2.14

Status

()

Bugzilla
Bugzilla-General
P2
enhancement
RESOLVED FIXED
17 years ago
5 years ago

People

(Reporter: timeless, Assigned: Jacob Steenhagen)

Tracking

unspecified
Bugzilla 2.14
x86
Windows 2000

Details

(Whiteboard: [please commit asap])

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
I just noticed this when the CC: field was added, but this applies to all 
fields.

it's a waste of bytes, and reading time to include blank fields. so let's fix 
the mailer to stop doing it.
(Assignee)

Comment 1

17 years ago
Created attachment 38297 [details] [diff] [review]
patch
(Assignee)

Updated

17 years ago
Assignee: tara → jake
Priority: -- → P2
Target Milestone: --- → Bugzilla 2.16
(Assignee)

Comment 2

17 years ago
Easy enough to do, but it's not security so it probably won't go in 2.14.  Of
course, if I get an r= sooner, there really isn't any reason it can't go in.
(Reporter)

Comment 3

17 years ago
r=timeless
Whiteboard: [please commit asap]
(Assignee)

Comment 4

17 years ago
Committed.
Obviously it won't show up unitl b.m.o updates to the tip again.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Target Milestone: Bugzilla 2.16 → Bugzilla 2.14
Moving to Bugzilla product
Component: Bugzilla → Bugzilla-General
Product: Webtools → Bugzilla
Version: Bugzilla 2.13 → unspecified
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.