Attachment 91994 Details for Bug 23427 – Bug 23427: Better sorting of previous checkouts

[patch] Bug 23427: Better sorting of previous checkouts

Bug-23427-Better-sorting-of-previous-checkouts.patch (text/plain), 2.07 KB, created by Fridolin Somers on 2019-08-05 13:33:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Fridolin Somers

Created: 2019-08-05 13:33:02 UTC

Size: 2.07 KB

patch

obsolete

>From e6a2d9a7d07a551788889cc566fefd3bc2c9d330 Mon Sep 17 00:00:00 2001
>From: Fridolin Somers <fridolin.somers@biblibre.com>
>Date: Mon, 5 Aug 2019 15:25:02 +0200
>Subject: [PATCH] Bug 23427: Better sorting of previous checkouts
>
>In patron circulation or details page, previous checkouts are displayed sorted by due date.
>Many checkouts may have same due date so it would be better to sort on timestamp as second sort criteria, like todays's checkouts.
>
>Test plan :
>1) Create for a patron two issues with same due date and a few seconds between them
>2) Create another issue with a different due date
>3) Come back a day later
>4) Set preference previousIssuesDefaultSortOrder = "latest to earliest"
>5) Go to patron circulation page and check sort order is OK : sorted by "Due date" then "Cheked out on"
>6) Go to patron details page and check sort order is OK : sorted by "Due date" then "Cheked out on"
>7) Set preference previousIssuesDefaultSortOrder = "earliest to latest"
>8) Go to patron circulation page and check sort order is OK : sorted by "Due date" then "Cheked out on"
>9) Go to patron details page and check sort order is OK : sorted by "Due date" then "Cheked out on"
>---
> svc/checkouts | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
>diff --git a/svc/checkouts b/svc/checkouts
>index 5bc985a564..278545a72c 100755
>--- a/svc/checkouts
>+++ b/svc/checkouts
>@@ -243,7 +243,9 @@ while ( my $c = $sth->fetchrow_hashref() ) {
> @checkouts_today = reverse(@checkouts_today)
>   unless ( C4::Context->preference('todaysIssuesDefaultSortOrder') eq 'desc' );      # earliest to latest
> 
>-@checkouts_previous = sort { $a->{date_due} cmp $b->{date_due} } @checkouts_previous;    # latest to earliest
>+@checkouts_previous =
>+  sort { $a->{date_due} eq $b->{date_due} ? $a->{timestamp} cmp $b->{timestamp} : $a->{date_due} cmp $b->{date_due} }
>+  @checkouts_previous;                                                               # latest to earliest
> @checkouts_previous = reverse(@checkouts_previous)
>   unless ( C4::Context->preference('previousIssuesDefaultSortOrder') eq 'desc' );    # earliest to latest
> 
>-- 
>2.17.1

Actions: View | Diff | Splinter Review

Attachments on bug 23427: 91993 | 91994 | 94259 | 94260 | 95149 | 95150 | 95151