else

[ecabrera]

how come the the else donest not work it shows up "Please enter your feedback or insert your email"

 

<form method="post" action="feedback.php">
Email: <input name="email" type="text"><br> 
Feedback:<br><textarea name="message" rows="15" cols="40">
</textarea>
<br> <input type="submit" name="sendbtn" value="Send" />
</form>

<?php

$to = "ecabrera9@ymail.com"; 
$subject = "Feedback";
$email = $_POST['email']; 
$message = $_POST['message'];
$headers = "From: $email"; 

if($_POST['sendbtn']){

if($email && $message)
$sent = mail($to, $subject, $message, $headers);
echo "Thank you for the feedback. We will be in touch with you very soon.";
}else{
echo "Please enter your feedback or insert your email";
}

?>

[Pikachu2000]

What? Maybe you could explain that a little bit better?

[jason310771]

try the following to see what was actually entered in the fields.

<form method="post" action="feedback.php">
Email: <input name="email" type="text"><br> 
Feedback:<br><textarea name="message" rows="15" cols="40">
</textarea>
<br> <input type="submit" name="sendbtn" value="Send" />
</form>

<?php

$to = "ecabrera9@ymail.com"; 
$subject = "Feedback";
$email = $_POST['email'];
$message = $_POST['message'];



echo("email = '".$email."'<br />");
echo("message = '".$message."'<br />");




$headers = "From: $email"; 

if($_POST['sendbtn']){

if($email && $message)
$sent = mail($to, $subject, $message, $headers);
echo "Thank you for the feedback. We will be in touch with you very soon.";
}else{
echo "Please enter your feedback or insert your email";
}

?>

 

if any the content of the fields will be echod between the two sets of single quotes.

[QuickOldCar]

try this

 

<form method="post" action="feedback.php">
Email: <input name="email" type="text"><br> 
Feedback:<br><textarea name="message" rows="15" cols="40">
</textarea>
<br> <input type="submit" name="sendbtn" value="Send" />
</form>

<?php

$to = "ecabrera9@ymail.com"; 
$subject = "Feedback";
$email = $_POST['email']; 
$message = $_POST['message'];
$headers = "From: $email"; 

if($_POST['sendbtn']){

if($email && $message){
$sent = mail($to, $subject, $message, $headers);
echo "Thank you for the feedback. We will be in touch with you very soon.";
}else{
echo "Please enter your feedback or insert your email";
}
}

[ecabrera]

QuickOldCar works thanks but why does it have to be put in a variable the $sent

[Drummin]

To answer original question, look at your IF statement and how it's missing the opening bracket that corresponds to the ELSE as QuickOldCar pointed out.

[spiderwell]

the mail function returns true or false depending on success or failure, you can capture that with the $sent = bit, you dont Have to do it.

 

you could also do this way:

if($email && $message ){
if (mail($to, $subject, $message, $headers))
{
echo "Thank you for the feedback. We will be in touch with you very soon.";
}
else
{
echo 'unable to send email';
}

}else{
echo "Please enter your feedback or insert your email";
}
}