Closed
Bug 578003
Opened 14 years ago
Closed 14 years ago
E-mail notifications are missing datetime for comments
Categories
(Bugzilla :: Email Notifications, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.6
People
(Reporter: reed, Assigned: LpSolit)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
glob
:
review+
|
Details | Diff | Splinter Review |
The "comment from" lines in e-mail notifications are missing their datetime (it's blank).
Example:
--- Comment #5 from Reed Loden [:reed] <reed@reedloden.com> ---
LpSolit said he was seeing this upstream, as well.
Flags: blocking4.0?
Flags: blocking3.6.2?
Reporter | ||
Updated•14 years ago
|
Keywords: regression
Assignee | ||
Comment 1•14 years ago
|
||
Note that I'm personally in favor of having the date going away definitely as this information is already in the Date: header of bugmails.
Assignee | ||
Comment 2•14 years ago
|
||
The method to get the time when a comment was added is creation_ts(), not time().
Assignee: email-notifications → LpSolit
Status: NEW → ASSIGNED
Attachment #456825 -
Flags: review?(bugzilla)
Assignee | ||
Comment 3•14 years ago
|
||
This is a regression due to bug 472217, which changed $comment->{'time'} to $comment->creation_ts (internally: $comment->{'bug_when'}). So this affects 3.6 and newer only.
Severity: normal → minor
Depends on: 472217
Comment on attachment 456825 [details] [diff] [review]
patch, v1
r=glob
Attachment #456825 -
Flags: review?(bugzilla) → review+
Reporter | ||
Updated•14 years ago
|
Flags: approval4.0?
Flags: approval3.6?
Comment 5•14 years ago
|
||
However, I do agree with LpSolit--for any branch where reed's fix went in that makes the Date header reflect delta_ts, it's probably unnecessary to include dates in the comment headers. The only case that this would be useful for is if there are multiple comments in one email, but that is a pretty rare case.
Flags: blocking4.0?
Flags: blocking4.0+
Flags: blocking3.6.2?
Flags: blocking3.6.2+
Flags: approval?
Flags: approval4.0?
Flags: approval4.0+
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
Comment 6•14 years ago
|
||
(The non-inclusion of Date headers should be a separate bug though.)
Assignee | ||
Comment 7•14 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified template/en/default/email/newchangedmail.txt.tmpl
Committed revision 7329.
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.0/
modified template/en/default/email/newchangedmail.txt.tmpl
Committed revision 7307.
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/3.6/
modified template/en/default/email/newchangedmail.txt.tmpl
Committed revision 7132.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•