Javascript validation: Return false not working, form still submitting

Question

I'm using javascript for form validation but after submitting and the code returning false the form still submits. I used the 'fixes' in other topics I found but the form still submits.

Here is my code:

function CheckFormi() {
    var lh = document.getElementById('op').value;
    var ph = document.getElementById('oc').value;
    var lb = document.getElementById('dp').value;

    if (lh == "País" || ph == "Codiga postal" || lb == "País") {
        if (lf == "País") {
            document.getElementById('op').style.border = "solid 1px #FF0000";
        }
        if (ph == "Codiga postal") {
            document.getElementById('oc').style.border = "solid 1px #FF0000";
        }
        if (lb == "País") {
            document.getElementById('dp').style.border = "solid 1px #FF0000";
        }
        alert('error');
        return false;
    } else {
        document.getElementById('a').submit();
    }
}

This is how I call the function:

<form action="processor.php" method="post" id="a" onsubmit="return CheckFormi()">\

Any help will be appreciated,

thanks in advance!

I forgot to mention that the check works, it does show the alert but then still proceeds — Nick
– Nick, Commented Nov 2, 2013 at 10:30

dooxe · Accepted Answer · 2013-11-02 10:36:20Z

1

You have an error at this line :

if (lf == "País") {

variable lf may not exist (lh may be ?). That could explain why your alert is not displayed.

answered Nov 2, 2013 at 10:36

dooxe

1,49012 silver badges19 bronze badges

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

1 Comment

Nick Over a year ago

Hey dooxe, thank you, that solved! What a stupid mistake there. What threw me off was that the alert was actually displaying. Thanks a lot for your sharp eye!

MugosDynamic · Accepted Answer · 2013-11-02 10:36:56Z

1

Maybe you should use 'event.preventDefault();' instead of 'return false;'. in ie, you should use event.returnValue = false;.

answered Nov 2, 2013 at 10:36

MugosDynamic

991 silver badge7 bronze badges

Collectives™ on Stack Overflow

Javascript validation: Return false not working, form still submitting

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related