Mokothemonkey Posted September 9, 2009 Posted September 9, 2009 They aren't working, but it's exactly the same code as the ones that ARE working. Check it out here
Byron Posted September 9, 2009 Posted September 9, 2009 Does your if statement look like this on your new form? <?php if ( $_REQUEST[email] && $_REQUEST[message] ) { If so then you need to change it to match the new fields your using now, like: <?php if ( $_REQUEST[name] && $_REQUEST[rating] ) { If that's not the problem then you'll need to post your entire code.
Mokothemonkey Posted September 10, 2009 Author Posted September 10, 2009 <?php if ( $_REQUEST[name] && $_REQUEST[review] ) { $name = $_REQUEST['name']; $rating = $_REQUEST['rating']; $review = $_REQUEST['review']; $headers = "From: $name <$email>\n"; $headers .= "Content-Type: text/plain; charset=iso-8859-1\n"; mail( "admin@replayband.heliohost.org", "Replay Band Review", $message, $headers ); header( "Location: http://replayband.heliohost.org/feedback.php" ); exit; } ?> Now it sends, but there is no message.
Byron Posted September 10, 2009 Posted September 10, 2009 I recommend that you add a place on your html form for the person to type in thier email address cause you really need a From: address like you had on your first form. The php below is already set for it. Put this instead of what you have now. <?php if ( $_REQUEST[name] && $_REQUEST[review] ) { $name = $_REQUEST['name']; $rating = $_REQUEST['rating']; $review = $_REQUEST['review']; $email = $_REQUEST['email']; $headers = "From: $name <$email>\n"; $headers .= "Content-Type: text/plain; charset=iso-8859-1\n"; $message = "Rating: $rating\n"; $message .= "Review: $review\n"; mail( "admin@replayband.heliohost.org", "Replay Band Review", $message, $headers ); header( "Location: http://replayband.heliohost.org/feedback.php" ); exit; } ?>
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now