summaryrefslogtreecommitdiffstats
path: root/web
diff options
context:
space:
mode:
authorLukas Fleischer <archlinux@cryptocrack.de>2014-04-06 18:16:16 +0200
committerLukas Fleischer <archlinux@cryptocrack.de>2014-04-06 18:42:00 +0200
commit32b5d466439b5c291472c68f6a732c5be4ca60cf (patch)
treeaeb811d09f19607fb37429026e5415d396430fbd /web
parent5b5a5f343e6232c9237f785f6763018cbb048fbc (diff)
downloadaur-32b5d466439b5c291472c68f6a732c5be4ca60cf.tar.gz
aur-32b5d466439b5c291472c68f6a732c5be4ca60cf.tar.xz
Fix misuses of pkgbase_from_pkgid()
All pkgbase_*() functions should operate on package base IDs. Drop the superfluous (and incorrect) parameter conversion from package IDs to package base IDs. Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
Diffstat (limited to 'web')
-rw-r--r--web/lib/pkgbasefuncs.inc.php11
1 files changed, 6 insertions, 5 deletions
diff --git a/web/lib/pkgbasefuncs.inc.php b/web/lib/pkgbasefuncs.inc.php
index 4e026f53..4eaa6404 100644
--- a/web/lib/pkgbasefuncs.inc.php
+++ b/web/lib/pkgbasefuncs.inc.php
@@ -350,7 +350,7 @@ function pkgbase_flag($atype, $base_ids) {
return array(false, __("You must be logged in before you can flag packages."));
}
- $base_ids = pkgbase_from_pkgid($base_ids);
+ $base_ids = sanitize_ids($base_ids);
if (empty($base_ids)) {
return array(false, __("You did not select any packages to flag."));
}
@@ -406,7 +406,7 @@ function pkgbase_unflag($atype, $base_ids) {
return array(false, __("You must be logged in before you can unflag packages."));
}
- $base_ids = pkgbase_from_pkgid($base_ids);
+ $base_ids = sanitize_ids($base_ids);
if (empty($base_ids)) {
return array(false, __("You did not select any packages to unflag."));
}
@@ -856,13 +856,14 @@ function pkgbase_delete_comment($atype) {
}
/**
- * Change package category
+ * Change package base category
*
+ * @param int Package base ID of the package base to modify
* @param string $atype Account type, output of account_from_sid
*
* @return array Tuple of success/failure indicator and error message
*/
-function pkgbase_change_category($pid, $atype) {
+function pkgbase_change_category($base_id, $atype) {
if (!$atype) {
return array(false, __("You must be logged in before you can edit package information."));
}
@@ -879,7 +880,7 @@ function pkgbase_change_category($pid, $atype) {
return array(false, __("Invalid category ID."));
}
- $base_id = pkgbase_from_pkgid($pid);
+ $base_id = intval($base_id);
/* Verify package ownership. */
$q = "SELECT MaintainerUID FROM PackageBases WHERE ID = " . $base_id;