Mail from PHP Script

cswales

I have a problem sending an email from within my php script. It just does not send the email. Can anyone help?

The script is:

<?php
$to = "me@you.com";
$from = "me@you.com";
$title = "Quotation Request";
$body = <<< emailbody
Dear Customer,

Thankyou for requesting a quotation from us.

We aim to respond within the same business day.

Thank you,

emailbody;
$success = mail($to,$from,$title,$body,
"From:$from\r\nReply-To:me@you.com");
?>

lozza1978

do you know if your mail is actually getting sent to the spool ? and its your mail server giving me grief ? or its this actuall script ?

if i were you i would run

if (!mail()){
echo 'error mail not sent';
}

this script will check that the mail actually got sent to the spool,

hope this helps

cswales

The error message not sent is showing.

If I send my contact form this works.

LoganK

Did you look at the manual for [man]mail[/man]? mail() takes for arguments, in this order: to, subject, message[, headers]. You're giving it 5 arguments. Get rid of the "$from" argument. The "from" is specified in the mail headers, which you would need to create a new variable for.

lozza1978

hi there

change

$success = mail($to,$from,$title,$body,
"From:$from\r\nReply-To:me@you.com");

mail($to,$from,$title,$body);

if (!mail($to,$from,$title,$body)) {
echo'ERROR MAIL NOT SET TO SPOOL';
}

should work

let us know how you get on,

cheers

LoganK

Lozza, did you even read what I wrote? I said to get rid of the "$from" argument - your change will not make any difference because the parameters are still wrong!!! The mail() function accepts the following parameters, in this order: to, subject, message[, headers]. NOT: to, from, subject, message, headers (which duplicate from), or your suggestion: to, from, subject, body.

Here's the correct code, along with proper error checking:


// $name and $email are gotten from POST, or DB, or GET, or COOKIE, or SESSION, etc.
$headers = "From: $name<$email>\r\n";
@mail("$to", "$subject", "$message", "$headers") or die("Error msg goes here!");

Edit: Sorry if this is a bit harsh but I want cswales to understand the right way to call the function before he learns how to implement error checking.