Hooks Docs: wp-admin/network/site-users.php

[sboisvert]

Patch for hooks inside wp-admin/network/site-users.php

[DrewAPicture]

Hi, thanks for the patch. Some notes on site-users.php.patch​:

• All short descriptions need a period.

show_network_site_users_add_existing_form filter:

• The suggested language for a boolean-style filter is to say something like, "Whether to <perform something>". It's also necessary to provide the context, in this case that would be Multisite. So maybe change "Disable showing add existing user form" to something like "Whether to show the Add Existing User form in the Site Users screen."

network_site_users_after_list_table hook:

• Remove the blank line after the @since line.
• The boolean parameter with a value of true is missing from the do_action() call
• Add a parameter doc for the missing boolean parameter

[kpdesign]

Second pass

[kpdesign]

Fixed doc block for show_network_site_users_add_new_form

[sboisvert]

Thanks kpdesign and DrewAPicture.
I'm really sorry I didn't get a chance to clean this up. I'll try to get a few more in, hopefully each better than the last in the coming week.

[DrewAPicture]

Clean up

[DrewAPicture]

25497.diff​ looks good to me. @kpdesign?

[kpdesign]

25497.diff​ looks good to me. Recommend commit.

[DrewAPicture]

In 25939:

Inline documentation for hooks in wp-admin/network/site-users.php.

Props sboisvert for the initial patch. Props kpdesign.
Fixes #25497.