Closed Bug 754981 Opened 12 years ago Closed 12 years ago

There is not a plain link to a bug (i.e. to the top of the bug) in html bugmail

Categories

(Bugzilla :: Email Notifications, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 4.2

People

(Reporter: mrbball, Assigned: koosha.khajeh)

Details

Attachments

(1 file, 1 obsolete file)

When only a comment is added to a bug (i.e. no fields are changed) the
html email that is generated does not contain a hyperlink to the top of
the bug (i.e. http://.../show_bug.cgi?id=XXX).

Instead, the bugmail has a link to the specific comment that was added
(i.e. http://.../show_bug.cgi?id=XXX#cYYY).

The link to the comment is not all that useful, since I already have the
text of the comment in my mailer.  I usually want to go to the top of
the bug, not the specific comment.  In general, this is annoying, but
when a bug has dozens or even hundreds of comments and the hyperlink
takes me to the bottom, it's quite sub-optimal.

I guess a link to the specific comment doesn't hurt, but there should
certainly be a link to the bug itself.
The plain text email has a link pointing to the bug itself, not its comment, but the HTML format doesn't. Not sure we will fix it for 4.2.x, though.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Whiteboard: [Good Intro Bug]
Target Milestone: --- → Bugzilla 4.4
Attached patch patch - v1 (obsolete) — Splinter Review
Attachment #624686 - Flags: review?(LpSolit)
Attached patch patch - v1.0.1Splinter Review
Attachment #624686 - Attachment is obsolete: true
Attachment #624686 - Flags: review?(LpSolit)
Attachment #624688 - Flags: review?(LpSolit)
Comment on attachment 624688 [details] [diff] [review]
patch - v1.0.1

>+              on [% "${terms.Bug} # ${bug.id}" FILTER bug_link( bug, { full_url => 1 }) %]

"Bug" should be lowercase, and we don't want # before the bug ID. Also, |FILTER none| is required, else it doesn't pass tests.

Otherwise looks good, and I think this is a good compromise to fix this bug. r=LpSolit
Attachment #624688 - Flags: review?(LpSolit) → review+
Assignee: email-notifications → koosha.khajeh
Severity: normal → minor
Status: NEW → ASSIGNED
Flags: approval+
Whiteboard: [Good Intro Bug]
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified template/en/default/email/bugmail.html.tmpl
Committed revision 8240.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Flags: approval4.2?
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.2/
modified template/en/default/email/bugmail.html.tmpl
Committed revision 8089.
Flags: approval4.2? → approval4.2+
Target Milestone: Bugzilla 4.4 → Bugzilla 4.2
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: