Bug #1680 [Opn->Csd]: Unapproved packages show in package browser

Edit report at http://pear.php.net/bugs/bug.php?id=1680&edit=1 ID: 1680 Updated by: danielc@analysisandsolutions.com Reported By: demenev at on-line dot jar dot ru -Status: Open +Status: Closed Type: Bug Package: Web Site Operating System: N/A PHP Version: Irrelevant -Assigned To: +Assigned To: danielc New Comment: Thanks so much. Previous Comments: ------------------------------------------------------------------------ [2004-06-19 06:38:12] demenev at on-line dot jar dot ru Description: ------------ Package browser ( http://pear.php.net/packages.php ) shows packages which are not yet approved. Provided patch fixes it Reproduce code: --------------- Index: packages.php =================================================================== RCS file: /repository/pearweb/public_html/packages.php,v retrieving revision 1.44 diff -u -r1.44 packages.php --- packages.php 9 Jun 2004 06:14:02 -0000 1.44 +++ packages.php 19 Jun 2004 10:44:43 -0000 @@ -103,7 +103,7 @@ // 1) Show categories of this level $sth = $dbh->query("SELECT c.*, COUNT(p.id) AS npackages FROM categories c LEFT JOIN packages p ON p.category = c.id ". - "WHERE p.package_type = 'pear' AND c.parent $category_where ". + "WHERE p.package_type = 'pear' AND p.approved = 1 AND c.parent $category_where ". "GROUP BY c.id ORDER BY name"); $table = new HTML_Table('border="0" cellpadding="6" cellspacing="2" width="100%"'); @@ -122,6 +122,7 @@ " FROM packages p, categories c". " WHERE c.parent $category_where ". " AND p.package_type = 'pear' ". + " AND p.approved = 1 ". " AND p.category = c.id ORDER BY p.name", false, null, DB_FETCHMODE_ASSOC, true); ------------------------------------------------------------------------ -- Edit this bug report at http://pear.php.net/bugs/bug.php?id=1680&edit=1