| Summary: | Sorting by due date in overdues listing does not work as expected | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Josef Moravec <josef.moravec> |
| Component: | Circulation | Assignee: | Josef Moravec <josef.moravec> |
| Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
| Severity: | minor | ||
| Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall, m.de.rooy |
| 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: | ||
| Circulation function: | |||
| Attachments: |
Bug 19438: Fix sorting by date due in overdues.pl
Bug 19438: Fix sorting by date due in overdues.pl Bug 19438: Fix sorting by date due in overdues.pl |
||
|
Description
Josef Moravec
2017-10-09 08:24:58 UTC
Created attachment 67831 [details] [review] Bug 19438: Fix sorting by date due in overdues.pl Test plan: 0) Do not apply the patch 1) Have some overdued checkouts in your database 2) Go to circulation -> overdues (overdues.pl) 3) Try to sort the overdues table by due date and notice it is sorted alphabetically, not by date 4) Apply the patch 5) Try again and confirm, it is sorted by date Created attachment 67971 [details] [review] Bug 19438: Fix sorting by date due in overdues.pl Test plan: 0) Do not apply the patch 1) Have some overdued checkouts in your database 2) Go to circulation -> overdues (overdues.pl) 3) Try to sort the overdues table by due date and notice it is sorted alphabetically, not by date 4) Apply the patch 5) Try again and confirm, it is sorted by date Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 68055 [details] [review] Bug 19438: Fix sorting by date due in overdues.pl Test plan: 0) Do not apply the patch 1) Have some overdued checkouts in your database 2) Go to circulation -> overdues (overdues.pl) 3) Try to sort the overdues table by due date and notice it is sorted alphabetically, not by date 4) Apply the patch 5) Try again and confirm, it is sorted by date Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Pushed to master for 17.11, thanks to everybody involved! |