From: Jason Stephenson <jstephenson@mvlc.org>
Date: Thu, 2 Feb 2012 14:19:11 +0000 (-0500)
Subject: Use getItemType() instead of getForm() in searchBarInit().
X-Git-Tag: sprint4-merge-nov22~4467
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=f4602609ff15772b4e18a23d4cf7b42e8a1689d8;p=working%2FEvergreen.git

Use getItemType() instead of getForm() in searchBarInit().

This seems to actually fix the problem with the item type limiter
being lost when going from advanced search to the results page.

Looks like the wrong method was being used to set the form_selector
value in search_bar.js. It was being set to the web form itself and
not to the value of the it (item type) parameter.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.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 20e8552704..be0239eba1 100644
--- a/Open-ILS/web/opac/skin/default/js/search_bar.js
+++ b/Open-ILS/web/opac/skin/default/js/search_bar.js
@@ -66,7 +66,7 @@ function searchBarInit() {
 	G.ui.searchbar.text.value = (getTerm() != null) ? getTerm() : "";
 	if (!isFrontPage) G.ui.searchbar.facets.value = (getFacet() != null) ? getFacet() : "";
 	setSelector(_ts,	getStype());
-	setSelector(_fs,	getForm());
+	setSelector(_fs,	getItemType());
 
 	depthSelInit();