Attachment 170556 Details for Bug 37636 – Bug 37636: Remove sort of pending_checkouts by timestamp

[patch] Bug 37636: Remove sort of pending_checkouts by timestamp

Bug-37636-Remove-sort-of-pendingcheckouts-by-times.patch (text/plain), 1.38 KB, created by David Nind on 2024-08-21 10:44:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Nind

Created: 2024-08-21 10:44:46 UTC

Size: 1.38 KB

patch

obsolete

>From 475be5b0ea31ee2cc0f42b85aa12d352825ddd9a Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Tue, 20 Aug 2024 13:25:26 +0000
>Subject: [PATCH] Bug 37636: Remove sort of pending_checkouts by timestamp
>
>Renewals or automatic renewal attempts can change the timestamp of an issue and cause a sort
>on this field to functionally randomize the order of checkouts returned.
>
>Prior to bug 19935 the GetPendingCheckouts routine returned items in issuedate order.
>
>Removing the sort on timestamp will return the previous behaviour and provide a more predictable sorting
>
>To test:
>1 - Checkout some items to a patron
>2 - Print a checkout slip - items are in the order they were checked out
>3 - Renew an item,print the slip again it is now at the top of the list
>4 - Apply patch, restart_all
>5 - Print slip again - the items are now ordered in the order they were checked out.
>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> Koha/Patron.pm | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/Koha/Patron.pm b/Koha/Patron.pm
>index 7234a3be5b..121073e679 100644
>--- a/Koha/Patron.pm
>+++ b/Koha/Patron.pm
>@@ -1438,7 +1438,6 @@ sub pending_checkouts {
>         {},
>         {
>             order_by => [
>-                { -desc => 'me.timestamp' },
>                 { -desc => 'issuedate' },
>                 { -desc => 'issue_id' }, # Sort by issue_id should be enough
>             ],
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 37636: 170496 | 170556 | 170558 | 170604 | 170605 | 171453 | 171454