Bug 37427

Summary: Searching for an empty string for clubs in an item's hold tab is not allowed
Product: Koha Reporter: Sam Lau <samalau>
Component: Hold requestsAssignee: Sam Lau <samalau>
Status: Pushed to main --- QA Contact: Lucas Gass (lukeg) <lucas>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, lucas
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.05.00
Circulation function:
Attachments: Bug 37427: Fix club search to display all clubs when search field is empty, consistent with patron search behavior
Bug 37427: Fix club search to display all clubs when search field is empty, consistent with patron search behavior
Bug 37427: Fix club search to display all clubs when search field is empty, consistent with patron search behavior

Description Sam Lau 2024-07-22 19:58:56 UTC
To reproduce: 
1) have some patron clubs
2) find a bib
3) Click Holds, then Clubs
4) Without entering any text, press the search button
5) Note that the list of clubs is not shown.

However, when searching for a patron with an empty string, it returns the entire list of patrons. I think it would be useful if club searching followed this same functionality.
Comment 1 Sam Lau 2024-07-22 20:09:50 UTC
Created attachment 169382 [details] [review]
Bug 37427: Fix club search to display all clubs when search field is empty, consistent with patron search behavior

To test:
1) Create multiple patron clubs
2) Find a bib
3) Click Holds, then Clubs
4) Without entering any text, press the search button
5) Note that the list of clubs is not shown.
6) Apply patch and restart_all
7) Press the search button again, the entire list of clubs should be shown.
8) Ensure original search functionality still works
Comment 2 Roman Dolny 2024-07-22 20:34:04 UTC
Created attachment 169385 [details] [review]
Bug 37427: Fix club search to display all clubs when search field is empty, consistent with patron search behavior

To test:
1) Create multiple patron clubs
2) Find a bib
3) Click Holds, then Clubs
4) Without entering any text, press the search button
5) Note that the list of clubs is not shown.
6) Apply patch and restart_all
7) Press the search button again, the entire list of clubs should be shown.
8) Ensure original search functionality still works

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Comment 3 Lucas Gass (lukeg) 2024-11-22 18:47:42 UTC
Created attachment 174945 [details] [review]
Bug 37427: Fix club search to display all clubs when search field is empty, consistent with patron search behavior

To test:
1) Create multiple patron clubs
2) Find a bib
3) Click Holds, then Clubs
4) Without entering any text, press the search button
5) Note that the list of clubs is not shown.
6) Apply patch and restart_all
7) Press the search button again, the entire list of clubs should be shown.
8) Ensure original search functionality still works

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 4 Katrin Fischer 2024-12-23 11:23:26 UTC
Pushed for 25.05!

Well done everyone, thank you!