LP#1549505: avoid doubling of the 'sort' CGI parameter

This patch ensures that the hidden input to set
the default sort value doesn't get applied to the results
page, which already supplied a widget for the purpose.

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 3c93b42..53ee4bc 100644 (file)
--- a/Open-ILS/src/templates/opac/parts/searchbar.tt2
+++ b/Open-ILS/src/templates/opac/parts/searchbar.tt2
@@ -100,10 +100,12 @@
         [% END %]
     </div>
     [%- END %]
-    [% IF NOT CGI.param('sort').defined AND ctx.default_sort %]
-    <input type="hidden" name="sort" value="[% ctx.default_sort %]"/>
+    [% UNLESS took_care_of_form %]
+        [% IF NOT CGI.param('sort').defined AND ctx.default_sort %]
+            <input type="hidden" name="sort" value="[% ctx.default_sort %]"/>
+        [% END %]
+        </form>
     [% END %]
-    [% UNLESS took_care_of_form %]</form>[% END %]
     [% IF (is_advanced AND NOT is_special) AND CGI.param('qtype') %]
     <div class="opac-auto-102">
         [ <a href="[% mkurl(ctx.opac_root _ '/advanced') %]" rel="nofollow" vocab="">[%