jQuery string to Date issue

Question

I have been wrestling with this for a couple hours now at least and would really appreciate any insight as to what I'm doing wrong.

I want to compare 2 date objects and need to convert one date that I'm grabbing from a datetime input. when I put the variable being set into a new Date(myVariableHere); it doesn't work, though if I add in the string equivalent new Date(2014-01-05T04:47Z); it works fine (so it seems).

My fiddle here: http://jsfiddle.net/centinel3/Y8kWf/

note: I tried also using moment.js and wasn't able to make it do what I needed it to.

Some code snippets below and thank you in advance.

        // HTML            
        <input id="start-date" type="datetime"/>            

        // Gets value from start date "datetime selectbox"
        var userStartDate = ""; 
        $("#start-date").change(function () {
            userStartDate = $(this).val();
        });

        var startDate = new Date("2014-01-05T04:47Z"); // works
        var startDate = new Date(userStartDate ); // doesn't work

Working/fixed code below (this also adjusts the format so that it plays well in safari:

        var userStartDate, startDate;
        $("#start-date").change(function () {
            userStartDate = ($(this).val()).replace(/-/g, "/").replace("T", " ");
            startDate = new Date(userStartDate);
        });

Can you do a jsfiddle ? With html :)

yachaka
– yachaka

2014-01-05 05:14:54 +00:00
Commented Jan 5, 2014 at 5:14 — yachaka
– yachaka, Commented Jan 5, 2014 at 5:14
Here is my fiddle: jsfiddle.net/centinel3/Y8kWf/3

Centinel3
– Centinel3

2014-01-05 12:47:30 +00:00
Commented Jan 5, 2014 at 12:47 — Centinel3
– Centinel3, Commented Jan 5, 2014 at 12:47

vinod · Accepted Answer · 2014-01-05 13:06:07Z

2

You need to add the code that uses userStartDate within the event handler:

    $("#start-date").change(function () {
        startDate = new Date($(this).val());
    });

By setting startDate outside the event handler, nothing happens when the value changes and startDate will only have the original value in it.

If you want something to happen when the user changes the date, that code should go in the event handler, or be called from within it.

edited Jan 5, 2014 at 13:06

answered Jan 5, 2014 at 5:30

vinod

2,36819 silver badges26 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Centinel3 Over a year ago

That isn't the issue. Please see my fiddle for what is going on. It's not that the variable isn't being populated, it's that it doesn't convert to a Date object. My fiddle is here: jsfiddle.net/centinel3/Y8kWf/3

vinod Over a year ago

@Centinel3 I see. Just replace userStartDate = $(this).val(); with startDate = new Date($(this).val()); and it looks like it works fine. I've updated my answer above.

Centinel3 Over a year ago

Thank you for your help! I chose another response that was a little more descriptive and also declared the variables outside the event handler. you were also correct. Thanks again!

Reza Mamun · Accepted Answer · 2014-01-05 12:54:28Z

0

Just update the following lines:

var userStartDate = "";
$("#start-date").change(function () {
    userStartDate = $(this).val();
});

var startDate = new Date(userStartDate);

TO:

var userStartDate = "", startDate;

$("#start-date").change(function () {
    userStartDate = $(this).val();
    startDate = new Date(userStartDate); //so each time it gets the updated time;
});

answered Jan 5, 2014 at 12:54

Reza Mamun

6,2531 gold badge47 silver badges44 bronze badges

1 Comment

Centinel3 Over a year ago

This works! Thanks man. I had to also refactor a bit due to the format not working in Safari. I adjusted this post to reflect that. Thanks again! -Chris

alexej_d · Accepted Answer · 2014-01-05 12:53:25Z

0

You need to define the startDate variable inside the "change" event. Because if you do it outside the change event would probably fire after you define the variable – hence it gets the value undefined.

    var userStartDate, startDate; 

    $("#start-date").change(function () {
        userStartDate = $(this).val();
        startDate = new Date(userStartDate);
    });

answered Jan 5, 2014 at 12:53

alexej_d

4485 silver badges13 bronze badges

1 Comment

Centinel3 Over a year ago

You had the same answer as the developer who posted just a hare ahead of you. Thanks for the response!

Collectives™ on Stack Overflow

jQuery string to Date issue

3 Answers 3

3 Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related