From 52f8a077fe8494040409724ca602e726b7b0975c Mon Sep 17 00:00:00 2001
From: Bill Erickson <berickxx@gmail.com>
Date: Tue, 17 Mar 2015 16:10:04 -0400
Subject: [PATCH] Recover auto search submission on sort selector change

Avoid applying two onchange() handlers to a single element, since only
one will ever fire.  In this case, the extra handler prevented the
automatic form submission from running a new search.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
 KCLS/openils/var/templates_kcls/opac/parts/filtersort.tt2 | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/KCLS/openils/var/templates_kcls/opac/parts/filtersort.tt2 b/KCLS/openils/var/templates_kcls/opac/parts/filtersort.tt2
index 72072ca2c7..65a0733414 100644
--- a/KCLS/openils/var/templates_kcls/opac/parts/filtersort.tt2
+++ b/KCLS/openils/var/templates_kcls/opac/parts/filtersort.tt2
@@ -1,7 +1,8 @@
 	<!-- Onchange below is part of a hack to keep the Sort Results order selector persistent across multiple searches in single tab 
 		 see templates_kcls/opac/parts/search.tt2 and simple.js-->
-<select class="results_header_sel" id='opac.result.sort' name="sort" onchange="setPersistentSort();"
-    [% IF submit_on_change %]onchange='this.form.submit()'[% END %]>
+<select class="results_header_sel" id='opac.result.sort' name="sort" 
+    [% IF submit_on_change %]onchange="setPersistentSort();this.form.submit()"
+    [% ELSE %]onchange="setPersistentSort()"[% END %]>
     <option value=''>[% l("Sort by Relevance") %]</option>
     <optgroup label='[% l("Sort by Title") %]'>
         <option value='titlesort'[% value == 'titlesort' ? ' selected="selected"' : '' %]>[% l("Title: A to Z") %]</option>
-- 
2.11.0