Bug 36974

Summary: Sort summary-print by ascending due date
Product: Koha Reporter: Thibaud Guillot (thibaud_g) <thibaud.guillot>
Component: PatronsAssignee: Thibaud Guillot (thibaud_g) <thibaud.guillot>
Status: RESOLVED DUPLICATE QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, s.deflache
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 36974: Sort summary print by ascending due date

Description Thibaud Guillot (thibaud_g) 2024-05-28 12:17:36 UTC
Hello,

This is a small correction, but the summary-print.pl sort by descending due date... it would be preferable to sort it in ascending order to obtain the earliest due date on the first line.
Comment 1 Katrin Fischer 2024-05-28 12:20:43 UTC
Maybe we could make the sort configurable - the page is supposed to use column configuration, but I seem to remember another bug about it.
Comment 2 Thibaud Guillot (thibaud_g) 2024-05-28 12:23:12 UTC
Created attachment 167220 [details] [review]
Bug 36974: Sort summary print by ascending due date

Test plan:
1) Choose a patron with some checkouts
2) On print menu choose "Print summary"
3) Normally the list is sorted on descending due date.. and if the date
   is the same for multiple items, on issue_id (from database).
4) Apply this patch
5) Now the sort on due date is ascending to show the earliest due date on the first line.

Sponsored by BibLibre
Comment 3 Thibaud Guillot (thibaud_g) 2024-05-28 12:24:25 UTC
(In reply to Katrin Fischer from comment #1)
> Maybe we could make the sort configurable - the page is supposed to use
> column configuration, but I seem to remember another bug about it.

Hi Katrin, you replied quickly :) 
I was already writing the commit... how do you see this sorting, where would it take place?
Comment 4 Thibaud Guillot (thibaud_g) 2024-05-28 12:24:49 UTC
"the sort configuration" I mean :)
Comment 5 Katrin Fischer 2024-05-28 12:51:26 UTC
In administration > table configuration.

See also:
Bug 36475 - members/summary-print.pl tables cannot be column configured

Some tables already allow configuration of sorting.
Comment 6 Owen Leonard 2024-06-18 17:38:58 UTC
I've submitted a patch for Bug 36475 which I think makes this bug obsolete.
Comment 7 Thibaud Guillot (thibaud_g) 2024-08-09 13:59:29 UTC
Hello

Yes I test it and make this one obsolete, thanks :)
Comment 8 Thibaud Guillot (thibaud_g) 2024-08-09 14:00:10 UTC

*** This bug has been marked as a duplicate of bug 36475 ***