Hook Docs: wp-admin/my-sites.php

[megane9988]

Attached patch has done for the all hooks in wp-admin/my-sites.php.

[megane9988]

patch of my-sites.php

[megane9988]

removed extra tab in line 70

[megane9988]

fixed comment spacing

[kpdesign]

@megane9988: Thanks for the patch. :)

Just a few things:

myblogs_allblogs_options

• "Fires after displays the selectbox for choose primary blog." should be "Fires after displaying the Choose Primary Site dropdown.".
• Add a blank line between description and @since tag, per the ​standard.
• Remove extra whitespace after opening and closing tags for that block.

myblogs_options

• Description should be "Add settings to the Edit Site Settings screen".
• Remove extra whitespace after * on blank lines.

myblogs_blog_actions

• Description should be "Filter the displayed links for each site."
• Remove extra whitespace after * on blank lines.
• Remove extra tabs at the end of the apply_filters line.

Second instance of myblogs_options (line 80) should be //duplicate_hook instead of the full doc block.

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

[megane9988]

fix kpdesign comment part

[megane9988]

Remove extra whitespace after * on blank lines.

[DrewAPicture]

Thanks for the updated patch.

We'll review the patch and get back to you shortly with any notes.

[kpdesign]

Second pass on wp-sites.5.diff

[kpdesign]

Add long description for myblogs_options filter, other minor fixes

[DrewAPicture]

25586.7.diff​ looks good. Thanks for the extra effort.

[DrewAPicture]

In 26284:

Inline documentation for hooks in wp-admin/my-sites.php.

Props megane9988, kpdesign.
Fixes #25586.