#26924 closed defect (bug) (fixed)
Misleading hook docs for action 'print_media_templates'
| Reported by: | ericlewis | Owned by: | DrewAPicture |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Script Loader | Version: | 3.8 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | docs |
Description
Attachments (2)
Change History (8)
#3
in reply to: ↑ 1
@
13 years ago
- Keywords needs-patch added
Replying to ericlewis:
In attachment:26924.diff, be clear that this hook doesn't actually do anything; it hits after the media manager templates have been printed.
I agree that the terminology of "Prints" is a bit misleading, though it has also been leveraged with other 'print_*' hooks in core. The hooks don't do anything themselves.
Perhaps 'Fires when xxxx are printed' would be better.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
In attachment:26924.diff, be clear that this hook doesn't actually do anything; it hits after the media manager templates have been printed.