[wp-trac] [WordPress Trac] #25531: Hook Docs: wp-includes/capabilities.php

[WordPress Trac]

#25531: Hook Docs: wp-includes/capabilities.php
-----------------------------------+------------------------------
 Reporter:  ptahdunbar             |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  normal                 |   Milestone:  Awaiting Review
Component:  Inline Docs            |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |
-----------------------------------+------------------------------
Changes (by rzen):

 * keywords:  has-patch => has-patch 2nd-opinion


Comment:

 I'd like to see what others have to say about my recommendations for
 `set_user_role`, `user_has_cap` and `auth_post_meta_{$meta_key}` in
 regards to naming the params different from the actual variables being
 used in the filter call.

 My feeling is that it makes the docs more human-readable when separated
 from the context of the code, with the obvious trade-off that the pairing
 becomes awkward when viewed in code (and possibly within an IDE). This may
 be a good topic to discuss during a weekly Inline Docs meeting.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/25531#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software