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 <[email protected]>
---
 web/lib/pkgfuncs.inc.php     |    3 ---
 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 6e86470..4243991 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
-- 
1.7.6

Reply via email to