diff options
| author | Dan McGee <dan@archlinux.org> | 2011-03-01 11:42:29 -0600 |
|---|---|---|
| committer | Lukas Fleischer <archlinux@cryptocrack.de> | 2011-03-04 10:29:16 +0100 |
| commit | 3d5b735faec49c03c77732c3c21c2f91e59e060c (patch) | |
| tree | ed54f96a8203bc29b070a6612f5c9457b5c849db | |
| parent | f4fd7f7c9b92bef2607b787d77257904f61ef27a (diff) | |
| download | aurweb-3d5b735faec49c03c77732c3c21c2f91e59e060c.tar.xz | |
Fix PHP notices in pkg_search_form
If we were displaying this without coming from a previous query submit,
notices were emitted all over the place. Fix them by adding an isset()
first to each variable we try to access.
Signed-off-by: Dan McGee <dan@archlinux.org>
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
| -rw-r--r-- | web/template/pkg_search_form.php | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/web/template/pkg_search_form.php b/web/template/pkg_search_form.php index 7592b69..896d47f 100644 --- a/web/template/pkg_search_form.php +++ b/web/template/pkg_search_form.php @@ -5,7 +5,7 @@ <div class='pgboxtitle'> <span class='f3'><?php print __("Search Criteria"); ?></span> <input type='hidden' name='O' value='0' /> - <input type='text' name='K' size='30' value="<?php print stripslashes(trim(htmlspecialchars($_REQUEST["K"], ENT_QUOTES))); ?>" maxlength='35' /> + <input type='text' name='K' size='30' value="<?php if (isset($_REQUEST["K"])) { print stripslashes(trim(htmlspecialchars($_REQUEST["K"], ENT_QUOTES))); } ?>" maxlength='35' /> <input type='submit' style='min-width:80px' class='button' name='do_Search' value='<?php print __("Go"); ?>' /> <?php if (!empty($_GET['detail'])): ?> <input type='submit' style='min-width:80px' class='button' name='do_Orphans' value='<?php print __("Orphans"); ?>' /> @@ -23,7 +23,7 @@ <option value='0'><?php print __("Any"); ?></option> <?php foreach (pkgCategories() as $id => $cat): - if (intval($_GET["C"]) == $id): + if (isset($_REQUEST['C']) && $_REQUEST['C'] == $id): ?> <option value="<?php print $id ?>" selected="selected"><?php print $cat; ?></option> <?php else: ?> @@ -40,7 +40,7 @@ <?php $searchby = array('nd' => __('Name, Description'), 'n' => __('Name Only'), 'm' => __('Maintainer'), 's' => __('Submitter')); foreach ($searchby as $k => $v): - if ($_REQUEST['SeB'] == $k): + if (isset($_REQUEST['SeB']) && $_REQUEST['SeB'] == $k): ?> <option value="<?php print $k; ?>" selected="selected"><?php print $v; ?></option> <?php else: ?> @@ -57,7 +57,7 @@ <?php $sortby = array('n' => __('Name'), 'c' => __('Category'), 'v' => __('Votes'), 'w' => __('Voted'), 'o' => __('Notify'), 'm' => __('Maintainer'), 'a' => __('Age')); foreach ($sortby as $k => $v): - if ($_REQUEST['SB'] == $k): + if (isset($_REQUEST['SB']) && $_REQUEST['SB'] == $k): ?> <option value='<?php print $k; ?>' selected="selected"><?php print $v; ?></option> <?php else: ?> @@ -74,7 +74,7 @@ <?php $orderby = array('a' => __('Ascending'), 'd' => __('Descending')); foreach ($orderby as $k => $v): - if ($_REQUEST['SO'] == $k): + if (isset($_REQUEST['SO']) && $_REQUEST['SO'] == $k): ?> <option value='<?php print $k; ?>' selected="selected"><?php print $v; ?></option> <?php else: ?> @@ -91,7 +91,7 @@ <?php $pages = array(25, 50, 75, 100); foreach ($pages as $i): - if ($_REQUEST['PP'] == $i): + if (isset($_REQUEST['PP']) && $_REQUEST['PP'] == $i): ?> <option value="<?php print $i; ?>" selected="selected"><?php print $i; ?></option> <?php else: ?> @@ -108,7 +108,7 @@ <?php $outdated_flags = array('' => __('All'), 'on' => __('Flagged'), 'off' => __('Not Flagged')); foreach ($outdated_flags as $k => $v): - if ($_REQUEST['outdated'] == $k): + if (isset($_REQUEST['outdated']) && $_REQUEST['outdated'] == $k): ?> <option value='<?php print $k; ?>' selected="selected"><?php print $v; ?></option> <?php else: ?> |