Summary: | DataTables search ( dt-search ) does not work on holds history page | ||
---|---|---|---|
Product: | Koha | Reporter: | Lucas Gass (lukeg) <lucas> |
Component: | Patrons | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | Signed Off --- | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, jonathan.druart, kkrueger, kyle, lisette |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | 35560 | ||
Bug Blocks: | |||
Attachments: |
Bug 40321: Fix hold history search
Bug 40321: Fix hold history search |
Description
Lucas Gass (lukeg)
2025-07-07 17:24:46 UTC
Matt clued me in on what is happening here. It has to do with checkboxes for pending, waiting, processing, and in transit. "It's one big OR query which includes an array of all the possible statuses if the boxes are checked. So if you don’t touch those filters then it will always return all results." The checkboxes are going to take precedent in any search one does, and that feels right to me. You can also filter per individual column and that works as excepted. So in this case I think what we should do is change the dataTable config to NOT include the dt-search "search bar". Created attachment 184583 [details] [review] Bug 40321: Fix hold history search The table global search didn't combine correctly with the status filters. Test plan: 1. Have a patron with holds history 2. Go to members/holdshistory.pl?borrowernumber=x where x is the borrowernumber 3. Try to us the DT seacch to filter => Filtering is working as expected Trivial fix that makes a whole lot of sense, this is the behavior most people want. Created attachment 184600 [details] [review] Bug 40321: Fix hold history search The table global search didn't combine correctly with the status filters. Test plan: 1. Have a patron with holds history 2. Go to members/holdshistory.pl?borrowernumber=x where x is the borrowernumber 3. Try to us the DT seacch to filter => Filtering is working as expected Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> |