diff options
author | Lukas Fleischer <archlinux@cryptocrack.de> | 2014-04-06 18:16:16 +0200 |
---|---|---|
committer | Lukas Fleischer <archlinux@cryptocrack.de> | 2014-04-06 18:42:00 +0200 |
commit | 32b5d466439b5c291472c68f6a732c5be4ca60cf (patch) | |
tree | aeb811d09f19607fb37429026e5415d396430fbd /web/lib | |
parent | 5b5a5f343e6232c9237f785f6763018cbb048fbc (diff) |
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/lib')
-rw-r--r-- | web/lib/pkgbasefuncs.inc.php | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/web/lib/pkgbasefuncs.inc.php b/web/lib/pkgbasefuncs.inc.php index 4e026f5..4eaa640 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; |