[wp-trac] [WordPress Trac] #26099: Hook Docs: wp-admin/includes/user.php

WordPress Trac noreply at wordpress.org
Sun Dec 1 03:55:34 UTC 2013


#26099: Hook Docs: wp-admin/includes/user.php
---------------------------+---------------------------
 Reporter:  aaronholbrook  |       Owner:  DrewAPicture
     Type:  defect (bug)   |      Status:  reviewing
 Priority:  normal         |   Milestone:  3.8
Component:  Inline Docs    |     Version:
 Severity:  normal         |  Resolution:
 Keywords:  needs-patch    |
---------------------------+---------------------------
Changes (by kpdesign):

 * keywords:  has-patch => needs-patch


Comment:

 @DrewAPicture: Thanks for the updated patch ([attachment:26099.2.patch]​).

 Just a few changes needed:

 `check_passwords`:
 - @since should be 1.5.1

 `editable_roles`:
 - Variable in @param line should be $all_roles.

 `delete_user`:
 - This is not a duplicate hook.
 - @since is 2.0.0
 - Space out the action line please.

 `deleted_user`:
 - Should be: `/** This action is documented in wp-admin/includes/ms.php
 */`
 - Space out the action line please.

 Could you make these changes and submit a new patch please? :)

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


More information about the wp-trac mailing list