QUnit test "jquery.tablesorter.parsers: ISO Dates" fails in Firefox
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Krinkle
	Jan 12 2017, 7:47 PM

Description

Trying to reproduce T112717 I found a (recently introduced) error that doesn't happen on Chrome or Safari:

jquery.tablesorter.parsers: ISO Dates

Sortkey: Date with a time

Expected: 946729830000

Result: 946758630000

Code from:

jquery.tablesorter.parsers.test.js

..
 [ '2000-01-01T12:30:30', true, 946729830000, 'Date with a time' ],
 [ '2000-01-01T12:30:30Z', true, 946729830000, 'Date with a UTC+0 time' ],
`

new Date(946729830000)
> Date 2000-01-01T12:30:30.000Z
new Date(946758630000)
> Date 2000-01-01T20:30:30.000Z

Looks like when the Z is omitted from the ISO date format, Firefox assumes the user's local timezone, whereas Chrome and Safari assume UTC.

Related Objects

Mentioned In: T112717: MediaWiki QUnit test suite has test failures on Firefox
Mentioned Here: T112717: MediaWiki QUnit test suite has test failures on Firefox

Event Timeline

Krinkle created this task.Jan 12 2017, 7:47 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 12 2017, 7:47 PM

Krinkle mentioned this in T112717: MediaWiki QUnit test suite has test failures on Firefox.Jan 12 2017, 7:48 PM

Is there already an upstream bug report with Firefox? http://ecma-international.org/ecma-262/5.1/#sec-15.9.1.15 clearly states "The value of an absent time zone offset is 'Z'."

Note there are an assortment of tests commented, because of a whole slew of inconsistencies between browsers date/time parsing. we can either make it match both browser behaviors, or comment this one as well.

I don't see it listed in their tracking but at least.
https://bugzilla.mozilla.org/show_bug.cgi?id=285663

matmarex added a project: Browser-Support-Firefox.Jan 19 2017, 12:11 PM

This task is now resolved with Change 287449 jquery.tablesorter: Improve detection and handling of isoDate.
Because the parser isoDate build the date instead with new Date("...")
now with isodate.setUTCHours(h,m,s,ms).

Krinkle edited projects, added MediaWiki-User-Interface (Tables); removed MediaWiki-jQuery-Tablesorter--archived.Jul 25 2020, 3:45 AM

QUnit test "jquery.tablesorter.parsers: ISO Dates" fails in FirefoxClosed, ResolvedPublicActions

Description

Related Objects

Event Timeline

QUnit test "jquery.tablesorter.parsers: ISO Dates" fails in Firefox
Closed, ResolvedPublic
Actions