diff mbox

Unset PackagerUID before deleting an account

Message ID 20170801050300.23783-1-lfleischer@archlinux.org
State Accepted, archived
Headers show

Commit Message

Lukas Fleischer Aug. 1, 2017, 5:03 a.m. UTC
When removing an account, remove the user from all last packager fields
before deletion to make sure that no package bases are deleted, even if
propagation constraints are missing.

Fixes FS#53956.

Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org>
 web/lib/acctfuncs.inc.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox


diff --git a/web/lib/acctfuncs.inc.php b/web/lib/acctfuncs.inc.php
index e45d735..bdcaaa8 100644
--- a/web/lib/acctfuncs.inc.php
+++ b/web/lib/acctfuncs.inc.php
@@ -924,7 +924,7 @@  function user_delete($id) {
 	$fields_set_null = array(
 		array("PackageBases", "SubmitterUID"),
 		array("PackageBases", "MaintainerUID"),
-		array("PackageBases", "SubmitterUID"),
+		array("PackageBases", "PackagerUID"),
 		array("PackageComments", "UsersID"),
 		array("PackageComments", "DelUsersID"),
 		array("PackageRequests", "UsersID"),