From 2a4e193c242372f488892591a72752dfd4ffb138 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 --- 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.2