[oe-commits] org.oe.dev feed-browser: simplified URLs (no 'action' var anymore), simplified code a bit

hrw commit openembedded-commits at lists.openembedded.org
Thu Jul 5 17:06:43 UTC 2007


feed-browser: simplified URLs (no 'action' var anymore), simplified code a bit

Author: hrw at openembedded.org
Branch: org.openembedded.dev
Revision: 6ecc6574ba0db38196a290448aaaf3706bcacee3
ViewMTN: http://monotone.openembedded.org/revision.psp?id=6ecc6574ba0db38196a290448aaaf3706bcacee3
Files:
1
contrib/feed-browser/includes/functions.inc
contrib/feed-browser/index.php
Diffs:

#
# mt diff -r958b4439c90908a21e42ec69bb3c538fd32e6422 -r6ecc6574ba0db38196a290448aaaf3706bcacee3
#
# 
# 
# patch "contrib/feed-browser/includes/functions.inc"
#  from [95453cbdd9652ae2bcc87d9b4cba36b4f74dbd93]
#    to [3059992209c9a86a48ebdb97976bf8060a3cdcba]
# 
# patch "contrib/feed-browser/index.php"
#  from [37f8acf6f82930c0ad591ed43115b87e30b22874]
#    to [bf33048c4a2921597d0181c9b565ff0e73cd2b97]
# 
============================================================
--- contrib/feed-browser/includes/functions.inc	95453cbdd9652ae2bcc87d9b4cba36b4f74dbd93
+++ contrib/feed-browser/includes/functions.inc	3059992209c9a86a48ebdb97976bf8060a3cdcba
@@ -112,34 +112,32 @@ function searchletter($searchletter = ''
 		}
 		else
 		{
-			$ipkgoutput .= sprintf(" <a href='?action=letter&amp;letter=%s' title='packages which names begins with \"%s\"'>%s</a> |", $letter, $letter, $letter );
+			$ipkgoutput .= sprintf(" <a href='?letter=%s' title='packages which names begins with \"%s\"'>%s</a> |", $letter, $letter, $letter );
 		}
 	}
 
-	$ipkgoutput .= " <a href='?action=letter&amp;letter=z' title='packages which names begins with \"z\"'>z</a></div>";
+	$ipkgoutput .= " <a href='?letter=z' title='packages which names begins with \"z\"'>z</a></div>";
 
 	return $ipkgoutput;
 }
 
-function searchpkg ($searchword)
+function searchpkg ($searchword, $searcharch = '')
 {
-	if($result = db_query("SELECT DISTINCT p_name,p_desc,p_section  FROM packages WHERE p_name LIKE '$searchword' ORDER BY p_name ASC"))
+	$query = "SELECT DISTINCT p_name,p_desc,p_section FROM packages WHERE p_name LIKE '$searchword' ";
+	
+	if(!empty($searcharch))
 	{
-	    return generate_list_of_packages($result);
+		$query .= " AND p_arch='{$searcharch}' ";
 	}
-}
 
-
-function searchpkgarch ($searchword, $searcharch)
-{
-	if($result = db_query("SELECT DISTINCT p_name,p_desc,p_section  FROM packages WHERE p_name LIKE '$searchword' AND p_arch LIKE '$searcharch'  ORDER BY p_name ASC"))
-	{   
-		return generate_list_of_packages($result);
+	$query .= 'ORDER BY p_name ASC';
+	
+	if($result = db_query($query))
+	{
+	    return generate_list_of_packages($result);
 	}
 }
 
-
-
 function generate_list_of_packages($query_result)
 {
 	$ipkgoutput = "<table>\n";
@@ -160,7 +158,7 @@ function generate_list_of_packages($quer
 			}
 
 			$ipkgoutput .= sprintf
-				("<tr><td><a href='?action=details&amp;pnm=%s'>%s</a></td><td><a href=\"?action=section&amp;section=%s\">%s</a></td><td> %s</td></tr>\n",
+				("<tr><td><a href='?pkgname=%s'>%s</a></td><td><a href=\"?section=%s\">%s</a></td><td> %s</td></tr>\n",
 				urlencode($package['p_name']), $package['p_name'], $package['p_section'], $package['p_section'], htmlentities($package['p_desc']));
 		}
 
@@ -205,7 +203,7 @@ function pkgdetails ($package)
 
 		if($package['packages.p_section'])
 		{
-			$details .= sprintf ("\n<dt>Section:</dt><dd><a href='?action=section&amp;section=%s'>%s</a></dd>", $package['packages.p_section'],$package['packages.p_section']);
+			$details .= sprintf ("\n<dt>Section:</dt><dd><a href='?section=%s'>%s</a></dd>", $package['packages.p_section'],$package['packages.p_section']);
 		}
 		
 		if($package['packages.p_depends'])
@@ -321,7 +319,7 @@ function addlinks ($input)
 		{
 			// find position of string in line
 			$pos = strpos ($input, $element, $offset);
-			$link = sprintf("<a href=\"?action=details&amp;pnm=%s\">$element</a>", urlencode ($element));
+			$link = sprintf("<a href=\"?pkgname=%s\">$element</a>", urlencode ($element));
 
 			// replace element with a link
 			$input = substr_replace ($input, $link, $pos, strlen ($element));
@@ -372,7 +370,7 @@ function sectionslist()
 
 		foreach($sections as $section_name1=>$item)
 		{
-			$output .= sprintf ("<li><a href='?action=section&amp;section=%s' title='%s'>%s</a>",
+			$output .= sprintf ("<li><a href='?section=%s' title='%s'>%s</a>",
 				urlencode($section_name1),
 				urlencode($section_name1),
 				$section_name1);
@@ -384,7 +382,7 @@ function sectionslist()
 				foreach($item as $section_name2=>$subitem)
 				{
 					$section_name = "{$section_name1}/{$section_name2}";
-					$output .= sprintf ("<li><a href='?action=section&amp;section=%s' title='%s'>%s</a>",
+					$output .= sprintf ("<li><a href='?section=%s' title='%s'>%s</a>",
 						urlencode($section_name),
 						urlencode($section_name),
 						$section_name2);
@@ -396,7 +394,7 @@ function sectionslist()
 						foreach($subitem as $section_name3=>$subitem2)
 						{
 							$section_name = "{$section_name1}/{$section_name2}/{$section_name3}";
-							$output .= sprintf ("<li><a href='?action=section&amp;section=%s' title='%s'>%s</a></li>",
+							$output .= sprintf ("<li><a href='?section=%s' title='%s'>%s</a></li>",
 								urlencode($section_name),
 								urlencode($section_name),
 								$section_name3);
============================================================
--- contrib/feed-browser/index.php	37f8acf6f82930c0ad591ed43115b87e30b22874
+++ contrib/feed-browser/index.php	bf33048c4a2921597d0181c9b565ff0e73cd2b97
@@ -38,40 +38,26 @@ if(!check_database())
 	die("Database not found and cannot be created.");
 }
 
-read_vars_from_get(array('name', 'arch', 'action', 'letter', 'pnm', 'section'));
+read_vars_from_get(array('name', 'arch', 'pkgsearch', 'letter', 'pkgname', 'section'));
 
-switch($action)
-{
-	case "details":
-		$ipkgoutput = pkgdetails ($pnm);
-		break;
+$ipkgoutput = '';
 
-	case "search":
-		if ( $arch == "" ) {
-		    $ipkgoutput = searchpkg ("%{$name}%");
-		    }
-		else {
-		    $ipkgoutput = searchpkgarch ("%{$name}%", "{$arch}");
-		    }
-		break;
-
-	case "section":
-		$ipkgoutput = searchsection($section);
-		break;
-
-	case "letter":
-		if ( $arch == "" ) {
-		     $ipkgoutput = searchpkg ("{$letter}%");
-		}
-                else {
-                    $ipkgoutput = searchpkgarch ("{$letter}%", "{$arch}");
-                    }
-                break;
-
-	default:
-		$ipkgoutput = searchpkg("a");
-		break;
+if(!empty($section))
+{
+	$ipkgoutput = searchsection($section);
 }
+elseif(!empty($letter))
+{
+	$ipkgoutput = searchpkg("{$letter}%", $arch);
+}
+elseif(!empty($pkgname))
+{
+	$ipkgoutput = pkgdetails($pkgname);
+}
+elseif(!empty($pkgsearch))
+{
+	$ipkgoutput = searchpkg("%{$pkgsearch}%", $arch);
+}
 
 ?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
@@ -88,7 +74,7 @@ switch($action)
 			<form action="" method="get">
 				<fieldset>
 					<label for="name">Package name</label>
-					<input type="text" name="name" value="<?php echo $name; ?>" />
+					<input type="text" name="pkgsearch" value="<?php echo $name; ?>" />
 					<select name="arch">
 					   <option value="" selected="selected">all architectures</option>
 					   <option value="all">noarch</option>
@@ -106,7 +92,6 @@ switch($action)
 					   <option value="ppc603e">ppc603e</option>
 					   <option value="sparc">sparc</option>
 					</select>
-					<input type="hidden" name="action" value="search" />
 					<input type="submit" value="Search" />
 				</fieldset>
 			</form>






More information about the Openembedded-commits mailing list