From: Galen Charlton <gmc@esilibrary.com>
Date: Thu, 10 Mar 2016 20:55:57 +0000 (-0500)
Subject: LP#1549505: fix handling of OPAC sort CGI parameter
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=99a6f39fa134879321a984be4d1b00f84687beee;p=working%2FEvergreen.git

LP#1549505: fix handling of OPAC sort CGI parameter

This patch ensures that the 'sort' parameter does not
get duplicate when a global default sort order is set
and a user switches from that to relevance to another
order.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
---

diff --git a/Open-ILS/src/templates/opac/parts/searchbar.tt2 b/Open-ILS/src/templates/opac/parts/searchbar.tt2
index 64cae0d926..3c93b42f86 100644
--- a/Open-ILS/src/templates/opac/parts/searchbar.tt2
+++ b/Open-ILS/src/templates/opac/parts/searchbar.tt2
@@ -100,7 +100,7 @@
         [% END %]
     </div>
     [%- END %]
-    [% IF NOT CGI.param('sort') %]
+    [% IF NOT CGI.param('sort').defined AND ctx.default_sort %]
     <input type="hidden" name="sort" value="[% ctx.default_sort %]"/>
     [% END %]
     [% UNLESS took_care_of_form %]</form>[% END %]