Why won't this form validate?

Question

The "onsubmit" statement is not called:

<form action="index.php" method="post" onsubmit="return validateSearchKeyword()">
     <input class="text_search" id="text_search" name="text_search" type="text" value="search" onfocus="if (this.value=='search') this.value = ''" onBlur="if (this.value=='') this.value = 'search'"  />
    </form>

This is my validation function:

function validateSearchKeyword()
{ alert ('sad');//for testing purposes - this alert is not showed
if (document.getElementById('text_search').value==""){creatediv('divWarnSearch','You must supply a value', '120px', '250px');return false;}
}

The function creatediv is working great with other forms, so the problem is not that one.

Any ideas? Thanks

Answer 1

You're missing a submit button, which may result in users not being able to submit the form in some browsers.

Furthermore, it would be helpful if you could post your validation function, so we can check if there problem is there, e.g. does it return a boolean?

What are you results if you use the following:

<form ... onsubmit="alert('Foo'); return true">

Answer 2

tried onsubmit="javascript: validateSearchKeyword();" ? AFAIK, it's already returning, no need to specify that.

You may also try removing the ().

And also, the submit button as Mr Potato Head says, if you didn't remove it to make the code terse.

EDIT: This was a badly thought out post. This is a good example of what not to do. If you find any of these suggestions implemented in your code, refactor!

Answer 3

Check to make sure that the function you are calling is actually there, then I'd go through that function to check for problems. Firebug might be of use to you, or you could just go with sticking the following in the top of the called function (to make sure it's getting that far):

alert('here');

Answer 4

Your problem is :

onBlur="if (this.value=='') this.value = 'search'"

When you submit, you lost the focus on the textbox and the event is launched: try to put

alert(document.getElementById('text_search').value);

and you will get "search" and not empty string...