From c09dcc4e98ec00329c2db43e47fef32121c76312 Mon Sep 17 00:00:00 2001 From: Lukas Fleischer Date: Thu, 6 Feb 2014 16:11:28 +0100 Subject: Simplify canDeleteComment() Use COUNT(*) instead of COUNT(ID) and PDO::FETCH_NUM instead of PDO::FETCH_ASSOC. This also includes some minor formatting fixes. Signed-off-by: Lukas Fleischer --- web/lib/pkgfuncs.inc.php | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'web/lib/pkgfuncs.inc.php') diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index 05be21c..59cc975 100644 --- a/web/lib/pkgfuncs.inc.php +++ b/web/lib/pkgfuncs.inc.php @@ -22,19 +22,19 @@ function canDeleteComment($comment_id=0, $atype="", $uid=0) { /* TUs and developers can delete any comment. */ return true; } + $dbh = DB::connect(); - $q = "SELECT COUNT(ID) AS CNT "; - $q.= "FROM PackageComments "; - $q.= "WHERE ID = " . intval($comment_id); - $q.= " AND UsersID = " . $uid; + + $q = "SELECT COUNT(*) FROM PackageComments "; + $q.= "WHERE ID = " . intval($comment_id) . " AND UsersID = " . $uid; $result = $dbh->query($q); - if ($result != NULL) { - $row = $result->fetch(PDO::FETCH_ASSOC); - if ($row['CNT'] > 0) { - return true; - } + + if (!$result) { + return false; } - return false; + + $row = $result->fetch(PDO::FETCH_NUM); + return ($row[0] > 0); } /** -- cgit v1.2.3-54-g00ecf