[buddypress-trac] [BuddyPress Trac] #5575: Deleting Users in Manage Signups Leaving Database Entries

[buddypress-trac]

#5575: Deleting Users in Manage Signups Leaving Database Entries
-----------------------------------+--------------------
 Reporter:  aaclayton              |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  high                   |   Milestone:  2.0.1
Component:  Administration         |     Version:  2.0
 Severity:  major                  |  Resolution:
 Keywords:  has-patch 2nd-opinion  |
-----------------------------------+--------------------

Comment (by boonebgorges):

 aaclayton - Thanks very much for the additional debugging. Your analysis
 sounds right.

 imath - Both option1 and option2 seem like hacks to me. The underlying
 issue is that `bp_disable_account_deletion()` is a setting that is
 supposed to limit only front-end, non-admin users, but we're checking it
 in a fairly low-level function that is used in a wide variety of
 instances. 5575.02.patch is, IMO, the correct fix: move the
 `bp_disable_account_deletion()` check into the screen function that
 applies to front-end deletion, removing it from
 `bp_core_delete_account()`. That way, delete_users users have unfettered
 ability to delete accounts (which is how it should be).

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5575#comment:16>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac