#57808 closed defect (bug) (fixed)
Correct duplicate hook reference for 'notify_moderator'
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.2 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Comments | Keywords: | good-first-bug has-patch |
| Focuses: | docs | Cc: |
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.
3 years ago
#1
- Keywords has-patch added; needs-patch removed
#3
@
3 years ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 55424:
@SergeyBiryukov commented on PR #4136:
3 years ago
#4
Thanks for the PR! Merged in r55424.
Note: See
TracTickets for help on using
tickets.
See https://core.trac.wordpress.org/ticket/57808