[Resolved] Form Help -- Choosing a Recipient

[l337godd3ss]

Hi. I am trying to create a form where the user can choose a recipient from a drop down menu. I am extremely new to php, and have managed to piece together some code from some tutorials:

The code for the form:

<form action="send_form.php" method="post">
Name: <input type="text" name="name"><br><br>
Email (optional): <input type="text" name="email"><br><br>
Choose a Recipient:
<select name="recipient">
	<option>(select one)
	<option name="person1">person1
	<option name="person2">person2
	</select><br><br>
Message:
<textarea name="message" rows="10" cols="10"></textarea><br><br>
<input type="submit" name="submit" value="Send">
<input type="reset" value="Reset">
</form>

And the code for send_form.php:

<?php
	{

    if($_POST['recipient'] == "person1")
	{		
	  $sendto     = "[e-mail]";	
	  $recipient  = $_POST['recipient'];
	  $name       = $_POST['name'];
	  $email      = $_POST['email'];
	  $message    = $_POST['message'];

	  mail( "$sendto", "Feedback From Site",
			"Name: $name\n\n$message",
		    "From: $name" );
	}

	else if($_POST['recipient'] == "person2")
	{
	  $sendto     = "[e-mail]";	
	  $recipient  = $_POST['recipient'];
	  $name       = $_POST['name'];
	  $email      = $_POST['email'];
	  $message    = $_POST['message'];

	  mail("$sendto", "Feedback From Site",
			"Name: $name\n\n$message",
		    "From: $name" );
	}
?>

How can I fix or re-write this code to make it work? Thanks!

[herve]

What's happening right now ?

What's the value inside your
$_POST['recipient'] variable when
the php program is run ?

[l337godd3ss]

When I try to use the form, everything seems to work perfectly. However, nothing gets e-mailed to me.

See...um, the problem is, I am really new to php, and I don't really know what all the variables mean...I think the value inside of $_POST['recipient'] is supposed to be person1 or person2.

[herve]

Originally posted by l337godd3ss
When I try to use the form, everything seems to work perfectly. However, nothing gets e-mailed to me.

See...um, the problem is, I am really new to php, and I don't really know what all the variables mean...I think the value inside of $_POST['recipient'] is supposed to be person1 or person2.

Yes, that's what is supposed to be but
check with a print statement,

ie :

<?php

print "recipient=".$_POST["recipient"];   // <= just to check the value, you don't need this line after

    if($_POST['recipient'] == "person1")
	{		
	  $sendto     = "[e-mail]";	
	  $recipient  = $_POST['recipient'];
	  $name       = $_POST['name'];
	  $email      = $_POST['email'];
	  $message    = $_POST['message'];

	  mail( "$sendto", "Feedback From Site",
			"Name: $name\n\n$message",
		    "From: $name" );
	}
	else if($_POST['recipient'] == "person2")
	{
	  $sendto     = "[e-mail]";	
	  $recipient  = $_POST['recipient'];
	  $name       = $_POST['name'];
	  $email      = $_POST['email'];
	  $message    = $_POST['message'];

	  mail("$sendto", "Feedback From Site",
			"Name: $name\n\n$message",
		    "From: $name" );
	}
?>

Also don't put a blank line before the "else if()" statement.

[tekky]

<?php
// lose this bracket	{

if($_POST['recipient'] == "person1")
{		
    $sendto     = "[e-mail]";	
    $recipient  = $_POST['recipient'];
    $name       = $_POST['name'];
    $email      = $_POST['email'];
    $message    = $_POST['message'];
}
//the empty line here doesn't matter PHP doesnt care if its back to back or if there are 20 blank lines between them -- but for readability its best not to do huge gaps like that
else if($_POST['recipient'] == "person2")
{
    $sendto     = "[e-mail]";	
    $recipient  = $_POST['recipient'];
    $name       = $_POST['name'];
    $email      = $_POST['email'];
    $message    = $_POST['message'];
}
//move this outside of the if statements, no reason to repeat this twice since its the same for both recipients
mail( "$sendto", "Feedback From Site",
				"Name: $name\n\n$message",
			    "From: $name" );
?>

also if you wish to check your posted data this works well

//just put this at the beginning of your php script
echo '<pre>';
print_r($_POST);
echo '</pre><br>';

this will display all of your $_POST data in a decently readable format.

[l337godd3ss]

Okay, I changed my code, but I am still not getting an e-mail! Did I type something wrong???

Form code:

<form action="thankyou.php" method="post">
Name: <input type="text" name="name"><br><br>
Email (optional): <input type="text" name="email"><br><br>
Choose a Recipient:
<select name="recipient">
	<option>(select one)
	<option name="person">Person
	<option name="webmaster">Webmaster
	</select><br><br>
Message:
<textarea name="message" rows="10" cols="10"></textarea><br><br>
<input type="submit" name="submit" value="Send">
<input type="reset" value="Reset">
</form>

For thankyou.php

<?php 

if($_POST['recipient'] == "person")
{
    $sendto     = "[e-mail]";
    $recipient  = $_POST['recipient'];
    $name       = $_POST['name'];
    $email      = $_POST['email'];
    $message    = $_POST['message'];
}

else if($_POST['recipient'] == "webmaster")
{
    $sendto     = "[e-mail]";
    $recipient  = $_POST['recipient'];
    $name       = $_POST['name'];
    $email      = $_POST['email'];
    $message    = $_POST['message'];
}

mail( "$sendto", "Feedback From Site",
                "Name: $name\n\n$message",
                "From: $name" ); 
?>
Thank you!! Your message has been sent!!

Does anyone know what I should fix/change so the form e-mails to me?

[l337godd3ss]

Tried messing around with the code with no luck. (just screwed everything up) If anyone has any ideas please reply!!! 🙂 Thanks a ton!!! 😉

[tekky]

have you checked that your host allows use of the mail options in PHP? ie. they have sendmail running etc?

[l337godd3ss]

Oh, geez....:o That's the problem!! I just searched around on my host's help section, and they won't allow forms unless you use e-mails they host. Arg, how stupid...time to find a new host....

Thanks SO much tekky!!!!!!!! 🙂

[tekky]

Originally posted by l337godd3ss
Oh, geez....:o That's the problem!! I just searched around on my host's help section, and they won't allow forms unless you use e-mails they host. Arg, how stupid...time to find a new host....

Thanks SO much tekky!!!!!!!! 🙂

Hehe, no problem, I shoulda asked that first 😃

[l337godd3ss]

Thanks again tekky!!! Okay everyone, this problem has been resolved!! Here's the final code:

Code for the Form:

<form action="send_form.php" method="post"> 
Name: <input type="text" name="name"><br><br> 
Email (optional): <input type="text" name="email"><br><br> 
Choose a Recipient: 
<select name="recipient"> 
    <option>(select one) 
    <option name="person">Person 
    <option name="webmaster">Webmaster 
    </select><br><br> 
Message: 
<textarea name="message" rows="10" cols="10"></textarea><br><br> 
<input type="submit" name="submit" value="Send"> 
<input type="reset" value="Reset"> 
</form>

Code for send_form.php

<?php  

if($_POST['recipient'] == "person") 
{ 
    $sendto     = "[e-mail]"; 
    $recipient  = $_POST['recipient']; 
    $name       = $_POST['name']; 
    $email      = $_POST['email']; 
    $message    = $_POST['message']; 
} 

else if($_POST['recipient'] == "webmaster") 
{ 
    $sendto     = "[e-mail]"; 
    $recipient  = $_POST['recipient']; 
    $name       = $_POST['name']; 
    $email      = $_POST['email']; 
    $message    = $_POST['message']; 
} 

mail( "$sendto", "Feedback From Site", 
                "Name: $name\n\n$message", 
                "From: $name" );  

?>

[tekky]

One more suggestion to save space and a tiny bit of time...

Code for send_form.php

<?php  

if($_POST['recipient'] == "person") 
{ 
    $sendto     = "[e-mail]"; 
} 
else if($_POST['recipient'] == "webmaster") 
{ 
    $sendto     = "[e-mail]"; 
} 
//these are always the same... so only need to do them the same way once
$recipient  = $_POST['recipient']; 
$name       = $_POST['name']; 
$email      = $_POST['email']; 
$message    = $_POST['message']; 
mail( "$sendto", "Feedback From Site", 
                "Name: $name\n\n$message", 
                "From: $name" );  

?>

The rest is good to go 😉

[l337godd3ss]

Thanx Karl!! 😃