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 Quote
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. Quote
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. Quote
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; } ?> Quote
Mokothemonkey Posted September 10, 2009 Author Posted September 10, 2009 Works PERFECTLY thanks! Quote
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.