If MM_validateForm returns true then the form should carry out the action, else kick out on false, as I read this. Perhaps the validation should occur "onsubmit" rather than "onclick" whereby true will effect an action, and false will revert to an error message?