<?PHP
$message = stripcslashes($message);
$message = "
Message: $message \n
From: $name \n
Phone: $phone \n
";
$from = "From: $email \r\n";

mail("my@adress.com", "New message from website", $message, $from);
header('Location: contact.html');
exit;
?>

Hi all,
This exact code used to work just fine untill a week ago.
But now for some reason all I'm recieving are BLANK email messages. Can anyone suggest what went wrong and why?
THNX...

Most likely, your host changed some settings in the configuration, such as version of php, or something like that. Look around and see if there are any changes that have been made that would affect something like this.

- Steve

I think this script relies on register_globals on, which it probably isn't because it's safer to have it off.
What that means is that you have to look in what way the message is coming to the script e.g. POST or GET and then change the $message, $name, $phone and $email to $_POST['message'], $_POST['name'] etc. or $_GET['message'], $_GET['name'] etc.

I am no expert at PHP, but I have a few things to add.
1) you don't need to put "exit;" (I've never used it at least- might be wrong).
2) You need to specify what the variables mean, and were in the form the information is coming from. So for example...
$message = $_REQUEST['message'];
where you retrieve information from the form.
3) Perhaps try something like this
$message = stripslashes($message1);
so you don't overuse the $message variable. I understand you don't have to, but it makes the script "easier"; if that makes sense (in my humble opinion).
4) If this still doesn't work, try contact your host manager, especially if this script WAS working.

Most likely they removed some sort of software that allows you to not have to declare the variables, and specify where to retrieve the information from in the form you submitted... though I don't know how that'd work.

Originally Posted by Insensus

I think this script relies on register_globals on, which it probably isn't because it's safer to have it off.
What that means is that you have to look in what way the message is coming to the script e.g. POST or GET and then change the $message, $name, $phone and $email to $_POST['message'], $_POST['name'] etc. or $_GET['message'], $_GET['name'] etc.

Agreed. Here is a quick idea for some code that should work: (assuming the menu is set to POST)

PHP Code:

<?php
$message = $_POST['message'];
$name = $_POST['name'];
$phone = $_POST['phone'];
$email = $_POST['email'];
 $body = "
Message: $message \n 
From: $name \n
Phone: $phone \n
";
 $from = "From: $email \r\n";

  mail("my@adress.com", "New message from website", $body, $from);
header('Location: contact.html');
?>

- Steve

stevej Thank you!
Zoltar1992 Thank you!
Insensus Thank you!

It's all better now

Glad you got it working but you may still want to change: stripcslashes( )

to:stripslashes ()