Feedback form validation not working

Question

Hi all,
My feedback form i am put form validation its not working, please help me,
my entire code is given below
<?php
          /* ========================= CODE WILL EXECUTE WHEN FORM WILL SUBMIT =====================*/
                    if($_GET["formFilledStatus"]=="t")
                              $name = $_GET['txtName'];
                              $email = $_GET['txtEmail'];
                              $phone          =          $_GET['txtPhone'];
                              $dateofappoitment          =          $_GET['txtDOA'];
                              $timeofappoitment          =          $_GET['txtTA'];
                              $userRequirement = $_GET['txtSuggestion'];
                              $to= "[email protected]";
                              $subject = "FEEDBACK & ENQUIRY FORM";
                              $message = "<BR><B> User Name:- </B>".$name."<BR><BR><B> Email:- </B>".$email."<BR><BR><B> Phone:- </B>".$phone."<BR><BR><B>Date of Appoitment:-</B><BR>".$dateofappoitment."<BR><BR><B> Time of Appoitment:- </B>".$timeofappoitment."<BR><BR>                                <B> Address:- </B>".$userRequirement;
                              $headers  = "MIME-Version: 1.0\r\n";
                              $headers .= "Content-type: text/html; charset=iso-8859-1\r\n";
                              $headers .= "From:".$email;
                              // Additional headers
                              //$headers .= 'Bcc: [email protected]' . "\r\n";
                              $mailStatus=mail($to, $subject, $message, $headers);
                    /* ========================= END CODE =====================*/
?>
<script language="javascript">
function isEmail(email)
                    if(email.indexOf("@") != "-1" && email.indexOf(".") != "-1")
                    return true;
                    else
                    return false;
          function validate()
                    if(document.enquiryForm.txtName.value=="")
                              alert("Name required!");
                              document.enquiryForm.txtName.focus();
                              return false;
                    if(document.enquiryForm.txtPhone.value=="")
                              alert("Please Enter your Phone!");
                              document.enquiryForm.txtPhone.focus();
                              return false;
                    if(document.enquiryForm.txtDOA.value=="")
                              alert("Please Select your date of appoitment!");
                              document.enquiryForm.txtDOA.focus();
                              return false;
                    if(document.enquiryForm.txtTA.value=="")
                              alert("Please Enter time of appoitment!");
                              document.enquiryForm.txtTA.focus();
                              return false;
                    if(document.enquiryForm.txtSuggestion.value=="")
                              alert("Please Enter your address!");
                              document.enquiryForm.txtSuggestion.focus();
                              return false;
</SCRIPT>
<style type="text/css">
<!--
.home_page_font {
          font-family: Verdana, Arial, Helvetica, sans-serif;
          font-size: 11px;
          font-style: normal;
          line-height: normal;
          font-weight: bold;
          font-variant: normal;
          text-transform: none;
          color: #333333;
          text-align: justify;
.home_page_font1 {          font-family: Verdana, Arial, Helvetica, sans-serif;
          font-size: 11.6px;
          font-style: normal;
          line-height: normal;
          font-weight: normal;
          font-variant: normal;
          text-transform: none;
          color: #333333;
          text-align: justify;
.home_page_font1 {          font-family: Verdana, Arial, Helvetica, sans-serif;
          font-size: 11px;
          font-style: normal;
          line-height: normal;
          font-weight: normal;
          font-variant: normal;
          text-transform: none;
          color: #868d98;
          text-align: justify;
.style6 {color: #000000}
-->
</style>
        <FORM ACTION="" METHOD="GET" NAME="enquiryForm1" ONSUBMIT=" return validate()">
                    <INPUT TYPE="HIDDEN" NAME="formFilledStatus" VALUE="t">
               <DIV ALIGN="LEFT">
        <TABLE WIDTH="100%" BORDER="0" ALIGN="CENTER" CELLPADDING="3" CELLSPACING="0" BORDERCOLOR="#F1D6D6" class="home_page_font1" STYLE="border-collapse:collapse;">
        <?php
                    /* ============ CODE IF MAIL SUCCESSFULLY SENT ====================== */
                    if($mailStatus)
                    ?>
              <TR>
            <TD COLSPAN="2" HEIGHT="25">
            <DIV class="c-text1" ALIGN="CENTER"><b>
            <h3 class="style6">Your Enquiry Has Been Successfully Sent.<b></h3>
            </DIV></TD>
          </TR>
                      <?
                      /*================== CODE END ====================================== */
?>
                      <TR>
            <TD COLSPAN="2" HEIGHT="25">
            <DIV class="home_page_font" ALIGN="CENTER">
              <div align="left">Suggestion &amp; Feedback Form</div>
            </DIV></TD>
          </TR>
          <TR>
            <TD class="power-com1"><FONT COLOR="#FF0000">* </FONT> <STRONG>Your
            Name :</STRONG></TD>
            <TD><INPUT NAME="txtName" SIZE="35" MAXLENGTH="48" VALUE="" class="txtfld"></TD>
          </TR>
          <TR>
            <TD class="power-com1"> </TD>
            <TD> </TD>
          </TR>
          <TR>
            <TD class="power-com1"><STRONG>Your
            E-Mail : </STRONG></TD>
            <TD><INPUT NAME="txtEmail" MAXLENGTH="48" SIZE="35" VALUE="" class="txtfld"></TD>
          </TR>
          <TR>
            <TD class="power-com1"> </TD>
            <TD> </TD>
          </TR>
          <TR>
            <TD class="power-com1"><FONT COLOR="#FF0000">* </FONT><STRONG>Phone/Mobile
            :</STRONG><br>(Include Country/Area Code)</TD>
            <TD><INPUT TYPE="TEXT" NAME="txtPhone" SIZE="22" MAXLENGTH="60" VALUE="" class="txtfld"></TD>
          </TR>
          <TR>
            <TD class="power-com1"> </TD>
            <TD> </TD>
          </TR>
          <TR>
            <TD class="power-com1"><STRONG><font color="#FF0000">*</font>Date of Appoitment :</STRONG><br>(Include Country/
            Area Code)</TD>
            <TD><INPUT TYPE="TEXT" NAME="txtDOA" SIZE="22" MAXLENGTH="60" VALUE="" class="txtfld" id="txtDOA"></TD>
          </TR>
          <TR>
            <TD class="power-com1"> </TD>
            <TD> </TD>
          </TR>
          <TR>
            <TD class="power-com1"><STRONG><font color="#FF0000">*</font>Time of Appoitment : </STRONG></TD>
            <TD><INPUT NAME="txtTA" SIZE="35" VALUE="" class="txtfld" id="txtTA"></TD>
          </TR>
          <TR>
            <TD class="power-com1"> </TD>
            <TD> </TD>
          </TR>
          <TR>
            <TD VALIGN="TOP" class="power-com1"> </TD>
            <TD> </TD>
          </TR>
          <TR>
            <TD VALIGN="TOP" class="power-com1"><FONT COLOR="#FF0000">* </FONT> <STRONG>Address</STRONG></TD>
            <TD><TEXTAREA NAME="txtSuggestion" ROWS="3" COLS="29" WRAP="Virtual" class="txtfld"></TEXTAREA></TD>
          </TR>
          <TR>
            <TD COLSPAN="2">
            <CENTER><INPUT TYPE="SUBMIT" NAME="" VALUE="Submit Query">
            <INPUT TYPE="RESET" NAME="Reset1"></CENTER></TD>
          </TR>
        </TABLE>
          </DIV></FORM>

Answer 1

Rather than reinventing the wheel, I tend to use SpryValidation widgets to validate the form using JavaScript. However, to ensure that no validation is untouched, you also need to validate the inputs using serverside script.
The example that you have shown is valiadtion in JavaScript. This means that anyone with JS turned off will bypass your validation.
Gramps