From cf4ea0171e588fdc9d44beb705e379a20ecbd652 Mon Sep 17 00:00:00 2001 From: Lukas Fleischer Date: Tue, 1 Jul 2014 19:43:27 +0200 Subject: Simplify code to bound integer values Suggested-by: Florian Pritz Signed-off-by: Lukas Fleischer --- web/html/pkgreq.php | 10 ++-------- web/lib/aur.inc.php | 13 +++++++++++++ web/lib/pkgfuncs.inc.php | 16 ++++------------ 3 files changed, 19 insertions(+), 20 deletions(-) diff --git a/web/html/pkgreq.php b/web/html/pkgreq.php index 156645a..401b60c 100644 --- a/web/html/pkgreq.php +++ b/web/html/pkgreq.php @@ -21,19 +21,13 @@ if (!isset($base_id)) { /* Sanitize paging variables. */ if (isset($_GET['O'])) { - $_GET['O'] = intval($_GET['O']); - if ($_GET['O'] < 0) - $_GET['O'] = 0; + $_GET['O'] = max(intval($_GET['O']), 0); } else { $_GET['O'] = 0; } if (isset($_GET["PP"])) { - $_GET["PP"] = intval($_GET["PP"]); - if ($_GET["PP"] < 50) - $_GET["PP"] = 50; - else if ($_GET["PP"] > 250) - $_GET["PP"] = 250; + $_GET["PP"] = bound(intval($_GET["PP"]), 50, 250); } else { $_GET["PP"] = 50; } diff --git a/web/lib/aur.inc.php b/web/lib/aur.inc.php index 3368696..7fa792b 100644 --- a/web/lib/aur.inc.php +++ b/web/lib/aur.inc.php @@ -584,3 +584,16 @@ function array_pkgbuild_merge($pkgbase_info, $section_info) { } return $pi; } + +/** + * Bound an integer value between two values + * + * @param int $n Integer value to bound + * @param int $min Lower bound + * @param int $max Upper bound + * + * @return int Bounded integer value + */ +function bound($n, $min, $max) { + return min(max($n, $min), $max); +} diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index f515864..7e65d09 100644 --- a/web/lib/pkgfuncs.inc.php +++ b/web/lib/pkgfuncs.inc.php @@ -519,22 +519,14 @@ function pkg_search_page($SID="") { /* Sanitize paging variables. */ if (isset($_GET['O'])) { - $_GET['O'] = intval($_GET['O']); - if ($_GET['O'] < 0) - $_GET['O'] = 0; - } - else { + $_GET['O'] = max(intval($_GET['O']), 0); + } else { $_GET['O'] = 0; } if (isset($_GET["PP"])) { - $_GET["PP"] = intval($_GET["PP"]); - if ($_GET["PP"] < 50) - $_GET["PP"] = 50; - else if ($_GET["PP"] > 250) - $_GET["PP"] = 250; - } - else { + $_GET["PP"] = bound(intval($_GET["PP"]), 50, 250); + } else { $_GET["PP"] = 50; } -- cgit v1.2.3-54-g00ecf