TableSorter: Sort unrecognized dates as -Infinity

Follow-up to: I664b4cc
Bug: T174814
Change-Id: I028681172be7a30e51cab42847c4fc35e9358aca

diff --git a/resources/src/jquery/jquery.tablesorter.js b/resources/src/jquery/jquery.tablesorter.js
index 922da31..ecd376a 100644 (file)
--- a/resources/src/jquery/jquery.tablesorter.js
+++ b/resources/src/jquery/jquery.tablesorter.js
@@ -1163,7 +1163,7 @@
                                match = s.match( ts.rgx.isoDate[ 1 ] );
                        }
                        if ( !match ) {
-                               return 0;
+                               return -Infinity;
                        }
                        // Month and day
                        for ( i = 2; i <= 4; i += 2 ) {

diff --git a/tests/qunit/suites/resources/jquery/jquery.tablesorter.parsers.test.js b/tests/qunit/suites/resources/jquery/jquery.tablesorter.parsers.test.js
index 257699a..01589c3 100644 (file)
--- a/tests/qunit/suites/resources/jquery/jquery.tablesorter.parsers.test.js
+++ b/tests/qunit/suites/resources/jquery/jquery.tablesorter.parsers.test.js
@@ -174,7 +174,8 @@
        parserTest( 'Y Dates', 'date', YDates );
 
        ISODates = [
-               [ '2000',               false,  946684800000, 'Plain 4-digit year' ],
+               [ '',           false,  -Infinity, 'Not a date' ],
+               [ '2000',       false,  946684800000, 'Plain 4-digit year' ],
                [ '2000-01',    true,   946684800000, 'Year with month' ],
                [ '2000-01-01', true,   946684800000, 'Year with month and day' ],
                [ '2000-13-01', false,  978307200000, 'Non existant month' ],