From 36d1b3e29d89c2af04c50d4f9ae91cb18c391f97 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Wed, 9 Jul 2014 15:56:54 +0200 Subject: [PATCH] Bug 12330: [QA Follow-up] Consistency between opac-search and staff client In opac-search two regexes are added for the results of splitting the pref AdvancedSearchTypes, allowing spaces before and after the names. The same regexes should be added to catalogue/search.pl to have the same behavior there. Test plan: Set AdvancedSearchTypes to "itemtypes | ccode | loc " (no quotes). Check OPAC and staff with and without this patch. Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi --- catalogue/search.pl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/catalogue/search.pl b/catalogue/search.pl index dbb75caab6..24abcb0963 100755 --- a/catalogue/search.pl +++ b/catalogue/search.pl @@ -258,6 +258,8 @@ my $advanced_search_types = C4::Context->preference("AdvancedSearchTypes") || "i my @advanced_search_types = split(/\|/, $advanced_search_types); foreach my $advanced_srch_type (@advanced_search_types) { + $advanced_srch_type =~ s/^\s*//; + $advanced_srch_type =~ s/\s*$//; if ($advanced_srch_type eq 'itemtypes') { # itemtype is a special case, since it's not defined in authorized values my @itypesloop; -- 2.39.5