Bugzilla – Attachment 31511 Details for
Bug 12899
Row grouping in checkouts table is alphabetical and layout depends on words chosen in translation
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 12899 - Row grouping in checkouts table is alphabetical and depends on translation
Bug-12899---Row-grouping-in-checkouts-table-is-alp.patch (text/plain), 3.28 KB, created by
Kyle M Hall
on 2014-09-10 15:00:48 UTC
(
hide
)
Description:
Bug 12899 - Row grouping in checkouts table is alphabetical and depends on translation
Filename:
MIME Type:
Creator:
Kyle M Hall
Created:
2014-09-10 15:00:48 UTC
Size:
3.28 KB
patch
obsolete
>From 779313bcfece8531130eb94779786fddbf96e582 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Wed, 10 Sep 2014 08:56:24 -0400 >Subject: [PATCH] Bug 12899 - Row grouping in checkouts table is alphabetical and depends on translation > >The sort order of the "today's checkouts" and "previous checkouts" row >groupings depends on the label, so in English "today's checkouts" comes >first. However, in other languages the reverse alphabetical order is >incorrect resulting in "previous checkouts" coming first. > >This patch adds a dummy column with numeric data on which the sorting >can be done. This should make it translation-agnostic. > >To test, apply the patch and install or update a translation which will >demostrate the problem (sv-SE for instance). > >- Clear your browser cache and switch to the English templates. >- Check out some items to a patron who has checkouts from a previous > day. >- Confirm that the sorting of the "today's checkouts" and "previous > checkouts" row groups is correct. >- Switch to the new/updated translation and reload the circulation page > for that patron. Confirm that the sort remains correct. >- Confirm that the checkouts table looks correct and that other features > (sorting, checkboxes) still work correctly. > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js | 12 +++++++++++- > .../prog/en/modules/circ/circulation.tt | 1 + > 2 files changed, 12 insertions(+), 1 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js b/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js >index 6efe9f9..105648a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js >+++ b/koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js >@@ -138,6 +138,15 @@ $(document).ready(function() { > { > "mDataProp": function( oObj ) { > if ( oObj.issued_today ) { >+ return "0"; >+ } else { >+ return "100"; >+ } >+ } >+ }, >+ { >+ "mDataProp": function( oObj ) { >+ if ( oObj.issued_today ) { > return "<strong>" + TODAYS_CHECKOUTS + "</strong>"; > } else { > return "<strong>" + PREVIOUS_CHECKOUTS + "</strong>"; >@@ -329,8 +338,9 @@ $(document).ready(function() { > }, > }).rowGrouping( > { >+ iGroupingColumnIndex: 1, > iGroupingOrderByColumnIndex: 0, >- sGroupingColumnSortDirection: "desc" >+ sGroupingColumnSortDirection: "asc" > } > ); > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >index 9734c0f..1204d1e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >@@ -688,6 +688,7 @@ No patron matched <span class="ex">[% message %]</span> > <thead> > <tr> > <th scope="col"> </th> >+ <th scope="col"> </th> > <th scope="col">Due date</th> > <th scope="col">Due date</th> > <th scope="col">Title</th> >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 12899
:
31502
|
31511
|
31599
|
31620
|
31621