JavaScript date comparison fails in IE8

Question

I have a function that converts a date to YYYY-MM-DD from DD/MM/YYYY.

This works in all browsers apart from IE8, for some reason, when creating a new Date object, it is returning NaN.

Basic implementation of the code http://jsfiddle.net/bX83c/1/

var compareDate = function(value){
    var dateFragements = value.split('/');
    if (dateFragements.length == 3) {
        var currentDate = new Date();
            currentDate.setHours(0, 0, 0, 0);

        var startDate = new Date(dateFragements[2] + '-' + dateFragements[1] + '-' + dateFragements[0]);

        if (startDate >= currentDate) {
            return true;
        } else {
            return false;
        }
    }   
}
alert(compareDate('17/09/2013'));

I think you have your answer in one of the following: http://stackoverflow.com/questions/2182246/javascript-dates-in-ie-nan-firefox-chrome-ok or: http://stackoverflow.com/questions/11020658/javascript-json-date-parse-in-ie7-ie8-returns-nan Simple googling... — Mircea Botez, Sep 05 '13 at 09:28

999k · Accepted Answer · 2014-12-15T09:50:57.023

3

Intialise your date like this. It will work in all browsers

var startDate = new Date(dateFragements[2] , dateFragements[1] , dateFragements[0]);

There are 4 ways in which Date object can be intialised using constructor

new Date() // current date and time
new Date(milliseconds) //milliseconds since 1970/01/01
new Date(dateString)
new Date(year, month, day, hours, minutes, seconds, milliseconds)

String in Date object doesn't mean it will accept all date strings. If you want to give a string as input give this. (dateFragements[2] +'/' + dateFragements[1] + '/' + dateFragements[0]);. (/ as the separator) It will be supported in all browsers

edited Dec 15 '14 at 09:50

answered Sep 05 '13 at 09:22

999k

6,257
2
29
32

That seems to have fixed it. Do you know why IE8 fails when the Date object is initialised with a string? – CharliePrynn Sep 05 '13 at 09:25
1

By accepting string, it doesn't mean it will accept all strings. If you want to give a string as input give this. (dateFragements[2] +'/' + dateFragements[1] + '/' + dateFragements[0]);. It will be supported in all browsers – 999k Sep 05 '13 at 09:32

score 2 · Answer 2 · answered Sep 05 '13 at 09:30

IE8 expects '/' as the separator in a date string, that's why your function fails. It can be simplified to:

var compareDate = function(value){
 var dateFragements = value.split('/');
 if (dateFragements.length == 3) {
  var currentDate = function(){ return (this.setHours(0),
                                        this.setMinutes(0),
                                        this.setSeconds(0),
                                        this); }.call(new Date)
     ,startDate = new Date([dateFragements[2],
                            dateFragements[1],
                            dateFragements[0]].join('/'));
  return startDate>=currentDate;
 }   
}

score 1 · Answer 3 · answered Sep 05 '13 at 09:33

1

new Date(dateString)

accepts the following formats (only):

"October 13, 1975 11:13:00"
"October 13, 1975 11:13"
"October 13, 1975"

answered Sep 05 '13 at 09:33

lshettyl

8,166
4
25
31

JavaScript date comparison fails in IE8

3 Answers3