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 /web/html | |
| parent | a48739508c8b9de9fb0af30b8b656c5f19378170 (diff) | |
| download | aurweb-cf4ea0171e588fdc9d44beb705e379a20ecbd652.tar.xz | |
Simplify code to bound integer values
Suggested-by: Florian Pritz <bluewind@xinu.at>
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
Diffstat (limited to 'web/html')
| -rw-r--r-- | web/html/pkgreq.php | 10 |
1 files changed, 2 insertions, 8 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; } |