Bug 10835

Summary: Patron search for using "contains" search type fails with only one search term
Product: Koha Reporter: Kyle M Hall (khall) <kyle>
Component: PatronsAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: normal    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, tomascohen
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 8845    
Bug Blocks: 10838    
Attachments: Bug 10835 - Patron search for using "contains" search type fails with only one search term
Bug 10835 - Patron search for using "contains" search type fails with only one search term
Bug 10835 - Patron search for using "contains" search type fails with only one search term
[PASSED QA] Bug 10835 - Patron search for using "contains" search type fails with only one search term

Description Kyle M Hall (khall) 2013-09-06 17:10:06 UTC
The patron search type option "contains" works fine for multiple strings, but returns no results for a single string search. For example, the patron "Henry Acevedo" will be returned for a "contains" search "en ev" but not for just "en" or "ev".
Comment 1 Kyle M Hall (khall) 2013-09-06 17:12:44 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall (khall) 2013-09-06 17:13:59 UTC Comment hidden (obsolete)
Comment 3 Chris Cormack 2013-09-07 03:19:47 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2013-09-07 18:16:00 UTC
Created attachment 20854 [details] [review]
[PASSED QA] Bug 10835 - Patron search for using "contains" search type fails with only one search term

The patron search type option "contains" works fine for multiple
strings, but returns no results for a single string search. For example,
the patron "Henry Acevedo" will be returned for a "contains" search
"en ev" but not for just "en" or "ev".

Test Plan:
1) Create 2 patrons named "Test One" and "Test Two"
2) Run a "contains" search for the term "est"
3) Note no results were found
4) Apply this patch
5) Repeat step 2
6) Note the patrons now display in the search results.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
All tests and QA script pass.
Also checked that 'starts with' still works as expected -
searching for ev will only return "Eva Dillon" from the
example patrons in this case.
Comment 5 Galen Charlton 2013-09-07 20:53:37 UTC
Pushed to master.  Thanks, Kyle!