Form Validation not Working - help please?

[AuctionHugh]

I basically use the same form which I modify from site to site as needed - why reinvent the wheel?

This one has a lot of fields, but I am only validating three of them - name cannot be blank, email has to be an email address, and the spam catcher has to be 777.

(Note that in my form2email.php file I do more checking for valid email, same referrer, 777, etc, which really cuts down on spam - probably many discussions on this elsewhere.)

Anyway, for some reason in this incantation of the form it is allowing the form to be submitted with both the name and/or email fields blank.

It is probably something pretty obvious, but I'm missing it. I'd appreciate it if one of you experts (and I don't mean that sarcastically) could glance this over and tell me what is going on. Thanks!

edit - my form seems to be too long to submit so I'm cutting out the parts I'm not trying to validate:

                  <h1 class="headings">Music Selection Form</h1>
                 <form name="MusicSelection" method="post" action="FormToEmail.php">
                  <table width="100%" border="0" cellpadding="8" cellspacing="0" class="bodytext">
                    <tr>
                      <td width="48%"><strong>Your Contact Information</strong></td>
                      <td width="52%">&nbsp;</td>
                    </tr>
                    <tr>
                      <td><div align="right">Your Name:</div></td>
                      <td width="52%"><input name="Name" type="text" class="formstyle" id="Name" size="24" maxlength="75" /></td>
                    </tr>
                    <tr>
                      <td><div align="right">Your Email:</div></td>
                      <td width="52%"><input name="email" type="text" class="formstyle" id="email" size="28" maxlength="100" /></td>
                    </tr>
                    <tr>
                      <td colspan="2"><div align="center"><em>Please note: We will not share or release<br />
                        your personal information with any other source.</em></div></td>
                      </tr>
......
                    <tr>
                      <td><div align="right"><span class="bluebold">Enter the number 777 to help us avoid spam:</span></div></td>
                      <td><input name="AvoidSpam" type="text" class="formstyle" id="AvoidSpam" tabindex="10" size="50" maxlength="3" /></td>
                    </tr>
                    <tr>
                      <td>&nbsp;</td>
                    

<td><input name="SendRequest" type="submit" class="formstyle" id="SendRequest" onClick="MM_validateForm('Name','','R','email','','RisEmail','AvoidSpam','','RinRange777:777');return document.MM_returnValue" value="Send Request">        
                      </td>
                    </tr>
                  </table>
                  </form>

[Dubbya]

Without seeing your Javascript validation routine, the onClick calling "MM_validateForm" looks like the culprit.

Try this:

<form name="MusicSelection" method="post" action="FormToEmail.php" onsubmit="return false;">

Your Javascript validation routine should submit the form after it validates and returns "true".

In the event that this doesn't work, you'd do well to post your MM_validateForm Javascript as we've no way of telling where you might have made an error.

[weegillis]

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?

[AuctionHugh]

Originally Posted by Dubbya

Without seeing your Javascript validation routine, the onClick calling "MM_validateForm" looks like the culprit.

You hit the nail right on the head. My Javascript had a problem - a little fixin' and all is well. Thanks much for your help!

[sharonjackson]

If you are using Dreamweaver, I think it is a bug. I have never gotten MM_ValidateForm to work properly if I specify that the email must be an email address. It works if you tell it that it can be anything, however.

[AuctionHugh]

Once I fixed my obvious mistake (I'm sparing myself embarrassment by not posting exactly what it was here), it works fine. Email addresses work, but blank or text that does not contain a @ receives an error.