Opened 9 years ago
Closed 9 years ago
#35063 closed defect (bug) (fixed)
user deletion form hook docs incorrect and could use ids of users to be deleted
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | 4.0 |
Component: | Users | Keywords: | has-patch |
Focuses: | docs, administration | Cc: |
Description
When deleting users in the wp admin, the user deletion form fires this action:
/** * Fires at the end of the delete users form prior to the confirm button. * * @since 4.0.0 * * @param WP_User $current_user WP_User object for the user being deleted. */ do_action( 'delete_user_form', $current_user );
the @param docs are wrong... the $current_user is not the user object for the user being deleted - its the user doing the deletion...
which actually is kind of pointless to pass... what really needs to be passed is the list of user ids to be deleted...
purpose of this ticket and patch is to fix the docs and add another argument to the hook...
Attachments (2)
Change History (6)
#1
@
9 years ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Future Release
- Version changed from 4.4 to 4.0
#2
@
9 years ago
- Keywords has-patch added; needs-patch removed
yeah, didnt replace the $current_user since might break backward compat and just added the new arg...
patch updated to include similar network change...
Note: See
TracTickets for help on using
tickets.
patch against trunk