[wp-trac] [WordPress Trac] #22380: Consolidate the API hooks comment
WordPress Trac
noreply at wordpress.org
Fri Feb 21 18:04:31 UTC 2014
#22380: Consolidate the API hooks comment
-------------------------------------------------+-------------------------
Reporter: momo360modena | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting
Component: Comments | Review
Severity: minor | Version: 3.4
Keywords: has-patch needs-testing needs- | Resolution:
refresh | Focuses:
-------------------------------------------------+-------------------------
Changes (by jorbin):
* keywords: has-patch needs-testing => has-patch needs-testing needs-
refresh
* type: defect (bug) => enhancement
Comment:
Thanks for the suggestion momo360modena. Sorry you never got a response.
I like adding additional context to many of these filters. It definitely
makes these filters more flexible and opens up a number of possibilities.
Perhaps though it make sense to be consistent with all of them and only
include $comment_ID for all of them rather then sometimes including both
$comment_ID and $comment.
When refreshing, some of these now have hook docs (which didn't exist when
this patch was originally written), so please make sure to add the new
@param value.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/22380#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list