From d7a7336dff3e33d566b2c85efd689963ef7826de Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Tue, 17 Dec 2013 16:04:10 -0500 Subject: [PATCH] Bug 11410: ensure cataloging search finds results for searches with ampersands When QueryParser is off, the cataloging search is passing the incorrect variable to SimpleSearch. I'm sure this is causing other issues, but the one I have identified is that any searches with an "&" in them return no results. This patch corrects the bug. Test Plan: 1) Catalog a new record titled "Cats & Dogs" 2) Rebuild your zebra index so this record is indexed 3) Browse to cataloguing/addbooks.pl 4) Search for "Cats & Dogs" 5) Notice you get no results 6) Apply this patch 7) Repeat the search 8) Notice your result now shows up! Signed-off-by: Heather Braum Signed-off-by: Suzanne Fayle Signed-off-by: Katrin Fischer This works as described, passes all tests and QA script. Note: Because $query = $builtquery when the QueryParser is used, this problem is only visible when UseQueryParser is set to "Don't try". Signed-off-by: Galen Charlton (cherry picked from commit 2a4e193c242372f488892591a72752dfd4ffb138) Signed-off-by: Fridolin SOMERS (cherry picked from commit 0c500f463001db06c8c1f72f8b81c8edde6487d7) Signed-off-by: Tomas Cohen Arazi --- cataloguing/addbooks.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cataloguing/addbooks.pl b/cataloguing/addbooks.pl index b22a6469ff..35cfa2cf70 100755 --- a/cataloguing/addbooks.pl +++ b/cataloguing/addbooks.pl @@ -82,7 +82,7 @@ if ($query) { } # find results - my ( $error, $marcresults, $total_hits ) = SimpleSearch($query, $results_per_page * ($page - 1), $results_per_page); + my ( $error, $marcresults, $total_hits ) = SimpleSearch($builtquery, $results_per_page * ($page - 1), $results_per_page); if ( defined $error ) { $template->param( error => $error ); -- 2.39.5