Need hook for links in common-links

RESOLVED FIXED in Bugzilla 5.0

Status

()

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

People

(Reporter: rojanu, Assigned: rojanu)

Tracking

Bugzilla 5.0
Bug Flags:
approval +

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
common-links need a hook to provide extensions a way to insert links

MyDashboard on bmo uses "[% Hook.process('action-links') %]" link on bmo common-links. This hook could be moved to mainstream bugzilla code
(Assignee)

Comment 1

5 years ago
Created attachment 753735 [details] [diff] [review]
action-links hook
Assignee: general → aliustek
Attachment #753735 - Flags: review?(LpSolit)
Comment on attachment 753735 [details] [diff] [review]
action-links hook

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

r=dkl. We also have this hook in the BMO code already to support the MyDashboard extension so I am sure it works well. This one and the others for
the saved searches, I meant to upstream those but never got to it so you beat me to it :)
Attachment #753735 - Flags: review+

Updated

5 years ago
Flags: approval?

Comment 3

5 years ago
Is this hook really needed in addition to the existing "link-row" hook in the same template? Is the intention to have more actions on the same line as other actions?
Target Milestone: --- → Bugzilla 5.0

Comment 4

5 years ago
(In reply to Frédéric Buclin from comment #3)
> Is this hook really needed in addition to the existing "link-row" hook in
> the same template? Is the intention to have more actions on the same line as
> other actions?

Given that BMO use it, and it could be useful to other installations that also have extensions that might modify action links, I'm happy to a+ this for upstream.
Status: NEW → ASSIGNED
Flags: approval? → approval+

Updated

5 years ago
Attachment #753735 - Flags: review?(LpSolit)
Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bugzilla/trunk
modified template/en/default/global/common-links.html.tmpl
Committed revision 8663.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.