Simple PHP contact form

[fathernugen]

Hi,

 

Today I have taken my first step towards familiarisation with php.

 

I have created a simple form form a tutorial I read.

 

The mail.php file is this:

 

<?php
$firstname = $_POST['firstname'];
$lastname = $_POST['lastname'];
$email = $_POST['email'];
$dropdown = $_POST['dropdown'];
$formcontent=" From: $firstname \n Surname: $lastname \n Email: $email \n Dropdown: $dropdown";
$recipient = "[email protected]";
$subject = "Newsletter Sign Up";
$mailheader = "From: $email \r\n";
mail($recipient, $subject, $formcontent, $mailheader) or die("Error!");
echo "You have been added to our newsletter subscription list!" . " -" . "<a href='newsletter.php' style='margin:0 auto; top: 100px; text-decoration:none;color:#ff0099;'> Return to Sitting Spiritually</a>";
?>

 

My problem is that users are redirected to a blank page with 'thank you return to sitting spiritually'.

 

I would like to use a custom thank you page with a url like;

 

www.sittingspiritually.co.uk/thankyou.php

 

so the look of the site is not completely ruined by my php inadequacies.

 

 

Is there a small piece of code i can add to the mail.php file so i can use a custom page for the success???

 

I hope some one can help, thanks in advance.

 

 

[snowman15]

Yes, you can use an if statement with the mail function, to forward users to a thank you page upon mail success. (Mail function returning true: )

 

if ( mail($recipient, $subject, $formcontent, $mailheader) ){
header('Location: http://www.mysite.com/thankyou.php');
} else {
die ("error");
}

[AbraCadaver]

Or just replace:

 

echo "You have been added to our newsletter subscription list!" . " -" . "<a href='newsletter.php' style='margin:0 auto; top: 100px; text-decoration:none;color:#ff0099;'> Return to Sitting Spiritually</a>";

With:

 

header('Location: http://www.sittingspiritually.co.uk/thankyou.php');
exit;

 

[snowman15]

The only problem with the above code is that it does not account for mail failure. Either one will work though. I would try to start your better php habits now though.

[fathernugen]

I have added the code Abracadaver posted and it works.

 

Sweet, thanks.

 

But how do I tackle the failure which snowman is talking about.

 

Also which piece of code is the best practice? Abracadavers or Snowmans???

 

I do have a simple jquery validation which stops the form being sent if the field are not filled.

 

Take a look at www.sittingspiritually.co.uk/newsletter.php

 

 

[BlueSkyIS]

robots ignore jquery and post directly to the script. client-side validation is not enough.

[fathernugen]

That, as an answer, is not enough, please expand...

 

Do you know what I can add to the php to make some server side validation?

[AbraCadaver]

The only problem with the above code is that it does not account for mail failure. Either one will work though. I would try to start your better php habits now though.

 

They already have:

 

mail($recipient, $subject, $formcontent, $mailheader) or die("Error!");

[snowman15]

^^ he is right I didn't see that. Enjoy!

[fathernugen]

In that case, you guys have done sterling work.

 

Thanks for your help.