From 0e304107677cbcd9aa73500d3e5f5fda692a260e Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Tue, 1 Mar 2011 10:33:00 -0600
Subject: Vastly simplify pkg_delete function

Since only TUs/Devs can delete packages, we can remove almost all checks
except the account type check. And now that our DB uses foreign keys,
all of the other deletes happen implicitly when a package is deleted so
we don't need to take care of it here.

Signed-off-by: Dan McGee <dan@archlinux.org>
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
---
 web/lib/pkgfuncs.inc | 66 +++++++---------------------------------------------
 1 file changed, 8 insertions(+), 58 deletions(-)

(limited to 'web/lib')

diff --git a/web/lib/pkgfuncs.inc b/web/lib/pkgfuncs.inc
index a0cd269..20e3880 100644
--- a/web/lib/pkgfuncs.inc
+++ b/web/lib/pkgfuncs.inc
@@ -700,71 +700,21 @@ function pkg_flag ($atype, $ids, $action = True) {
  */
 function pkg_delete ($atype, $ids) {
 	if (!$atype) {
-		return __("You must be logged in before you can disown packages.");
+		return __("You must be logged in before you can delete packages.");
 	}
 
-	if (empty($ids)) {
-		return __("You did not select any packages to delete.");
-	}
-
-	# Delete the packages in $ids array (but only if they are Unsupported)
-	#
-	$dbh = db_connect();
-
-	# Delete the packages in $ids array
-	#
-	$first = 1;
-	foreach ($ids as $pid) {
-		if ($first) {
-			$first = 0;
-			$delete = $pid;
-		} else {
-			$delete .= ", ".$pid;
-		}
-	}
-
-	$field = "MaintainerUID";
-
-	# Only grab Unsupported packages that "we" own or are not owned at all
-	$ids_to_delete = array();
-	$q = "SELECT Packages.ID FROM Packages ";
-	$q.= "WHERE Packages.ID IN (" . $delete . ") ";
-
 	# If they're a TU or dev, can delete
-	if ($atype == "Trusted User" || $atype == "Developer") {
-		$result = db_query($q, $dbh);
+	if ($atype != "Trusted User" && $atype != "Developer") {
+		return __("You do have permission to delete packages.");
 	}
 
-	if ($result != Null && mysql_num_rows($result) > 0) {
-		while ($row = mysql_fetch_assoc($result)) {
-			$ids_to_delete[] = $row['ID'];
-		}
-	}
-
-	if (empty($ids_to_delete)) {
-		return __("None of the selected packages could be deleted.");
+	if (empty($ids)) {
+		return __("You did not select any packages to delete.");
 	}
 
-	# These are the packages that are safe to delete
-	foreach ($ids_to_delete as $id) {
-		$q = "DELETE FROM PackageVotes WHERE PackageID = " . $id;
-		$result = db_query($q, $dbh);
-
-		$q = "DELETE FROM PackageDepends WHERE PackageID = " . $id;
-		$result = db_query($q, $dbh);
-
-		$q = "DELETE FROM PackageSources WHERE PackageID = " . $id;
-		$result = db_query($q, $dbh);
-
-		$q = "DELETE FROM PackageComments WHERE PackageID = " . $id;
-		$result = db_query($q, $dbh);
-
-		$q = "DELETE FROM Packages WHERE ID = " . $id;
-		$result = db_query($q, $dbh);
-
-		$q = "DELETE FROM CommentNotify WHERE PkgID = " . $id;
-		$result = db_query($q, $dbh);
-	}
+	$dbh = db_connect();
+	$q = "DELETE FROM Packages WHERE ID IN (" . implode(",", $ids) . ")";
+	$result = db_query($q, $dbh);
 
 	return __("The selected packages have been deleted.");
 }
-- 
cgit v1.2.3-70-g09d2