diff options
| author | Dan McGee <dan@archlinux.org> | 2011-08-11 09:04:01 -0500 |
|---|---|---|
| committer | Lukas Fleischer <archlinux@cryptocrack.de> | 2011-08-11 16:17:09 +0200 |
| commit | 45a50d4a66f5e57d260109e1f332ee7d06183d46 (patch) | |
| tree | f328c090e1a1ff6d8618b080704a33bdf0cddf29 | |
| parent | 34fe0bec4cd29d1f8bcdfb34fb810556bfd939b2 (diff) | |
| download | aurweb-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.php | 3 | ||||
| -rw-r--r-- | web/template/pkg_details.php | 2 |
2 files changed, 2 insertions, 3 deletions
diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index 7dc0ae6..3e89fa3 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 5239123..c03493d 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 |