mail form with headers problem

[danielandlisa]

hi i got this mail form that works but i got some problems with i get the mail sent to my email but in the mail it dont show the email submitted by the user from the form but my own mail.

can somebody tell me how the headers should bset in the code to
send the email to my mailbox showing the email set by the user
i also wonder if its possible to add a functin for sending acopy of the mail
to the user
/ thanks Lisa
the code
<?php
if(!isset($_POST['submit']))
{
//This page should not be accessed directly. Need to submit the form.
echo "error; you need to submit the form!";
}
$name = $_POST['name'];
$visitor_email = $_POST['email'];
$message = $_POST['message'];

//Validate first
if(empty($name)||empty($visitor_email))
{
echo "Name and email are mandatory!";
exit;
}

if(IsInjected($visitor_email))
{
echo "Bad email value!";
exit;
}

$email_from = 'yourname@yoursite.com';//<== update the email address
$email_subject = "New Form submission";
$email_body = "You have received a new message from the user $name.\n".
"Here is the message:\n $message".

$to = "your@email.com";//<== update the email address
$headers = "From: $email_from \r\n";
$headers .= "Reply-To: $visitor_email \r\n";
//Send the email!
mail($to,$email_subject,$email_body,$headers);
//done. redirect to thank-you page.
header('Location: thank-you.html');


// Function to validate against any email injection attempts
function IsInjected($str)
{
$injections = array('(\n+)',
'(\r+)',
'(\t+)',
'(%0A+)',
'(%0D+)',
'(%08+)',
'(%09+)'
);
$inject = join('|', $injections);
$inject = "/$inject/i";
if(preg_match($inject,$str))
{
return true;
}
else
{
return false;
}
}

?>

[masterofollies]

Change this

PHP Code:

$email_from = 'yourname@yoursite.com';//<== update the email address 


to this

PHP Code:

$email_from = "$otheremail";//<== update the email address 


Then in your form that you type in the information (which isn't listed in that code) change it to this

PHP Code:

<input type="text" name="otheremail"> 


[danielandlisa]

thanks alot ollie, do you know how to send a copy to the user to?

[masterofollies]

You're welcome, I don't but you can try this, it seems like it might work.

PHP Code:

$to = "your@email.com";//<== update the email address
$to = "their@email.com";//<== update the email address 


Also option 2.

You could do in 2 send mail forms, the first to send to you, and the second to send to them with a different message. You can use the same variables.

[danielandlisa]

i tried changing email from to $other email but it dont work, when recieving the email it shows from $other in the from field in the email client.

[masterofollies]

Sorry I should have been a little clearer. You need to change the form. The textbox that they enter their name into. That name needs to be "otheremail" then when the form is submitted you need to extract into a variable.

Like this

PHP Code:

$otheremail = $_POST['otheremail']; 


[danielandlisa]

i changed it in both the form and the php page but dont work

[masterofollies]

Post the whole script so I can see whats causing it the problem.

[happeemom]

try doing this:

take out this:

$email_from = 'yourname@yoursite.com';//<== update the email address

and put your header

$headers = "From: $email_from \r\n";

to this:

$headers = "From: ".$_POST["email"]."\r\n"

I'm still learning, forgive me if it doesnt work :-)

[DJCMBear]

Everything your asking for is on here:- http://php.net/manual/en/function.mail.php

and to send to more than one person just do this:

PHP Code:

$to  = 'email1@email.com';
$to .= 'email2@email.com';
$to .= 'email3@email.com'; 


[tomws]

I realize this post was abandoned a few days ago, but this should be addressed:

Originally Posted by DJCMBear

and to send to more than one person just do this:

PHP Code:

$to  = 'email1@email.com';
$to .= 'email2@email.com';
$to .= 'email3@email.com'; 


No. All you've done there is create a long and bad email address. They would at least need to be comma separated.

PHP Code:

$to  = 'email1@example.com';
$to .= ',email2@example.com';
$to .= ',email3@example.com'; 


Using additional headers would be another solution.

[masterofollies]

Everything given will work, just need to see the whole script so we can see where the placement error is.