Closed Bug 1100942 Opened 5 years ago Closed 5 years ago

Attachment links in request.cgi should go to the attachment and not default to &action=edit

Categories

(Bugzilla :: Attachments & Requests, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
Bugzilla 6.0

People

(Reporter: ted, Assigned: dkl)

References

Details

Attachments

(1 file)

I go through my review queue using request.cgi. The attachment links on there all link to the attachment with &action=edit in the URL. For patch attachments there's also a [Review] link that takes me to the Splinter view, which is what I've historically used. In this modern era of ReviewBoard and GitHub issues, however, that doesn't exist, and clicking the attachment link is not useful. It would be nice if those could just link directly to the RB/GH issue.
Lets work this upstream as I think it would be useful there as well to be able to differentiate the two actions.

I propose that when you click on the attachment id, it should take you to the raw attachment similar to what happens on show_bug.cgi for attachments. Then add a [details] link next to it that will take you to the action=edit page.

Just as an added note related to BMO, after this patch is implemented, when clicking on the attachment link, if the attachment data is a reviewboard URL, it will automatically redirect. Which will solve the original request with this bug.

dkl
Assignee: nobody → dkl
Status: NEW → ASSIGNED
Component: User Interface → Attachments & Requests
Product: bugzilla.mozilla.org → Bugzilla
QA Contact: default-qa
Summary: Attachment links in request.cgi default to &action=edit, not great for ReviewBoard/GitHub issues → Attachment links in request.cgi should go to the attachment and not default to &action=edit
Version: Production → unspecified
Attached patch 1100942_1.patchSplinter Review
Attachment #8524763 - Flags: review?(gerv)
Comment on attachment 8524763 [details] [diff] [review]
1100942_1.patch

Review of attachment 8524763 [details] [diff] [review]:
-----------------------------------------------------------------

r=gerv.

Gerv
Attachment #8524763 - Flags: review?(gerv) → review+
Flags: approval?
Severity: normal → enhancement
Flags: approval? → approval+
Target Milestone: --- → Bugzilla 6.0
To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git
   47da91d..259615d  master -> master
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
See Also: → 1158117
You need to log in before you can comment on or make changes to this bug.