Hooks Docs: wp-admin/user-edit.php & 2 more files

[admiralthrawn]

The files have documentation for:

wp-includes/ms-default-filters.php
wp-admin/user-edit.php
wp-admin/user-new.php

[DrewAPicture]

Would you mind regenerating your patch? It's coming through as malformed.

[DrewAPicture]

Full pass

[DrewAPicture]

25726.2.diff​ is a full pass through wp-admin/user-new.php and has some fixes for wp-admin/user-edit.php.

This should be good to go after a secondary review and recommendation.

[kpdesign]

@DrewAPicture: Thanks for the updated patch (25726.2.diff​​). Just a few changes needed, and we should be good:

user-new.php:

pre_user_login:

• @since should be 2.0.3

autocomplete_users_for_site_admins:

• @param description: s/Default is false./Default false.

user_new_form_tag:

• Space out the action line (both instances).

show_password_fields:

• @param description: s/Default is true./Default true.

user-edit.php:

personal_options_update:

• @since should be 2.0.0

admin_color_scheme_picker:

• Replace 6 with do_action in action line.

additional_capabilities_display:

• @since should be 2.8.0
• @param for $enable - add "Default true." at end of description.

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

[DrewAPicture]

Final pass

[DrewAPicture]

25726.3.diff​ applies the suggested changes in comment:4. I think that should do it.

[SergeyBiryukov]

pre_user_login should probably be documented in wp-includes/user.php, along with other pre_user_* filters: tags/3.7.1/src/wp-includes/user.php#L1325​.

[DrewAPicture]

In 26493:

Inline documentation for hooks in wp-admin/user-new.php & wp-admin/user-edit.php.

Also fixes one parameter type in wp-includes/user.php.

Fixes #25726.

[DrewAPicture]

Replying to SergeyBiryukov:

pre_user_login should probably be documented in wp-includes/user.php, along with other pre_user_* filters: tags/3.7.1/src/wp-includes/user.php#L1325​.

Sorry, didn't see this comment before commit. I'll take a look a little later and adjust accordingly. Thanks :)