anything wrg with this code?

[Myriadz]

I keep receiving a parse error....

if ($_POST[dept] = $support) {

  $sendto = "[e-mail]";

  } else if ($_POST[dept] = $sales) {

  $sendto = "[e-mail]";

  } else if($_POST[dept] = $billing) {

  $sendto = "[e-mail]";

  } 

[pumpkinapo]

Your missing curly braces in your else-if statements. Try the following code:

if ($_POST[dept] = $support) {
	$sendto = "[e-mail]";
} else {
	if ($_POST[dept] = $sales) {
		$sendto = "[e-mail]";
	} else {
		if($_POST[dept] = $billing) {
			$sendto = "[e-mail]";
		} 
	}
}

[BuzzLY]

1. use the proper elseif statement.
2. use the proper comparator (==).
3. use quotes in your variables (not necessary, but best coding practice).

if ($_POST['dept'] == $support) {
      $sendto = "[e-mail]";
} elseif ($_POST['dept'] == $sales) {
      $sendto = "[e-mail]";
} elseif ($_POST['dept'] == $billing) {
      $sendto = "[e-mail]";
}

You may also want to take a look at the switch statement. It is a much more elegant solution, especially if you need to add new email addresses, or have a default email address.

Hope that helps!

[Myriadz]

Hi, I'm still unable to receive the emails though it states "successfully sent".

However, if i do this....i will receive the email

if ($_POST[dept] == $support) {

  $sendto = "[e-mail]";

  } 

But I won't receive the emails with this....
I'm at a loss...please help me

if ($_POST[dept] == $support) {

  if ($_POST[dept] == $support) {

  $sendto == "[e-mail]";

  } elseif ($_POST[dept] == $sales) {

  $sendto = "[e-mail]";

  } elseif ($_POST[dept] == $billing) {

  $sendto = "[e-mail]";

  }

  } 

[BuzzLY]

Try the code I gave you. Your code won't work if $_POST['dept'] does not equal $support. The code I wrote in my previous post should do the trick.

After your if statements, simply print $_POST['dept'] and $sendto to see what the values are. That should give you a clue as to why your code isn't working.

[Myriadz]

Hi, I followed your heed and the code still isn't working properly.
I had echo the $sendto in each statement.

However, not matter what I selected, [e-mail] (the 1st option always prints out)

Sales, Support though selected, prints out [e-mail] as well. Help!!

<form method= 'POST' action= '<?= $_SERVER['PHP_SELF'] ?>'>
<select name= 'dept'>
<option value=" ">Select a Department</option> 
<option value= "<?= $support?>">Technical</option>
<option value= "<?= $sales?>">Sales</option>
<option value= "<?= $billing?>">Billing</option>
</select>

if ($_POST['dept'] == $support) {

  $sendto = "[e-mail]";
  echo $sendto;

  } elseif ($_POST['dept'] == $sales) {

  $sendto = "[e-mail]";
  echo $sendto;

  } elseif ($_POST['dept'] == $billing) {

  $sendto = "[e-mail]";
  echo $sendto;

  } // end nested if / elseif structure

[BuzzLY]

Aha... I think I see your problem. In this code:

<form method= 'POST' action= '<?= $_SERVER['PHP_SELF'] ?>'>
<select name= 'dept'>
<option value=" ">Select a Department</option> 
<option value= "<?= $support?>">Technical</option>
<option value= "<?= $sales?>">Sales</option>
<option value= "<?= $billing?>">Billing</option>
</select>

... when you first load the form, $support, $sales, and $billing all equal the same thing... nothing! So, whatever you select makes $_POST['dept'] equal nothing. Since the first condition in your if statment (if nothing equals nothing) is met, $sendto will always be [e-mail]. Make sense?

Here's what you need to do:

1. Your select tag needs to be called dept[] (with the brackets). This tells PHP that it's an array (even though it will only have one value).

2. each option should either not have a value, or have a value equivalent to its choice (like "support" for Technical, etc.).

3. You need this code to extract the value of "dept" once the form is posted:

   if (!empty($_POST["dept"])) {
   	foreach($_POST["dept"] as $value) {
   		$dept = $value;
   	}
   }

4. Now, modify your if statement. Better yet, make it a switch statement:

switch ($dept) {
    case "support":
        $sendto = "[e-mail]";
        break;
    case "sales":
        $sendto = "[e-mail]";
        break;
    case "billing":
        $sendto = "[e-mail]";
        break;
    default:
        $sendto = "[e-mail]";
        break;
}

Let us know if that does the trick.