[wp-trac] [WordPress Trac] #32796: User deletion API is inconsistent between MS and non-MS

WordPress Trac noreply at wordpress.org
Sun Jul 5 03:25:23 UTC 2015


#32796: User deletion API is inconsistent between MS and non-MS
----------------------------------------+-----------------------------
 Reporter:  boonebgorges                |       Owner:
     Type:  enhancement                 |      Status:  new
 Priority:  normal                      |   Milestone:  Future Release
Component:  Users                       |     Version:
 Severity:  normal                      |  Resolution:
 Keywords:  needs-unit-tests has-patch  |     Focuses:  multisite
----------------------------------------+-----------------------------
Changes (by jeremyfelt):

 * keywords:  needs-patch needs-unit-tests => needs-unit-tests has-patch


Comment:

 [https://core.trac.wordpress.org/attachment/ticket/32796/32796.diff
 32796.diff] is a first attempt at a merge of `wpmu_delete_user()` into
 `wp_delete_user()`.

 It's not as bad as I thought it would be so far, though a full suite of
 tests to go along with this will be nice. I have not done any extensive
 testing, but it passes our current (limited) tests and I'm able to do
 basic things as expected.

 Notes:
 * I've added a `$context` parameter to `wp_delete_user()` so that we can
 determine if a user is being deleted from a network or from an individual
 site on the network. Single site should always be in the site context.
 * It's worth looking at #12720 as part of this ticket. @danielbachhuber's
 [https://core.trac.wordpress.org/ticket/12720#comment:5 comment about
 explicit core APIs] makes sense. There could be room for something like
 `wp_reassign_user_content()` and `wp_delete_user_content()` here.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/32796#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list