Opened 8 years ago
Closed 8 years ago
#45308 closed defect (bug) (fixed)
Use duplicate hook reference syntax for duplicates of 'link_category' and `pre_user_login`
| Reported by: | coffee2code | Owned by: | pento |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | General | Version: | 2.2 |
| Severity: | minor | Keywords: | has-patch |
| Cc: | Focuses: | docs |
Description
link_category and pre_user_login appear to be the last remaining hooks that are defined in more than one place and still uniquely documented in each place. The convention for duplicated hooks is for one instance of the hook to be considered canonical with full documentation while all subsequent instances get documented with a special comment referencing that canonical documentation.
See attached patch for fix.
Related: the hook user_confirmed_action_email_content is also duplicated, but each instance's documentation differs in meaningful ways. See #44314 for discussion on the matter (particularly my pertinent comment).
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Patch mentioned in ticket.