![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 13 years ago
Closed 12 years ago
#27230 closed enhancement (fixed)
New Action On User Delete Form
| Reported by: | usermrpapa | Owned by: | wonderboymusic |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.0 |
| Component: | Users | Version: | 3.8 |
| Severity: | normal | Keywords: | has-patch 4.0-early commit |
| Cc: | Focuses: | administration |
Description
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.
Attachments (8)
Change History (25)
#2
@
13 years ago
- Component Administration → Users
- Focuses administration added
- Version trunk → 3.8
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.
#3
follow-up:
↓ 4
@
13 years ago
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...
#4
in reply to: ↑ 3
@
13 years ago
- Keywords needs-docs added
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.
#10
@
12 years ago
- Milestone Future Release → 4.0
- Please use tabs rather than spaces for indentation (per WordPress Coding Standards).
@sinceshould use the 3-digit x.x.x style.
#11
@
12 years ago
hmmm... not sure why they wouldnt be tabs... please try this one... and used 3 digit version notation... thanks..
#12
@
12 years ago
@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."
patch for delete user form action