Attachment 48677 Details for Bug 11203 – Bug 11203 - Datatables in acqusitions do not ignore "stopwords" in titles

[patch] Bug 11203 - Datatables in acqusitions do not ignore "stopwords" in titles

Bug-11203---Datatables-in-acqusitions-do-not-ignor.patch (text/plain), 3.68 KB, created by Jonathan Druart on 2016-03-04 11:09:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2016-03-04 11:09:16 UTC

Size: 3.68 KB

patch

obsolete

>From 71673e6aa4ea15dfebae945594ddd987ba560d5f Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Tue, 5 Nov 2013 11:31:20 -0500
>Subject: [PATCH] Bug 11203 - Datatables in acqusitions do not ignore
> "stopwords" in titles
>
>Sorting by the "summary" column does not produce the preferred results.
>Title beginning with 'a', 'an', or 'the' are sorted using those
>articles.
>
>Test plan:
>1) Place an order for 2 items with the titles "Alpha" and "The Alpha",
>   along with some other records with titles starting with something
>   between 'a' and 't', and 't' and 'z'
>2) Sort the "pending orders" table, note the incorrect sorting
>3) Receive all the items
>4) Sort the "already received" table, note the incorrect sorting
>5) Apply the patch
>6) Repeat steps 1-4, note the corrected sorting
>7) Check acqui/basket.pl for correct sorting
>8) Check acqui/invoice.pl for correct sorting
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt  | 7 ++++---
> koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt | 5 ++++-
> koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt  | 4 ++--
> 3 files changed, 10 insertions(+), 6 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt
>index 7bcbbbe..11a09b5 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt
>@@ -85,11 +85,12 @@
>         var orderst = $("#orders").dataTable($.extend(true, {}, dataTablesDefaults, {
>             "sPaginationType": "four_button",
>             [% IF ( active ) %]
>-              [% UNLESS ( closedate ) %]
>                 "aoColumnDefs": [
>-                    { "aTargets": [ -1, -2 ], "bSortable": false, "bSearchable": false },
>+                    [% UNLESS ( closedate ) %]
>+                        { "aTargets": [ -1, -2 ], "bSortable": false, "bSearchable": false },
>+                    [% END %]
>+                    { "sType": "anti-the", "aTargets": [ 0 ] }
>                 ],
>-              [% END %]
>             [% END %]
>         } ) );
>         var cancelledorderst = $("#cancelledorderst").dataTable($.extend(true, {}, dataTablesDefaults, {
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt
>index 3c0991e..8071355 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt
>@@ -27,7 +27,10 @@
>             bInfo: false,
>             bPaginate: false,
>             bFilter: false,
>-            sDom: "t"
>+            sDom: "t",
>+            "aoColumnDefs": [
>+                { "sType": "anti-the", "aTargets": [ 0 ] }
>+            ]
>         }));
> [% IF ( (Koha.Preference('AcqEnableFiles')) && files ) %]
>         $("#invoice_files_table").dataTable($.extend(true, {}, dataTablesDefaults, {
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
>index 05de288..f705694 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
>@@ -32,7 +32,7 @@
>                 { "sType": "html" },
>                 { "sType": "html" },
>                 { "sType": "num-html" },
>-                { "sType": "html" },
>+                { "sType": "anti-the" },
>                 null,
>                 null,
>                 null,
>@@ -76,7 +76,7 @@
>                 { "sType": "html" },
>                 { "sType": "html" },
>                 { "sType": "num-html" },
>-                null,
>+                { "sType": "anti-the" },
>                 null,
>                 null,
>                 null,
>-- 
>2.7.0

Actions: View | Diff | Splinter Review

Attachments on bug 11203: 22736 | 22818 | 48677 | 50399 | 50409 | 50482