When sorting a search results set the isFilterSort variable is passed

to searchBarSubmit.  In side searchBarSumbit there is a call to the
opac_utils.js function clearSearchParams function.  This removes all
search paramaters when sorting a result set, which is not the desired
effect.  I have wraped clearSearchParams in a check to make sure
isFilterSort is not set before calling the clearSearchParams function,
and this check ensures that the current search's paramaters are passed
to sort operations.

Signed-off-by: Liam Whalen <lwhalen@evergreen-dev.catalogue.nrcan.gc.ca>
Signed-off-by: Jason Stephenson <jason@sigio.com>

diff --git a/Open-ILS/web/opac/skin/default/js/search_bar.js b/Open-ILS/web/opac/skin/default/js/search_bar.js
index 76d123d..1801edc 100644 (file)
--- a/Open-ILS/web/opac/skin/default/js/search_bar.js
+++ b/Open-ILS/web/opac/skin/default/js/search_bar.js
@@ -54,7 +54,9 @@ function searchBarSubmit(isFilterSort) {
        var text = G.ui.searchbar.text.value;
        var facet_text = isFrontPage ? '' : G.ui.searchbar.facets.value;
 
-       clearSearchParams();
+    if (!isFilterSort) {       
+        clearSearchParams();
+    }
 
        if(!text || text == "") return;