PHP Redirect Issue

Question

I'm having a difficult time getting my page to redirect after form submission. I've followed the advice that I've been able to find so far and no luck. Any help would be greatly appreciated.

<?php
//If the form is submitted
if(isset($_POST['submit'])) {

$subject = "CONTACT FORM";
//Check to make sure that the name field is not empty
if(trim($_POST['contactname']) == '') {
    $hasError = true;
} else {
    $name = trim($_POST['contactname']);
}

//Check to make sure sure that a valid email address is submitted
if(trim($_POST['email']) == '')  {
    $hasError = true;
} else {
    $email = trim($_POST['email']);
}

//If there is no error, send the email
if(!isset($hasError)) {
    $emailTo = 'email@domain.com'; 
    $body = "Name: $name \nEmail: $email";
    $headers = 'From: Bond Limo (Newsletter Signup) <'.$emailTo.'>' . "\r\n" . 'Reply-To: ' . $email;

    mail($emailTo, $subject, $body, $headers);
    $emailSent = true;

    }
}
?>

  <?php if(isset($hasError)) { //If errors are found ?>
  <p class="error">Please check if you've filled all the fields with valid information. Thank you.</p>
  <?php } ?>
  <?php if(isset($emailSent) && $emailSent == true) { 

    header("Location: http://www.website.com");

   } 
   ?>

  <form method="post" action="<?php echo $_SERVER['PHP_SELF']; ?>" class="side-form">

Maybe you can find some help from here: http://stackoverflow.com/questions/768431/how-to-make-a-redirect-in-php ? — Suprsonic, Oct 05 '14 at 17:40
possible duplicate of [How can I redirect in PHP without header errors?](http://stackoverflow.com/questions/1255518/how-can-i-redirect-in-php-without-header-errors) — toesslab, Oct 05 '14 at 17:41
-1 Don't do any output before the header. It has been asked sooo many times, please do better research! — toesslab, Oct 05 '14 at 17:41

Baraa · Answer 1 · 2014-10-05T18:36:17.320

1

Your header is not being processed. In order for your header to be processed and the redirect to occur, you will have to call the die() function after your header. Like so:

<?php
    if(isset($emailSent) && $emailSent) {
        header("Location: http://www.website.com");
        die();
    }
?>

Additionally, your code can be optimized by not checking:<?php if(isset($hasError)) { //If errors are found ?> again. Rather, just connect it with the above if statement and use an else statement like so:

// If there is no error, send the email
if (!isset($hasError)) {
    $emailTo = 'email@domain.com'; 
    $body = "Name: $name \nEmail: $email";
    $headers = 'From: Bond Limo (Newsletter Signup) <'.$emailTo.'>' . "\r\n" . 'Reply-To: ' . $email;

    $emailSent = mail($emailTo, $subject, $body, $headers);
} else {
    // Errors found
    <p class="error">Please check if you've filled all the fields with valid information. Thank you.</p>
}

edited Oct 05 '14 at 18:36

answered Oct 05 '14 at 17:45

Baraa

1,341
11
8

1

Why not `$emailSent = mail($emailTo, $subject, $body, $headers);` – Daniel W. Oct 05 '14 at 18:03
Thanks Baraa H. I did that but the redirect didn't work still. The rest of the page just stopped showing up. Any suggestions? – Fishin' Musician Oct 21 '14 at 13:22
Hey @Fishin'Musician, can you post your updated code in the question and then I'll take another look for you. – Baraa Oct 23 '14 at 21:07

PHP Redirect Issue

1 Answers1