New Action On User Delete Form

[usermrpapa]

When deleting a user, it would be nice to have an action fired before the end of the form in order for plugins to add additional questions/options to the form before a user is deleted.

[usermrpapa]

patch for delete user form action

[SergeyBiryukov]

If we add a hook in there, it should also be added to confirm_delete_users() for Multisite: tags/3.8.1/src/wp-admin/network/users.php#L19​.

It also needs to be documented: ​http://make.wordpress.org/core/handbook/inline-documentation-standards/php-documentation-standards/#4-hooks-actions-and-filters.

[usermrpapa]

new patch for ms too... unsure if it should be same hook name or different... left it same...

can update docs too once its committed...

[DrewAPicture]

Replying to usermrpapa:

new patch for ms too... unsure if it should be same hook name or different... left it same...

can update docs too once its committed...

Actually, the hook docs need to be included for the patch to even be considered for commit :)

If you need assistance writing the hook docs, I can provide feedback for you.

[usermrpapa]

doh... got it - was thinking codex...

how's this?

[usermrpapa]

patch refresh for latest trunk changes

[usermrpapa]

patch refresh for 4.0

[usermrpapa]

its 4.0 early... any chance of getting this in? ;)

[SergeyBiryukov]

• Please use tabs rather than spaces for indentation (per ​WordPress Coding Standards).
• @since should use the 3-digit x.x.x style.

[usermrpapa]

hmmm... not sure why they wouldnt be tabs... please try this one... and used 3 digit version notation... thanks..

[kpdesign]

@usermrpapa: Thanks for the updated patch.

You will need to include the ​documentation for the parameter $current_user in the doc block in wp-admin/users.php.

The second instance of the hook in wp-admin/network/users.php should be marked as a ​duplicate hook.

I'd also suggest shortening the description to "Fires at the end of the delete users form prior to the confirm button."

[usermrpapa]

okay, lets try this one...

[samuelsidler]

Anything else needed here?

[usermrpapa]

hope not, but dont think you were asking me... :)

[SergeyBiryukov]

Fixed formatting

[wonderboymusic]

In 28885:

Add new action 'delete_user_form' in wp-admin/users.php and wp-admin/network/users.php.

Props SergeyBiryukov, usermrpapa.
Fixes #27230.