From: Garry Collum <gcollum@gmail.com>
Date: Wed, 15 Jul 2009 02:16:27 +0000 (-0400)
Subject: Bug 3322: Fixes expandedSearchOption in opac. Makes "fewer options" possible.
X-Git-Tag: v3.00.04~340
X-Git-Url: http://git.rot13.org/?a=commitdiff_plain;h=a9b908c6292a525806ef3d5a60a1ee38c9a368be;p=koha.git

Bug 3322: Fixes expandedSearchOption in opac. Makes "fewer options" possible.

Copies the logic of catalogue/search.pl to fix the expanded search option in opac. When expanded search is the default, it was impossible to go to the "fewer" options.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
---

diff --git a/opac/opac-search.pl b/opac/opac-search.pl
index 72fa0af09e..0c55cd3d2a 100755
--- a/opac/opac-search.pl
+++ b/opac/opac-search.pl
@@ -231,11 +231,11 @@ if ( $template_type eq 'advsearch' ) {
                       search_boxes_loop => \@search_boxes_array);
 
 # use the global setting by default
-	if ( C4::Context->preference("expandedSearchOption") ) {
+	if ( C4::Context->preference("expandedSearchOption") == 1 ) {
 		$template->param( expanded_options => C4::Context->preference("expandedSearchOption") );
 	}
 	# but let the user override it
-   	if ( $cgi->param("expanded_options") && (($cgi->param('expanded_options') == 0) || ($cgi->param('expanded_options') == 1 )) ) {
+   	if ( ($cgi->param('expanded_options') == 0) || ($cgi->param('expanded_options') == 1 ) ) {
     	$template->param( expanded_options => $cgi->param('expanded_options'));
 	}