Javascript confirm() not working correctly

Question

I have this page and I am trying to confirm a date that's from an input text field vs the current date. If the date from the input field is later then the current date, the confirm box needs to come up to confirm that they are putting down the right date. So the 'OK' button needs to finish submitting the data and the 'Cancel' button needs to return them back to the AJAX form.

if($('sdate' > curdate)) {
    confirm("The date you chose " + curdate + "is in the future OK to continue");
}

I also wanted the 'sdate' to appear in the confirm box instead of curdate, but whenever I try that the form no longer appears.

What is `curdate`? And what will `$(true)` or `$(false)` yield? Is that jQuery? — Bergi, May 01 '13 at 13:46

Denys Séguret · Answer 1 · 2013-05-01T13:50:35.167

2

Instead of

$('sdate' > curdate)

you probably wanted

$('sdate').val() > curdate

that is a comparison of the dates (as strings but it should work if the dates are well formed which is the case for an input of type date or an input of type text with a standard plugin).

Note that you don't use the returned value from the confirm call. You would usually do something like

if ($('sdate').val() > curdate) {
    if (confirm("someText")) {
        // do something
    }
}

edited May 01 '13 at 13:50

answered May 01 '13 at 13:45

Denys Séguret

372,613
87
782
758

To be more specific about "well formed," I don't think string comparison will work unless both dates are "yyyy-mm-dd" format. – CrayonViolent May 01 '13 at 13:50
@CrayonViolent Well, that's exactly how they're formed with an input of type date. – Denys Séguret May 01 '13 at 13:50
it didnt work when I put that in but I did use if($('sdate').value > curdate){ ...and that seemed to work in the comparison part. But its still saving even if I hit 'Cancel'...Also how can I put sdate in the confirm() instead of curdate? – Lynx May 01 '13 at 13:59
@dystroy note that the date input type is html5. Also, the OP alluded to using an "input *text* field" not input date field. – CrayonViolent May 01 '13 at 14:03
Also want to point out that currently Chrome is the only browser that really supports the date input type. – CrayonViolent May 01 '13 at 14:12

score 1 · Answer 2 · answered May 01 '13 at 13:50

You need to do this:

var date1 = new Date($('#sdate').val());
var date2 = new Date(curdate);
if (date1 > date2 ) {
    confirm("The date you chose " + curdate + "is in the future OK to continue");
}

Element not fully wrapped. It should be like this $('#sdate')
'#' for the id missing. Not sure, if its a id or class. If class, add a '.' in place of #

score 0 · Answer 3 · edited May 23 '17 at 12:18

In your current script, the confirm() call doesn't actually do anything.

What ever you're trying to do it will need to incorporate the following in some form or another.

if(confirm("The date you chose " + curdate + "is in the future OK to continue"))
    //Do stuff

Goin' by what you've posted and the answer posted by @dystroy you're probably looking for

if($("#sdate").val() > curdate && confirm("The date you chose " + curdate + "is in the future OK to continue")
    //Do stuff

In the above script both conditions are present in the one conditional. As per @dystroy's answer, I've included the .val() call. The other change is that it has been presumed that you're looking for an element with id="sdate", so the selector include the id symbol #

Edit (after OP's comment) - Updated based on this stackoverflow post

If you're trying to prevent a form submission then you should attach the function to the form in a manner similar to this

<form name="myForm" onsubmit="return validateMyForm();">

Once you've bound the event you could update your code to something like

if($("#sdate").val() > curdate && confirm("The date you chose " + curdate + "is in the future OK to continue")
    return true;
else
    return false;

This should only submit the form if the confirm returns true and the date inequality is met.

this is what I have that works currently if($('sdate').value > curdate ){ confirm("The date you chose " + curdate + "is in the future OK to continue"); } it all works except when I hit cancel on the future date..it will still save. using && made the form not submit at all. — Lynx, May 01 '13 at 14:13

score 0 · Accepted Answer · answered May 02 '13 at 01:36

0

I was able to get it to work with this

var d=$('sdate').value;

if(d > curdate)) {

  var dates confirm("The date you chose " + d + "is in the future OK to continue");
  if(!dates) return false;
}

answered May 02 '13 at 01:36

Lynx

1,462
5
32
59

Javascript confirm() not working correctly

4 Answers4