Attachment 40242 Details for Bug 14404 – Bug 14404 - Checkouts default sort order for Self Checkout (SCO) confusing for patrons

[patch] Bug 14404 - Checkouts default sort order for Self Checkout (SCO) confusing for patrons

Bug-14404---Checkouts-default-sort-order-for-Self-.patch (text/plain), 3.85 KB, created by Kyle M Hall (khall) on 2015-06-17 13:28:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2015-06-17 13:28:21 UTC

Size: 3.85 KB

patch

obsolete

>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>
>                                     
>                                     <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

Actions: View | Diff | Splinter Review

Attachments on bug 14404: 40242 | 40249 | 40832 | 40861 | 40862