Jump to content

Validation Server Side Not Sending Email

CBaZ

By CBaZ
in PHP Coding Help

 Share

Recommended Posts

CBaZ Member

CBaZ

Posted
Posted

It validates fine just doesn't send email and no errors.  Totally stomped.

 

<?php
//simple captcha
if(isset($_POST['answer']) && $_POST['answer']!='14') { echo "<script>alert('You provided a wrong answer for the Security Question')</script>"; 
echo "<a href='contact.php'>Please return to the Form and try again.</a><br /><br />\n";


 //simple captcha
$field_name = $_POST['name'];
$field_email = $_POST['email'];
$field_phone = $_POST['phone'];
$field_preferred = $_POST['preferred'];
$field_datepicker = $_POST['datepicker'];
$field_passengers = $_POST['passengers'];
$field_pickup = $_POST['pickup'];
$field_drop = $_POST['drop'];
$field_message = $_POST['message'];

if(!ereg("^[a-zA-Z0-9_]+@[a-zA-Z0-9\-]+\.[a-zA-Z0-9\-\.]+$", $field_email))
   {
    echo "That is not a valid <b>email</b> address. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
    exit;
   }

   if(empty($_POST['name']) || strlen(trim($_POST['name'])) ==0)
   {
    echo "Your <b>name</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else if(empty($_POST['email']) || strlen(trim($_POST['email'])) ==0)
   {
    echo "Your <b>email</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else if(empty($_POST['message']) || strlen(trim($_POST['message'])) ==0)
   {
    echo "You <b>message</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else if(empty($_POST['pickup']) || strlen(trim($_POST['pickup'])) ==0)
   {
    echo "Your <b>pickup</b> location was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else if(empty($_POST['drop']) || strlen(trim($_POST['drop'])) ==0)
   {
    echo "Your <b>drop off</b> location was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else
   {

$mail_to = '[email protected]';
$subject = 'New Message from '.$field_name;

$body_message = 'From: '.$field_name."\n";
$body_message .= 'E-mail: '.$field_email."\n";
$body_message .= 'Phone: '.$field_phone."\n";
$body_message .= 'Preferred Method: '.$field_preferred."\n";
$body_message .= 'Appointment Date: '.$field_datepicker."\n";
$body_message .= 'Number of Passengers: '.$field_passengers."\n";
$body_message .= 'Pick Up Location: '.$field_pickup."\n";
$body_message .= 'Drop Off Location: '.$field_drop."\n";
$body_message .= 'Message: '.$field_message."\n";

$headers = 'From: '.$field_email."\r\n";
$headers .= 'Reply-To: '.$field_email."\r\n";


$mail_status = mail($mail_to, $subject, $body_message, $headers);

if ($mail_status) { ?>
<script language="javascript" type="text/javascript">
alert('Thank you for the message. We will contact you shortly.');
window.location = 'index.htm';
</script>
<?php
}
else { ?>
<script language="javascript" type="text/javascript">
alert('Message failed. Please, send an email to techsuppor');
window.location = 'index.htm';
</script>
<?php
}
}
}
?>

 

 

cutielou22 Regular Member

cutielou22

Posted
Posted

How about you try not having the mail() command as a variable. Like below:

 

<?php
//simple captcha
if(isset($_POST['answer']) && $_POST['answer']!='14') { echo "<script>alert('You provided a wrong answer for the Security Question')</script>"; 
echo "<a href='contact.php'>Please return to the Form and try again.</a><br /><br />\n";


 //simple captcha
$field_name = $_POST['name'];
$field_email = $_POST['email'];
$field_phone = $_POST['phone'];
$field_preferred = $_POST['preferred'];
$field_datepicker = $_POST['datepicker'];
$field_passengers = $_POST['passengers'];
$field_pickup = $_POST['pickup'];
$field_drop = $_POST['drop'];
$field_message = $_POST['message'];

if(!ereg("^[a-zA-Z0-9_]+@[a-zA-Z0-9\-]+\.[a-zA-Z0-9\-\.]+$", $field_email))
   {
    echo "That is not a valid <b>email</b> address. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
    exit;
   }

   if(empty($_POST['name']) || strlen(trim($_POST['name'])) ==0)
   {
    echo "Your <b>name</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else if(empty($_POST['email']) || strlen(trim($_POST['email'])) ==0)
   {
    echo "Your <b>email</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else if(empty($_POST['message']) || strlen(trim($_POST['message'])) ==0)
   {
    echo "You <b>message</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else if(empty($_POST['pickup']) || strlen(trim($_POST['pickup'])) ==0)
   {
    echo "Your <b>pickup</b> location was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else if(empty($_POST['drop']) || strlen(trim($_POST['drop'])) ==0)
   {
    echo "Your <b>drop off</b> location was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
   }
   else
   {

$mail_to = '[email protected]';
$subject = 'New Message from '.$field_name;

$body_message = 'From: '.$field_name."\n";
$body_message .= 'E-mail: '.$field_email."\n";
$body_message .= 'Phone: '.$field_phone."\n";
$body_message .= 'Preferred Method: '.$field_preferred."\n";
$body_message .= 'Appointment Date: '.$field_datepicker."\n";
$body_message .= 'Number of Passengers: '.$field_passengers."\n";
$body_message .= 'Pick Up Location: '.$field_pickup."\n";
$body_message .= 'Drop Off Location: '.$field_drop."\n";
$body_message .= 'Message: '.$field_message."\n";

$headers = 'From: '.$field_email."\r\n";
$headers .= 'Reply-To: '.$field_email."\r\n";

if (mail($mail_to, $subject, $body_message, $headers)) { ?>
<script language="javascript" type="text/javascript">
alert('Thank you for the message. We will contact you shortly.');
window.location = 'index.htm';
</script>
<?php
}
else { ?>
<script language="javascript" type="text/javascript">
alert('Message failed. Please, send an email to techsuppor');
window.location = 'index.htm';
</script>
<?php
}
}
}
?>

 

Let me know if it works! :)

JLT Newbie

JLT

Posted
Posted (edited)

You have got it muddled up, the first if statement contains everything else. I tidied up your code, here is what it looks like.

 

<?php
//simple captcha
if (isset($_POST['answer']) && $_POST['answer'] != '14') {
    echo "<script>alert('You provided a wrong answer for the Security Question')</script>";
    echo "<a href='contact.php'>Please return to the Form and try again.</a><br /><br />\n";
    
    //simple captcha
    $field_name       = $_POST['name'];
    $field_email      = $_POST['email'];
    $field_phone      = $_POST['phone'];
    $field_preferred  = $_POST['preferred'];
    $field_datepicker = $_POST['datepicker'];
    $field_passengers = $_POST['passengers'];
    $field_pickup     = $_POST['pickup'];
    $field_drop       = $_POST['drop'];
    $field_message    = $_POST['message'];
    
    if (!ereg("^[a-zA-Z0-9_]+@[a-zA-Z0-9\-]+\.[a-zA-Z0-9\-\.]+$", $field_email)) {
        echo "That is not a valid <b>email</b> address. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
        exit;
    }
    
    if (empty($_POST['name']) || strlen(trim($_POST['name'])) == 0) {
        echo "Your <b>name</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
    } else if (empty($_POST['email']) || strlen(trim($_POST['email'])) == 0) {
        echo "Your <b>email</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
    } else if (empty($_POST['message']) || strlen(trim($_POST['message'])) == 0) {
        echo "You <b>message</b> was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
    } else if (empty($_POST['pickup']) || strlen(trim($_POST['pickup'])) == 0) {
        echo "Your <b>pickup</b> location was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
    } else if (empty($_POST['drop']) || strlen(trim($_POST['drop'])) == 0) {
        echo "Your <b>drop off</b> location was not entered into the field. Please return to the" . " <a href='contact.php'>previous page and try again.</a>";
    } else {
        
        $mail_to = '[email protected]';
        $subject = 'New Message from ' . $field_name;
        
        $body_message = 'From: ' . $field_name . "\n";
        $body_message .= 'E-mail: ' . $field_email . "\n";
        $body_message .= 'Phone: ' . $field_phone . "\n";
        $body_message .= 'Preferred Method: ' . $field_preferred . "\n";
        $body_message .= 'Appointment Date: ' . $field_datepicker . "\n";
        $body_message .= 'Number of Passengers: ' . $field_passengers . "\n";
        $body_message .= 'Pick Up Location: ' . $field_pickup . "\n";
        $body_message .= 'Drop Off Location: ' . $field_drop . "\n";
        $body_message .= 'Message: ' . $field_message . "\n";
        
        $headers = 'From: ' . $field_email . "\r\n";
        $headers .= 'Reply-To: ' . $field_email . "\r\n";
        
        
        $mail_status = mail($mail_to, $subject, $body_message, $headers);
        
        if ($mail_status) {
?>
<script language="javascript" type="text/javascript">
alert('Thank you for the message. We will contact you shortly.');
window.location = 'index.htm';
</script>
<?php
        } else {
?>
<script language="javascript" type="text/javascript">
alert('Message failed. Please, send an email to techsuppor');
window.location = 'index.htm';
</script>
<?php
        }
    }
}
?>

 

If your captcha answer is incorrect, the rest of the form will validate. Your logic there is a little bit off. Now that I've given you something to think about, you should be able to fix it.
 
Also, ereg functions are deprecated. It is advised to use preg_match and other similar functions.
 
If you are still stuck, perhaps try placing the following block of code after your opening <?php at the top of the page just incase.
 
error_reporting(E_ALL);
ini_set("display_errors", "On");
 
Otherwise, paste your updated code and we can have another look.
Edited by JLT
This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
 Share
Go to topic listing
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.