These were never used in the function, causing at least two totally
unnecessary queries, and potentially two more as each retrieved a DB
handle and would execute the 'SET NAMES...' bit in there.

Signed-off-by: Dan McGee <[email protected]>
---
 web/lib/pkgfuncs.inc.php |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php
index ca6a0f9..65758c8 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();
        }
-- 
1.7.6

Reply via email to