Can someone check my cobbled-together script?

[weaselandalf]

Hi, I've got an email script that works (which I am disgustingly proud about), which some kind people on here helped me with, and I fixed a bit too. However, there's still an excellent reason I'm posting in the Newbies section.... It all works except one or two bits.

What I want to happen is that the user puts in their email address: <input type="text" name="email" value="<? echo $_POST[email]; ?>"> which then replaces my own with theirs using the headers function..... I think this is possible, I remember a tutorial that said it was.... However, I think I've done it wrong somehow, I tried to set the headers to use the email variable that I tried to set with the text input field (above in red), but I'm pretty sure that's not how you do it. Can anyone help???

Just to confirm:

The user puts in a message ->
Then their email address ->
My script replaces the email address my server uses for outgoing mail using headers with the address the user input so I can reply to them (or it appears that way, I don't actually want to use a different one!)
-> I am inordinately happy and get too enthusiastic and break the script again.

<?

if (isset($_POST['Send'])) {

$to = '[e-mail]';
ini_set("sendmail_from", " [e-mail] ");
$subject = 'A new client';
$body= $_POST['message'];


mail($to,$subject,$body,$from);
echo "Thanks for your email";

} else {

$headers .= 'From: Client $email'.$eol;
$headers .= 'Reply-To: Client $email'.$eol;
$headers .= 'Return-Path: Client $email'.$eol;

?>

<form action="<? $PHP_SELF; ?>" method="post">
<textarea name="message" value="<? echo $POST['message']; ?>"></textarea>
<input type="text" name="email" value="<? echo $POST[email]; ?>">
<input type="submit" name="Send" value="Send">
</form>

<? } ?>

Can anyone either help or find me a psychologist??

[obsidian]

basically, you've got the idea right, but there are a couple things you need to clean up about your header declaration... try something like this:

<?

if (isset($_POST['Send'])) {

$to = '[e-mail]';
ini_set("sendmail_from", " [e-mail] ");
$subject = 'A new client';
$body= $_POST['message'];
$from = $_POST['email'];

$headers = "From: $from\r\n Reply-to: $from\r\n ";

mail($to, $subject, $body, $from);
echo "Thanks for your email";

}
?> 

one other note of interest, in your message field, you need to put the "echo $POST['message']" between the textarea tags instead of as a value attribute. the textarea doesn't support the value attribute. also, you may want to make sure that the form has been submitted before you try to echo the $POST values out like that. something like this should work nicely:

<textarea name="message"><?= isset($_POST['Send']) ? $_POST['message'] : ''; ?></textarea>

hope this helps

[weaselandalf]

That's absolutely fantastic, thank you so much! it works, now I can customise it and find new and exciting ways to break it!!!