Validate Date input with Javascript

Question

I have a input field date in my form and would like to add Javascript validation to ensure that its always greater than today's date. I tried the following but the function doesnt seem to be called when the user clicks on the input field and makes a change.

<input type="text" name="InsertRecordGuestOrders_Delivery_date" id="InsertRecordGuestOrders_Delivery_date" value=""/>

<script type="text/javascript">

function validateDate()
{
var del_date = document.getElementById('InsertRecordGuestOrders_Delivery_date').value;

if (Date.parse(del_date) < Date.now()) {
    document.getElementById('InsertRecordGuestOrders_Delivery_date').value = '';
    alert("Delivery date has to be later than today");
    }
}

document.getElementById('InsertRecordGuestOrders_Delivery_date').onChange = validateDate();

</script>

Any suggestions on what I'm doing wrong?

Thanks in advance.

Possible duplicate of How to validate date with format "mm/dd/yyyy" in JavaScript? — Zorgatone
– Zorgatone, Commented Dec 20, 2015 at 17:12

jbabey · Accepted Answer · 2013-04-03 16:57:25Z

5

You want to assign the validateDate function to the onchange property, not execute the function and assign its return value. Change this:

document.getElementById('...').onChange = validateDate();

to this:

document.getElementById('...').onChange = validateDate;

answered Apr 3, 2013 at 16:57

jbabey

46.7k12 gold badges73 silver badges94 bronze badges

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

Comments

Trevor Dixon · Accepted Answer · 2013-04-03 17:01:39Z

1

This line:

document.getElementById('InsertRecordGuestOrders_Delivery_date').onChange = validateDate();

Should be:

document.getElementById('InsertRecordGuestOrders_Delivery_date').onchange = validateDate;

Notice the parentheses are gone. If you invoke it immediately, you're assigning the return value of validateDate() as the onchange handler. You want to assign the function itself as the handler, not the return value of the function.

In addition, it should be onchange, not onChange.

answered Apr 3, 2013 at 17:01

Trevor Dixon

24.6k13 gold badges77 silver badges114 bronze badges

1 Comment

Maverick Over a year ago

By the way, good catch on the "onchange". I tried it out in JsFiddle and it didnt work until I changed "onChange" -> "onchange". Appreciate the help! Thanks again!

Collectives™ on Stack Overflow

Validate Date input with Javascript

2 Answers 2

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related