diff mbox

Fix sorting order when clicking table headings

Message ID 20171107203649.3032-1-lfleischer@archlinux.org
State Accepted, archived
Headers show

Commit Message

Lukas Fleischer Nov. 7, 2017, 8:36 p.m. UTC 
A bug introduced in commit 7d7e079 (Hide the table sorting links on the
dashboard, 2017-02-04) resulted in multiple clicks on a table heading in
the package search results table no longer having any effect, instead of
changing the sorting order. Fix this by removing erroneous spaces from
the GET parameters in the search URL.

Fixes FS#56261.

Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org>
---
 web/template/pkg_search_results.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox

Patch

diff --git a/web/template/pkg_search_results.php b/web/template/pkg_search_results.php
index 7f92685..d7512b1 100644
--- a/web/template/pkg_search_results.php
+++ b/web/template/pkg_search_results.php
@@ -3,7 +3,7 @@  if ($show_headers) {
 	$fmtth = function($title, $sb=false, $so=false, $hint=false) {
 		echo '<th>';
 		if ($sb) {
-			echo '<a href="?' . mkurl('SB=' . $sb . '&SO = ' . $so) . '">' . $title . '</a>';
+			echo '<a href="?' . mkurl('SB=' . $sb . '&SO=' . $so) . '">' . $title . '</a>';
 		} else {
 			echo $title;
 		}