diff options
author | Lukas Fleischer <archlinux@cryptocrack.de> | 2012-05-06 12:38:59 +0200 |
---|---|---|
committer | Lukas Fleischer <archlinux@cryptocrack.de> | 2012-05-06 12:47:19 +0200 |
commit | 5b0f1f39dba251e5ae29df432587da7ce0098c87 (patch) | |
tree | 5916a4b5e8a0f87c51d0753da0dc8abbb5383730 /web/html/packages.php | |
parent | 97085daadcbc8b1ae1a01cd777605f66464a02ea (diff) |
web/html/packages.php: Remove duplicate code
Retrieve the package ID early and merge the if-branches calling
package_details().
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
Diffstat (limited to 'web/html/packages.php')
-rw-r--r-- | web/html/packages.php | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/web/html/packages.php b/web/html/packages.php index 8f3b143..aa1d04c 100644 --- a/web/html/packages.php +++ b/web/html/packages.php @@ -89,28 +89,24 @@ html_header($title); <?php if (isset($_GET['ID'])) { - include('pkg_search_form.php'); - if (!$_GET['ID'] = intval($_GET['ID'])) { - print __("Error trying to retrieve package details.")."<br />\n"; - } else { - if (isset($_COOKIE["AURSID"])) { - package_details($_GET['ID'], $_COOKIE["AURSID"]); - } - else { - package_details($_GET['ID'], null); - } - } + $pkgid = intval($_GET['ID']); } else if (isset($_GET['N'])) { + $pkgid = pkgid_from_name($_GET['N']); +} else { + unset($pkgid); +} + +if (isset($pkgid)) { include('pkg_search_form.php'); - if (!$pkgid = pkgid_from_name($_GET['N'])) { - print __("Error trying to retrieve package details.")."<br />\n"; - } else { + if ($pkgid) { if (isset($_COOKIE["AURSID"])) { package_details($pkgid, $_COOKIE["AURSID"]); } else { package_details($pkgid, null); } + } else { + print __("Error trying to retrieve package details.")."<br />\n"; } } else { if (!isset($_GET['K']) && !isset($_GET['SB'])) { |