Bug 12089: Remove use of dt_add_type_uk_date() - JavaScript
authorOwen Leonard <oleonard@myacpl.org>
Thu, 17 Apr 2014 16:02:35 +0000 (12:02 -0400)
committerGalen Charlton <gmc@esilibrary.com>
Mon, 28 Apr 2014 18:43:57 +0000 (18:43 +0000)
This patch removes the now unused dt_add_type_uk_date function from
Koha's custom DataTables javascript file.

To test, be sure all other patches on Bug 12089 are applied. Apply this
patch and search for instances of dt_add_type_uk_date. There should be
none.

Signed-off-by: David Cook <dcook@prosentient.com.au>
Works as described.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes all tests and QA script, works as described.
No regressions found, sorting and searching in all tables
touched by these patches works ok.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
koha-tmpl/intranet-tmpl/prog/en/js/datatables.js

index aa2dcca..c3b882c 100644 (file)
@@ -160,46 +160,6 @@ function dt_add_rangedate_filter(begindate_id, enddate_id, dateCol) {
     );
 }
 
-//Sorting for dates (uk format)
-function dt_add_type_uk_date() {
-  jQuery.fn.dataTableExt.aTypes.unshift(
-    function ( sData )
-    {
-      if (sData.match(/(0[1-9]|[12][0-9]|3[01])\/(0[1-9]|1[012])\/(19|20|21)\d\d/))
-      {
-        return 'uk_date';
-      }
-      return null;
-    }
-  );
-
-  jQuery.fn.dataTableExt.oSort['uk_date-asc']  = function(a,b) {
-    var re = /(\d{2}\/\d{2}\/\d{4})/;
-    a.match(re);
-    var ukDatea = RegExp.$1.split("/");
-    b.match(re);
-    var ukDateb = RegExp.$1.split("/");
-
-    var x = (ukDatea[2] + ukDatea[1] + ukDatea[0]) * 1;
-    var y = (ukDateb[2] + ukDateb[1] + ukDateb[0]) * 1;
-
-    return ((x < y) ? -1 : ((x > y) ?  1 : 0));
-  };
-
-  jQuery.fn.dataTableExt.oSort['uk_date-desc'] = function(a,b) {
-    var re = /(\d{2}\/\d{2}\/\d{4})/;
-    a.match(re);
-    var ukDatea = RegExp.$1.split("/");
-    b.match(re);
-    var ukDateb = RegExp.$1.split("/");
-
-    var x = (ukDatea[2] + ukDatea[1] + ukDatea[0]) * 1;
-    var y = (ukDateb[2] + ukDateb[1] + ukDateb[0]) * 1;
-
-    return ((x < y) ? 1 : ((x > y) ?  -1 : 0));
-  };
-}
-
 // Sorting on html contains
 // <a href="foo.pl">bar</a> sort on 'bar'
 function dt_overwrite_html_sorting_localeCompare() {