[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