I have a JavaScript validation function that worked before I changed the sendmail file to a php version (had to becfause of host's linux server). The sendmail code in the php file has some validation checking, but I would prefer to use the validation.js file with the form's onSubmit function. I've tried all kinds of changes. Figure it is something simple I'm missing. On submit, the php runs validation, but the JS seems to be ignored. Any ideas? The site is under development: www.insiderspetra.com/New/contact.asp

<form class="form" name="contactform" method="post" action="send_form_email.php" onsubmit="return validate(this)" >
    ..usual form fields..
<input type="submit" value="Submit" class="submit" style="margin-left:112px;">   `

Have tried with and without semi-colon after validate(this);

PHP file:

if(isset($_POST['email'])) {

    $email_to = "webmaster@insiderspetra.com";
    $email_subject = "IP Submission Form";

    function died($error) {
        // your error code can go here
        echo "We are very sorry, but there were error(s) found with the form you submitted. ";
        echo "These errors appear below.<br /><br />";
        echo $error."<br /><br />";
        echo "Please go back and fix these errors.<br /><br />";

    // validation expected data exists
    if(!isset($_POST['first_name']) ||
        !isset($_POST['last_name']) ||
        !isset($_POST['email']) ||
        !isset($_POST['telephone']) ||
        !isset($_POST['comments'])) {
        died('We are sorry, but there appears to be a problem with the form you submitted.');      

    $first_name = $_POST['first_name']; // required
    $last_name = $_POST['last_name']; // required
    $email_from = $_POST['email']; // required
    $telephone = $_POST['telephone']; // not required
    $comments = $_POST['comments']; // required

    $error_message = "";
    $email_exp = '/^[A-Za-z0-9._%-]+@[A-Za-z0-9.-]+\.[A-Za-z]{2,4}$/';
  if(!preg_match($email_exp,$email_from)) {
    $error_message .= 'The Email Address you entered does not appear to be valid.<br />';
    $string_exp = "/^[A-Za-z .'-]+$/";
  if(!preg_match($string_exp,$first_name)) {
    $error_message .= 'The First Name you entered does not appear to be valid.<br />';
  if(!preg_match($string_exp,$last_name)) {
    $error_message .= 'The Last Name you entered does not appear to be valid.<br />';
  if(strlen($comments) < 2) {
    $error_message .= 'The Comments you entered do not appear to be valid.<br />';
  if(strlen($error_message) > 0) {
    $email_message = "Form details below.\n\n";

    function clean_string($string) {
      $bad = array("content-type","bcc:","to:","cc:","href");
      return str_replace($bad,"",$string);

    $email_message .= "First Name: ".clean_string($first_name)."\n";
    $email_message .= "Last Name: ".clean_string($last_name)."\n";
    $email_message .= "Email: ".clean_string($email_from)."\n";
    $email_message .= "Telephone: ".clean_string($telephone)."\n";
    $email_message .= "Comments: ".clean_string($comments)."\n";

// create email headers
$headers = 'From: '.$email_from."\r\n".
'Reply-To: '.$email_from."\r\n" .
'X-Mailer: PHP/' . phpversion();
@mail($email_to, $email_subject, $email_message, $headers); 

<!-- include your own success html here -->

Thank you for contacting us. We will be in touch with you very soon.


Any idea? I haven't included the JS code as I know it is working when I do not use the php file. Thank you!

Recommended Answers

All 3 Replies

Thank you. That's not the problem tho. Same issue. Appears the JS is completely ignored.

onsubmit="return validate(this)"
add a semicolon;
onsubmit="return validate(this);"
also, make sure your validate function is returning true if validated, false if not.

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, learning, and sharing knowledge.