#57808 closed defect (bug) (fixed)
Correct duplicate hook reference for 'notify_moderator'
| Reported by: | SergeyBiryukov | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.2 |
| Component: | Comments | Version: | |
| Severity: | normal | Keywords: | good-first-bug has-patch |
| Cc: | Focuses: | docs |
Description
Per the documentation standards, duplicate hook instances should be marked as such:
/** This filter is documented in path/to/filename.php */
The instance of the notify_moderator filter in wp_new_comment_notify_moderator() is marked with:
/** This filter is documented in wp-includes/comment.php */
However, the original filter location is wp_notify_moderator() in wp-includes/pluggable.php, so the reference in wp_new_comment_notify_moderator() should be corrected to:
/** This filter is documented in wp-includes/pluggable.php */
Change History (4)
This ticket was mentioned in PR #4136 on WordPress/wordpress-develop by @zevilz.
4 years ago
#1
- Keywords has-patch added; needs-patch removed
@SergeyBiryukov commented on PR #4136:
4 years ago
#4
Thanks for the PR! Merged in r55424.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
See https://core.trac.wordpress.org/ticket/57808