From 7146495d2dfcff825a7e786e129b58b5d393b8c2 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Tue, 16 Jun 2015 19:05:10 -0400
Subject: [PATCH] Bug 14404 - Checkouts default sort order for Self Checkout (SCO) confusing for patrons

Libraries are reporting that patrons are very confused during
self-checkout. The problem is they are expecting the list of checkouts
to be in the order they checked out the items ( first checkout on the
bottom, last item checked out on top ). However, the checkouts
table is sorted by title ( ascending ) then due date ( descending ).
This is not intuitive.

Test Plan:
1) Enable Koha's self checkout
2) Use the SCO to check out a random assortment of items,
   make sure you don't check them out in alphabetical order
3) Note the order of the items in the list is not based on the order
   you checked them out in
4) Apply this patch
5) Refresh the page
6) Note the items are now in the order you checked them out
   with the last on top and the first on bottom
---
 .../opac-tmpl/bootstrap/en/modules/sco/sco-main.tt |   17 ++++++++++++++---
 1 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
index cb0b752..c7cfe68 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
@@ -213,11 +213,21 @@
                                     <caption>Checkouts for [% borrowername %] <span class="count">([%   issues_count %] total)</span></caption>
                                     <!-- ISSUES TABLE ROWS -->
                                     <thead>
-                                        <tr><th class="anti-the">Title</th><th>Call no.</th><th class="title-string">Due</th><th class="nosort">Renew</th>[% UNLESS ( nofines ) %]<th>Fines</th>[% END %]</tr>
+                                        <tr>
+                                            <th class="no-show">Checked out on</th>
+                                            <th class="anti-the">Title</th>
+                                            <th>Call no.</th>
+                                            <th class="title-string">Due</th>
+                                            <th class="nosort">Renew</th>
+                                            [% UNLESS ( nofines ) %]
+                                                <th>Fines</th>
+                                            [% END %]
+                                        </tr>
                                     </thead>
                                     <tbody>
                                         [% FOREACH ISSUE IN ISSUES %]
                                             <tr>
+                                                <td>[% ISSUE.issuedate %]</td>
                                                 <td>
                                                     [% UNLESS ( noitemlinks ) %]
                                                         <a href="/cgi-bin/koha/opac-detail.pl?bib=[% ISSUE.  biblionumber %]">[% ISSUE.title |html %]</a>
@@ -347,9 +357,10 @@
             dofocus();
             [% IF ( patronid ) %]sco_init();[% END %]
             $("#loanTable").dataTable($.extend(true, {}, dataTablesDefaults, {
-                "aaSorting": [ [2, "desc"], [0, "asc"] ],
+                "aaSorting": [ 0 ],
                 "aoColumnDefs": [
-                    { "aTargets": [ "nosort" ],"bSortable": false,"bSearchable": false },
+                    { "aTargets": [ "nosort" ], "bSortable": false, "bSearchable": false },
+                    { "aTargets": [ "no-show" ], "bVisible": false, "bSearchable": false },
                     { "sType": "anti-the", "aTargets" : [ "anti-the" ] },
                     { "sType": "title-string", "aTargets" : [ "title-string" ] }
                 ]
-- 
1.7.2.5