Bugzilla – Attachment 32299 Details for
Bug 13039
Vendor search: sorting of "item count" and "biblio count" columns can be incorrect
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 13039 - Vendor search: sorting of "item count" and "biblio count" columns can be incorrect
Bug-13039---Vendor-search-sorting-of-item-count-an.patch (text/plain), 5.59 KB, created by
Chris Cormack
on 2014-10-14 12:56:59 UTC
(
hide
)
Description:
Bug 13039 - Vendor search: sorting of "item count" and "biblio count" columns can be incorrect
Filename:
MIME Type:
Creator:
Chris Cormack
Created:
2014-10-14 12:56:59 UTC
Size:
5.59 KB
patch
obsolete
>From 5c9ebc0db57e3c6e05e790631a14dac1542ba559 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Tue, 7 Oct 2014 14:54:04 -0400 >Subject: [PATCH] Bug 13039 - Vendor search: sorting of "item count" and > "biblio count" columns can be incorrect > >On the vendor search results page if some cells contain textual data the >"item count" and "biblio count" columns will sort incorrectly. This >patch sets an explicit numeric sort on these columns. In doing so this >patch also changes the existing column sorting configuration to use >table header cell classes instead of sorting based on index. > >This patch also corrects two instances of unescaped ampersands. > >To test, search for a vendor which has multiple baskets, at least one of >which should contain canceled orders. Confirm that sorting by item >count, biblio count, date, and closed all work correctly. > >Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> >--- > .../prog/en/modules/acqui/booksellers.tt | 35 +++++++++++----------- > 1 file changed, 18 insertions(+), 17 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt >index 546f597..3db9188 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt >@@ -18,10 +18,9 @@ $(document).ready(function() { > 'bFilter': false, > 'bInfo': false, > "aaSorting": [[ 1, "asc" ]], >- "aoColumns": [ >- null,null,null,null,null,null,{ "sType": "title-string" },null,{ "sType": "title-string" },null >- ], >- 'aoColumnDefs': [ >+ "aoColumnDefs": [ >+ { "sType": "title-string", "aTargets" : [ "title-string" ] }, >+ { "sType": "title-numeric", "aTargets" : [ "title-numeric" ] }, > { 'aTargets': [-1], 'bSortable': false } > ] > })); >@@ -97,13 +96,13 @@ $(document).ready(function() { > <tr> > <th>No.</th> > <th>Name</th> >- <th>Item count</th> >- <th>Biblio count</th> >+ <th class="title-numeric">Item count</th> >+ <th class="title-numeric">Biblio count</th> > <th>Items expected</th> > <th>Created by</th> >- <th>Date</th> >+ <th class="title-string">Date</th> > <th>Basket group</th> >- <th>Closed</th> >+ <th class="title-string">Closed</th> > <th> </th> > </tr> > </thead> >@@ -117,16 +116,18 @@ $(document).ready(function() { > <td>[% basket.basketno %]</td> > <td>[% basket.basketname %]</td> > <td> >- [% basket.total_items %] >- [% IF basket.total_items_cancelled %] >- ([% basket.total_items_cancelled %] cancelled) >- [% END %] >+ <span title="[% basket.total_items %]">[% basket.total_items %] >+ [% IF basket.total_items_cancelled %] >+ ([% basket.total_items_cancelled %] cancelled) >+ [% END %] >+ </span> > </td> > <td> >- [% basket.total_biblios %] >- [% IF basket.total_biblios_cancelled %] >- ([% basket.total_biblios_cancelled %] cancelled) >- [% END %] >+ <span title="[% basket.total_biblios %]">[% basket.total_biblios %] >+ [% IF basket.total_biblios_cancelled %] >+ ([% basket.total_biblios_cancelled %] cancelled) >+ [% END %] >+ </span> > </td> > <td>[% basket.expected_items %]</td> > <td> >@@ -140,7 +141,7 @@ $(document).ready(function() { > [% IF basketgroup.closed %] > [% basketgroup.name %] (closed) > [% ELSE %] >- <a href="/cgi-bin/koha/acqui/basketgroup.pl?op=add&booksellerid=[% basket.booksellerid %]&basketgroupid=[% basketgroup.id %]">[% basketgroup.name %]</a> >+ <a href="/cgi-bin/koha/acqui/basketgroup.pl?op=add&booksellerid=[% basket.booksellerid %]&basketgroupid=[% basketgroup.id %]">[% basketgroup.name %]</a> > [% END %] > [% END %] > </td> >-- >1.9.1
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 13039
:
32058
|
32299
|
32329