Attachment 41553 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.02 KB, created by Joonas Kylmälä on 2015-08-17 12:56:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Joonas Kylmälä

Created: 2015-08-17 12:56:11 UTC

Size: 1.02 KB

patch

obsolete

>From dc8cc914fcef89fc42fb2325557e7078d9d1284d 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.
>---
> C4/Members.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/C4/Members.pm b/C4/Members.pm
>index 95fb4e7..1f37de0 100644
>--- a/C4/Members.pm
>+++ b/C4/Members.pm
>@@ -1167,7 +1167,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