[aur-dev] [PATCH] Bump up default per-page value to 50

Dan McGee dan at archlinux.org
Tue Mar 22 23:01:34 EDT 2011


25 is woefully small for the number of packages many searches can
return, and with 28000+ packages in AUR, it makes sense to show a lot
more per page by default.

The new choices of (50, 100, 250) happen to match those from the main
site.

Signed-off-by: Dan McGee <dan at archlinux.org>
---
 web/lib/pkgfuncs.inc             |    8 ++++----
 web/template/pkg_search_form.php |    2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/web/lib/pkgfuncs.inc b/web/lib/pkgfuncs.inc
index 03deea2..65091e3 100644
--- a/web/lib/pkgfuncs.inc
+++ b/web/lib/pkgfuncs.inc
@@ -419,13 +419,13 @@ function pkg_search_page($SID="") {
 
 	if (isset($_GET["PP"])) {
 		$_GET["PP"] = intval($_GET["PP"]);
-		if ($_GET["PP"] < 25)
-			$_GET["PP"] = 25;
+		if ($_GET["PP"] < 50)
+			$_GET["PP"] = 50;
 		else if ($_GET["PP"] > 100)
 			$_GET["PP"] = 100;
 	}
 	else {
-		$_GET["PP"] = 25;
+		$_GET["PP"] = 50;
 	}
 
 	// FIXME: pull out DB-related code. all of it.
@@ -560,7 +560,7 @@ function pkg_search_page($SID="") {
 
 	if ($total > 1 || $total == 0) {
 		# calculation of pagination links
-		$per_page = ($_GET['PP'] > 0) ? $_GET['PP'] : 25;
+		$per_page = ($_GET['PP'] > 0) ? $_GET['PP'] : 50;
 		$current = ceil($first / $per_page);
 		$pages = ceil($total / $per_page);
 		$templ_pages = array();
diff --git a/web/template/pkg_search_form.php b/web/template/pkg_search_form.php
index 896d47f..e570dfb 100644
--- a/web/template/pkg_search_form.php
+++ b/web/template/pkg_search_form.php
@@ -89,7 +89,7 @@
 						<label><?php print __("Per page"); ?></label>
 						<select name='PP'>
 							<?php
-							$pages = array(25, 50, 75, 100);
+							$pages = array(50, 100, 250);
 							foreach ($pages as $i):
 								if (isset($_REQUEST['PP']) && $_REQUEST['PP'] == $i):
 							?>
-- 
1.7.4.1



More information about the aur-dev mailing list