summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-08-11 16:04:01 +0200
committerLukas Fleischer <archlinux@cryptocrack.de>2011-08-11 16:17:09 +0200
commit45a50d4a66f5e57d260109e1f332ee7d06183d46 (patch)
treef328c090e1a1ff6d8618b080704a33bdf0cddf29
parent34fe0bec4cd29d1f8bcdfb34fb810556bfd939b2 (diff)
downloadaur-45a50d4a66f5e57d260109e1f332ee7d06183d46.tar.gz
aur-45a50d4a66f5e57d260109e1f332ee7d06183d46.tar.xz
Remove unnecessary atype and uid lookup from package_details()
These were never used in the function. Where they are used is in the pkg_details.php template, so move them closer to their actual usage so as not to confuse poor programmers such as myself. Signed-off-by: Dan McGee <dan@archlinux.org> Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
-rw-r--r--web/lib/pkgfuncs.inc.php3
-rw-r--r--web/template/pkg_details.php2
2 files changed, 2 insertions, 3 deletions
diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php
index 7dc0ae64..3e89fa35 100644
--- a/web/lib/pkgfuncs.inc.php
+++ b/web/lib/pkgfuncs.inc.php
@@ -301,9 +301,6 @@ function pkgname_is_blacklisted($name, $dbh=NULL) {
# display package details
#
function package_details($id=0, $SID="", $dbh=NULL) {
- $atype = account_from_sid($SID);
- $uid = uid_from_sid($SID);
-
if(!$dbh) {
$dbh = db_connect();
}
diff --git a/web/template/pkg_details.php b/web/template/pkg_details.php
index 52391232..c03493d8 100644
--- a/web/template/pkg_details.php
+++ b/web/template/pkg_details.php
@@ -1,4 +1,6 @@
<?php
+$atype = account_from_sid($SID);
+$uid = uid_from_sid($SID);
$pkgid = intval($_REQUEST['ID']);
if ($uid == $row["MaintainerUID"] or