Summary: | Sort by "circ note" is broken on the patron search result view | ||
---|---|---|---|
Product: | Koha | Reporter: | Alexis Ripetti <alexis.ripetti> |
Component: | Patrons | Assignee: | Alexis Ripetti <alexis.ripetti> |
Status: | CLOSED FIXED | QA Contact: | Nick Clemens (kidclamp) <nick> |
Severity: | normal | ||
Priority: | P5 - low | CC: | david, fridolin.somers, gmcharlt, jonathan.druart, kyle.m.hall, kyle, nick, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: |
This fixes the patron search result page so that the results can be sorted using the 'Circ note' column. Before this fix you could not sort the results by this column.
|
Version(s) released in: |
21.11.00,21.05.01,20.11.07
|
Circulation function: | |||
Attachments: |
Bug 28350: Fix borrowernotes sorting for patron search
Bug 28350: Fix borrowernotes sorting for patron search Bug 28350: Fix borrowernotes sorting for patron search |
Description
Alexis Ripetti
2021-05-13 15:00:40 UTC
Created attachment 120937 [details] [review] Bug 28350: Fix borrowernotes sorting for patron search Test plan : - Do a patron search (from the patrons home page). - Try to sort by Circulation note - See that does not work - Apply patch - Do the same steps and see that's working Created attachment 121636 [details] [review] Bug 28350: Fix borrowernotes sorting for patron search Signed-off-by: David Nind <david@davidnind.com> Testing notes (koha-testing-docker): - Added some circulation notes for patrons, including notes starting with letters, numbers and symbols - Followed the test plan in Comment #2 Note: also changed the status to normal, rather than an enhancement - if it isn't working I think it should be a standard bug rather than an enhancement Created attachment 121860 [details] [review] Bug 28350: Fix borrowernotes sorting for patron search Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Pushed to master for 21.11, thanks to everybody involved! Pushed to 21.05.x for 21.05.01 Pushed to 20.11.x for 20.11.07 Not backported to oldoldstable (20.05.x). Feel free to ask if it's needed. |