From 2ca4841ea4daa109cb4310a22d88de15d915c373 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Tue, 18 Nov 2014 07:40:24 -0500
Subject: [PATCH] Bug 12971: [QA Followup]

* Makes the status column display "Overdue!" if overdue
* Fixes the due date formatting
* Sorts the checkouts by date due ( oldest to newest )
  Note: I found no evidence that this data was previously sorted,
  so I kept it simple. Sorting based on system preferences could
  be a future enhancement.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

This fixes the issues described for patrons with existing checkouts.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
---
 .../en/modules/members/moremember-print.tt    | 24 +++++++++----------
 members/summary-print.pl                      |  3 +++
 2 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
index e60a8e13ce..1d62a239f5 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt
@@ -41,20 +41,20 @@
                     <th>Status</th>
                 </tr>
 
-                [% FOREACH issueloo IN issues %]
-                    [% IF ( issueloo.red ) %]<tr class="overdue">[% ELSE %]<tr>[% END %]
+                [% FOREACH issue IN issues %]
+                    [% IF ( issue.overdue ) %]<tr class="overdue">[% ELSE %]<tr>[% END %]
                         <td>
-                            [% issueloo.title |html %]
-                            <a href="/cgi-bin/koha/catalogue/detail.pl?item=[% issueloo.itemnumber %]&amp;biblionumber=[% issueloo.biblionumber %]&amp;bi=[% issueloo.biblioitemnumber %]"></a>
+                            [% issue.title |html %]
+                            <a href="/cgi-bin/koha/catalogue/detail.pl?item=[% issue.itemnumber %]&amp;biblionumber=[% issue.biblionumber %]&amp;bi=[% issue.biblioitemnumber %]"></a>
                         </td>
-                        <td>[% issueloo.author %]</td>
-                        <td>[% issueloo.itemcallnumber %]</td>
-                        <td>[% issueloo.itemtype_description %]</td>
-                        <td>[% issueloo.date_due %]</td>
-                        <td>[% issueloo.barcode %]</td>
-                        <td>[% issueloo.charge %]</td>
-                        <td>[% issueloo.replacementprice %]</td>
-                        <td>[% IF ( issueloo.red ) %]Overdue![% ELSE %]&nbsp;[% END %]</td>
+                        <td>[% issue.author %]</td>
+                        <td>[% issue.itemcallnumber %]</td>
+                        <td>[% issue.itemtype_description %]</td>
+                        <td>[% issue.date_due | $KohaDates %]</td>
+                        <td>[% issue.barcode %]</td>
+                        <td>[% issue.charge %]</td>
+                        <td>[% issue.replacementprice %]</td>
+                        <td>[% IF ( issue.overdue ) %]Overdue![% ELSE %]&nbsp;[% END %]</td>
                     </tr>
                 [% END %]
 
diff --git a/members/summary-print.pl b/members/summary-print.pl
index ea227dd6f0..208d2f9b20 100755
--- a/members/summary-print.pl
+++ b/members/summary-print.pl
@@ -97,5 +97,8 @@ sub build_issue_data {
 
         push( @{$return}, \%row );
     }
+
+    @{$return} = sort { $a->{date_due} <=> $b->{date_due} } @{$return};
+
     return $return;
 }
-- 
2.39.5