Attachment 44532 Details for Bug 14687 – Bug 14687: Patron's transaction history changes items' order after paying fines.

[patch] Bug 14687: Patron's transaction history changes items' order after paying fines.

Bug-14687-Patrons-transaction-history-changes-item.patch (text/plain), 1.06 KB, created by Liz Rea on 2015-11-06 01:45:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Liz Rea

Created: 2015-11-06 01:45:04 UTC

Size: 1.06 KB

patch

obsolete

>From 497c99669a953c2d5320cfeb7c781e0580e66180 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Joonas=20Kylm=C3=A4l=C3=A4?= <j.kylmala@gmail.com>
>Date: Mon, 17 Aug 2015 12:40:05 +0000
>Subject: [PATCH] Bug 14687: Patron's transaction history changes items' order
> after paying fines.
>
>Sorts Patron's accounting data consistently from newest to oldest.
>It doesn't depend on anymore to timestamp (which can be same for multiple
>entries) from database but instead uses accountline's id to sort.
>
>Signed-off-by: Liz Rea <liz@catalyst.net.nz>
>---
> C4/Members.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/C4/Members.pm b/C4/Members.pm
>index 171f3c0..e83706e 100644
>--- a/C4/Members.pm
>+++ b/C4/Members.pm
>@@ -1173,7 +1173,7 @@ sub GetMemberAccountRecords {
>                         SELECT * 
>                         FROM accountlines 
>                         WHERE borrowernumber=?);
>-    $strsth.=" ORDER BY date desc,timestamp DESC";
>+    $strsth.=" ORDER BY accountlines_id desc";
>     my $sth= $dbh->prepare( $strsth );
>     $sth->execute( $borrowernumber );
> 
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 14687: 41553 | 44532 | 44552