external/aurweb
0
0
Fork 0
mirror of https://gitlab.archlinux.org/archlinux/aurweb.git synced 2025-02-03 10:43:03 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Clean up user references in user_delete()

Browse source 
Explicitly clean up all references before deleting a user.

Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
This commit is contained in:
Lukas Fleischer 2014-07-25 10:28:15 +02:00
parent daceef50eb
commit f4ee1278e5
1 changed files with 35 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
web/lib/acctfuncs.inc.php
View file

@ -829,6 +829,41 @@ function user_suspended($id) {
*/ */
function user_delete($id) { function user_delete($id) {
$dbh = DB::connect(); $dbh = DB::connect();
$id = intval($id);
/*
* These are normally already taken care of by propagation constraints
* but it is better to be explicit here.
*/
$fields_delete = array(
array("Sessions", "UsersID"),
array("PackageVotes", "UsersID"),
array("CommentNotify", "UsersID")
);
$fields_set_null = array(
array("PackageBases", "SubmitterUID"),
array("PackageBases", "MaintainerUID"),
array("PackageBases", "SubmitterUID"),
array("PackageComments", "UsersID"),
array("PackageComments", "DelUsersID"),
array("PackageRequests", "UsersID"),
array("TU_VoteInfo", "SubmitterID"),
array("TU_Votes", "UserID")
);
foreach($fields_delete as list($table, $field)) {
$q = "DELETE FROM " . $table . " ";
$q.= "WHERE " . $field . " = " . $id;
$dbh->query($q);
}
foreach($fields_set_null as list($table, $field)) {
$q = "UPDATE " . $table . " SET " . $field . " = NULL ";
$q.= "WHERE " . $field . " = " . $id;
$dbh->query($q);
}
$q = "DELETE FROM Users WHERE ID = " . $id; $q = "DELETE FROM Users WHERE ID = " . $id;
$dbh->query($q); $dbh->query($q);
return; return;