diff options
author | Brion Vibber <brion@pobox.com> | 2010-03-15 15:08:16 -0700 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-03-15 16:08:00 -0700 |
commit | 441e52718e4db4eb45bd5c76c5af446496f56f96 (patch) | |
tree | cc64045cb8368ecdf05ab8dcc87ef252965c3805 /actions/deleteuser.php | |
parent | 9ec24f59ca61bbbb45667b548a872e724f31ab3e (diff) |
Background deletion of user accounts. Notices are deleted in chunks, then the user itself when they're all gone.
While deletion is in progress, the account is locked with the 'deleted' role, which disables all actions with rights control.
Todo:
* Pretty up the notice on the profile page about the pending delete. Show status?
* Possibly more thorough account disabling, such as disallowing all use for login and access.
* Improve error recovery; worst case is that an account gets left locked in 'deleted' state but the queue jobs have gotten dropped out. This would leave the username in use and any undeleted notices in place.
Diffstat (limited to 'actions/deleteuser.php')
-rw-r--r-- | actions/deleteuser.php | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/actions/deleteuser.php b/actions/deleteuser.php index c4f84fad2..4e6b27395 100644 --- a/actions/deleteuser.php +++ b/actions/deleteuser.php @@ -162,7 +162,15 @@ class DeleteuserAction extends ProfileFormAction function handlePost() { if (Event::handle('StartDeleteUser', array($this, $this->user))) { - $this->user->delete(); + // Mark the account as deleted and shove low-level deletion tasks + // to background queues. Removing a lot of posts can take a while... + if (!$this->user->hasRole(Profile_role::DELETED)) { + $this->user->grantRole(Profile_role::DELETED); + } + + $qm = QueueManager::get(); + $qm->enqueue($this->user, 'deluser'); + Event::handle('EndDeleteUser', array($this, $this->user)); } } |