PHP Email Form

Question

Could anyone tell me if there is something wrong with the code or a better way to modify it? for some reason when i hit submit on my contact form it displays the "error.html" page but i still get an email sent to my account.

$EmailTo = "[email protected]";
$Subject = "Contact Submission";

$Name = Trim(stripslashes($_POST['name'])); 
$Email = Trim(stripslashes($_POST['email'])); 
$Budget = Trim(stripslashes($_POST['budget'])); 
$Message = Trim(stripslashes($_POST['message']));

// prepare email body text
$Body = 'Contact Submission'."\n";
$Body .= 'Name:        '   .$Name."\n";
$Body .= 'Email:       '   .$Email."\n";
$Body .= 'Budget:      '   .$Budget."\n";
$Body .= 'Message:     '   .$Message."\n";


// send email 
$success_email = mail($EmailTo, $Subject, $Body, "From: <$Email>");

// redirect to success page
// CHANGE THE URL BELOW TO YOUR "THANK YOU" PAGE
    if ($success){
     header ('location:thankyou.html');
    } 
    else{
    header ('location:error.html');
    }

it's good form to put an exit; under your location headers - to ensure the page won't be parsed in an unlikely event — alex
– alex, Commented Aug 14, 2009 at 3:03

Jefe · Accepted Answer · 2009-08-14 03:02:15Z

8

Change

if ($success){

to

if ($success_email){

$success doesn't exist...

answered Aug 14, 2009 at 3:02

Jefe

5781 gold badge7 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Sri · Accepted Answer · 2009-08-14 03:14:41Z

 $EmailTo = "[email protected]";
$Subject = "Contact Submission";

$Name = Trim(stripslashes($_POST['name'])); 
$Email = Trim(stripslashes($_POST['email'])); 
$Budget = Trim(stripslashes($_POST['budget'])); 
$Message = Trim(stripslashes($_POST['message']));

// prepare email body text
$Body = 'Contact Submission'."\n";
$Body .= 'Name:            '   .$Name."\n";
$Body .= 'Email:           '   .$Email."\n";
$Body .= 'Budget:          '   .$Budget."\n";
$Body .= 'Message:     '   .$Message."\n";


// send email 
$success_email = mail($EmailTo, $Subject, $Body, "From: <$Email>");

// redirect to success page
// CHANGE THE URL BELOW TO YOUR "THANK YOU" PAGE
    if ($success_email){
     header ('location:thankyou.html');
    } 
    else{
    header ('location:error.html');
    }

Littm · Accepted Answer · 2012-10-11 01:33:40Z

0

//Checking for empty for redirecting error page
if(empty($Name) || empty($Email)||empty($Budget)) 
{
    header('Location:error.html');
    exit;
}

//Send the email! if the fileds are not empty
mail($EmailTo, $Subject,$Body,"From: <$Email>");

//One the email is sent, Redirect to thankyou page.
header('Location: thankyou.html');

edited Oct 11, 2012 at 1:33

Littm

4,9454 gold badges33 silver badges40 bronze badges

answered Jul 14, 2012 at 6:51

web

11 silver badge4 bronze badges

Collectives™ on Stack Overflow

PHP Email Form

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related