summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Fleischer <archlinux@cryptocrack.de>2012-05-06 12:38:59 +0200
committerLukas Fleischer <archlinux@cryptocrack.de>2012-05-06 12:47:19 +0200
commit5b0f1f39dba251e5ae29df432587da7ce0098c87 (patch)
tree5916a4b5e8a0f87c51d0753da0dc8abbb5383730
parent97085daadcbc8b1ae1a01cd777605f66464a02ea (diff)
downloadaur-5b0f1f39dba251e5ae29df432587da7ce0098c87.tar.gz
aur-5b0f1f39dba251e5ae29df432587da7ce0098c87.tar.xz
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>
-rw-r--r--web/html/packages.php24
1 files changed, 10 insertions, 14 deletions
diff --git a/web/html/packages.php b/web/html/packages.php
index 8f3b143d..aa1d04ca 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'])) {