Javascript form validation doesn't work

Question

I understand that this is very stupid, but the code between tags just doesn't work.

I have a simple HTML registration form. And the js-code makes a simple check is any input is empty.

function validate() 
{
    if (document.forms.registration.email.value == "")
    {
        alert("You must provide an email address.");
        return false;
    }
    else if (document.forms.registration.password1.value != document.forms.registration.password2.value)
    {
        alert("You must provide the same password twice");
        return false;
    }
    else if (!document.forms.registration.agreement.checked)
    {
        alert("You must agree to ours terms and conditions");
        return false;
    }
    return true;
}

For some reason the browser kind of ignores this code and submits the form, even if it's empty.

EDIT:(Added based on OP comment)

<form action="process.php" id="registration" method="get" onsubmit="return.validate();">
        <!-- Other child elements of the form -->
</form>

Are you actually binding this function to your form's submission event? — user229044
– user229044 ♦, Commented Aug 29, 2013 at 16:17
@meagar & Claudio Redi. For both of you. Here is how I am using this js-code: <pre><code><form action="process.php" id="registration" method="get" onsubmit="return.validate();"></code></pre> — user1499804
– user1499804, Commented Aug 29, 2013 at 16:25
You don't need the dot, just return validate();. Check for javascript errors in your debugging console. — showdev
– showdev, Commented Aug 29, 2013 at 16:30
:) I don't have "pre" and "code" tags in my original code. I wanted to add it here, in comments for stylistic reasons. — user1499804
– user1499804, Commented Aug 29, 2013 at 16:52

vdubus · Accepted Answer · 2013-08-29 17:51:25Z

1

Replace onsubmit="return.validate();" by onsubmit="return validate();".
As @showdev did say :

You don't need the dot.

You could also use a javascript library such as jQuery to produce your code.

answered Aug 29, 2013 at 17:51

vdubus

4563 silver badges16 bronze badges

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

1 Comment

user1499804 Over a year ago

Thank you so much! It works now! I am studying web development by Harvard's CS-75 course. And this code I wrote is from example. I was really stupid because the right version was written even in the example. So, really thank you. By the way, why did apache give me an error like this? I mean favicon.ico.

Collectives™ on Stack Overflow

Javascript form validation doesn't work

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related