diff options
author | Lukas Fleischer <archlinux@cryptocrack.de> | 2014-07-01 19:43:27 +0200 |
---|---|---|
committer | Lukas Fleischer <archlinux@cryptocrack.de> | 2014-07-01 19:46:16 +0200 |
commit | cf4ea0171e588fdc9d44beb705e379a20ecbd652 (patch) | |
tree | c06049077e9761a3961ab45d3b3d093c4f3cb472 | |
parent | a48739508c8b9de9fb0af30b8b656c5f19378170 (diff) |
Simplify code to bound integer values
Suggested-by: Florian Pritz <bluewind@xinu.at>
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
-rw-r--r-- | web/html/pkgreq.php | 10 | ||||
-rw-r--r-- | web/lib/aur.inc.php | 13 | ||||
-rw-r--r-- | 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; } |