Bug 11410

Summary: Cataloging search finds no results for searches with ampersands
Product: Koha Reporter: Kyle M Hall <kyle>
Component: CatalogingAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: fridolin.somers, gmcharlt, gwilliams, m.de.rooy, marjorie.barry-vila, sfayle, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 11410 - Cataloging search finds no results for searches with ampersands
Bug 11410 - Cataloging search finds no results for searches with ampersands
[PASSED QA] Bug 11410 - Cataloging search finds no results for searches with ampersands

Description Kyle M Hall 2013-12-17 21:03:12 UTC
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.
Comment 1 Kyle M Hall 2013-12-17 21:06:33 UTC Comment hidden (obsolete)
Comment 2 HB-NEKLS 2013-12-17 22:10:22 UTC
I have followed this test plan, and can confirm the patch fixes this bug.
Comment 3 Suzanne Fayle 2013-12-20 01:42:42 UTC
This is working.
Comment 4 Kyle M Hall 2013-12-20 16:12:47 UTC Comment hidden (obsolete)
Comment 5 Katrin Fischer 2013-12-20 23:49:58 UTC
Created attachment 23742 [details] [review]
[PASSED QA] Bug 11410 - Cataloging search finds no results for searches with ampersands

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.

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 <hbraum@nekls.org>
Signed-off-by: Suzanne Fayle <sfayle@roseman.edu>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
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".
Comment 6 Galen Charlton 2013-12-21 00:35:59 UTC
Pushed to master.  Thanks, Kyle!
Comment 7 Fridolin Somers 2013-12-30 09:59:41 UTC
Patch pushed to 3.14.x, will be in 3.14.2
Comment 8 Tomás Cohen Arazi 2014-01-15 17:48:49 UTC
This patch has been pushed to 3.12.x, will be in 3.12.9.

Thanks Kyle!