PHP Form Help!

[Nyris]

I have this form setup to allow customers to fill in a little info and send an email to particular departments that they have their questions in. For some reason I cannot get it to work and in further playing around with it I get an HTTP 500 IS Error and it won't even load the page. Please help! Oh and I took the @ addres off the emails for this post. I know they're not there.

  $Dept   = $_REQUEST['Department'];
  $First  = $_REQUEST['firstname'];
  $Last   = $_REQUEST['lastname'];
  $Com    = $_REQUEST['company'];
  $Acc    = $_REQUEST['account'];
  $Add    = $_REQUEST['adderss'];
  $City   = $_REQUEST['city'];
  $State  = $_REQUEST['state'];
  $Zip    = $_REQUEST['zip'];
  $Phone  = $_REQUEST['phone'];
  $Email  = $_REQUEST['email'];
  $Help   = $_REQUEST['help'];
  $Sub    = $_REQUEST['subject'];
  $Mess   = $_REQUEST['message'];
 
  $message = $First." ".$Last."\n";
  $message .= $Com."\n";
  $message .= $Acc."\n\n";
  $message .= $Add."\n";
  $message .= $City." ".",".$State." ".",".$Zip."\n";
  $message .= $Phone."\n";
  $message .= $Email."\n";
  $message .= $Help."\n\n";
  $message .= $Mess;
 
  switch ($Dept) 
  {
    case "Repair Parts":
      $email_to = "rlacey", "hrichwine";
      break;
    case "Rebuilt Exchange Parts":
      $email_to="rlacey", "hrichwine";
      break;
     case "Conversions & Conversion Kits":
      $email_to="rlacey", "hrichwine";
      break;
    case "Test Equipment & Specialty Items":
      $email_to="rlacey", "hrichwine";
      break;
    case "Test Equipment & Specialty Items":
      $email_to="rlacey", "hrichwine";
      break;
    case "Customer Service":
      $email_to="rlacey", "hrichwine";
      break;
    case "Account Receivable":
      $email_to="kfatherree", "tfraker";
      break;
    case "Accounts Payable":
      $email_to="jlehman", "tfraker";
      break;
    case "Service Schools":
      $email_to="hrichwine";
      break;
    case "Rebuilt Exchange Parts":
      $email_to="sfrank";
      break;
    case "Operations":
      $email_to="ecowan", "eburkhart";
      break;
    case "President":
      $email_to="bbowman", "hrichwine";
      break;
    case "Service Center":
      $email_to="eburkhart", "ecowan";
      break;
    case "Sales / Marketing":
      $email_to="pjones", "tgasull";
      break;
    case "Tech Support":
      $email_to="dfreet", "ssteinour";
      break;
    default:
      $email_to = "ijimenez";  // fail-safe value    
  }
  mail( $email_to, "Feedback Form Results", $message, "From: $email" );

[Nyris]

What kind of file does this need to point to? I'm still somewhat confused. Sorry x.x!

[Dormilich]

any file that does form data processing. this includes PHP, ASP, Perl or any Server Side Script Language. but in your case it's the php file you originally posted.

regards

[Nyris]

The PHP file that I posted at the beginning is apart of the page that contains the form in. I put the PHP right above the form. So then I would just point it back to itself?

[Dormilich]

@Nyris
yes. but make sure you do not process the script when loading the page for the first time. this is usually one by a check for post data

if (isset($_POST['send'])) // this should be the name of the submit button
{
  // process form data
}

and one more note. when you specify the post protocol for data sending, you should use $_POST instead of $_REQUEST (for security reasons).

[Nyris]

I no longer get an error when I hit the send button!!! Getting somewhere, thanks a lot for your help so far Dormilich. I still didn't receive an email from it though, however.

This is how I have it setup now with the above if statement you suggested to me.

 <td colspan="2"><div align="center"><input type="submit" value="Send" name="send"/>
    <?php
    if (isset($_POST['send'])) // this should be the name of the submit button 
{ 
  $Dept   = $_POST['Department'];
  $First  = $_POST['firstname'];
  $Last   = $_POST['lastname'];
  $Com    = $_POST['company'];
  $Acc    = $_POST['account'];
  $Add    = $_POST['adderss'];
  $City   = $_POST['city'];
  $State  = $_POST['state'];
  $Zip    = $_POST['zip'];
  $Phone  = $_POST['phone'];
  $Email  = $_POST['email'];
  $Help   = $_POST['help'];
  $Sub    = $_POST['subject'];
  $Mess   = $_POST['message'];
 
  $message = $First." ".$Last."\n";
  $message .= $Com."\n";
  $message .= $Acc."\n\n";
  $message .= $Add."\n";
  $message .= $City." ".",".$State." ".",".$Zip."\n";
  $message .= $Phone."\n";
  $message .= $Email."\n";
  $message .= $Help."\n\n";
  $message .= $Mess;
 
  switch ($Dept) 
  {
    case "Repair Parts":
      $email_to = "rlacey". "hrichwine";
      break;
    case "Rebuilt Exchange Parts":
      $email_to="rlacey". "hrichwine";
      break;
     case "Conversions & Conversion Kits":
      $email_to="rlacey". "hrichwine";
      break;
    case "Test Equipment & Specialty Items":
      $email_to="rlacey". "hrichwine";
      break;
    case "Test Equipment & Specialty Items":
      $email_to="rlacey". "hrichwine";
      break;
    case "Customer Service":
      $email_to="rlacey". "hrichwine";
      break;
    case "Account Receivable":
      $email_to="kfatherree". "tfraker";
      break;
    case "Accounts Payable":
      $email_to="jlehman". "tfraker";
      break;
    case "Service Schools":
      $email_to="hrichwine";
      break;
    case "Rebuilt Exchange Parts":
      $email_to="sfrank";
      break;
    case "Operations":
      $email_to="ecowan". "eburkhart";
      break;
    case "President":
      $email_to="bbowman". "hrichwine";
      break;
    case "Service Center":
      $email_to="eburkhart". "ecowan";
      break;
    case "Sales / Marketing":
      $email_to="pjones". "tgasull";
      break;
    case "Tech Support":
      $email_to="dfreet". "ssteinour";
      break;
    default:
      $email_to = "ijimenez";  // fail-safe value     
  }
  mail( $email_to, "Feedback Form Results", $message, "From: $email" );
  }
?>

[Dormilich]

if you want to send email to several persons don't forget that the addresses mus be comma-separated.

$email_to="[e-mail], [e-mail]";

maybe a more convenient way to do the email addresses is to store the addresses in an array and call that.

$emails = array("john" => "[e-mail]", "don" => "[e-mail]");
// and so forth
 
switch ($Dept)
{
  case "Repair Parts":
    $email_to = $emails['don'] . ", " . $emails['john']; break;
  // ...
}

you could also define a constant for each email address

define("EMAIL_ADDR_JDOE", "[e-mail]");
// ...
$email_to = EMAIL_ADDR_JDOE . ", " . EMAIL_ADDR_DWORRY;
 

though that's only a possibility, I'd go for the array.

if that's not working do a var_dump() on the variables just before sending. this could give you a clue, if a variable has an invalid value.

regards

note: to protect your co-workers from spam, do not post any email addresses. every reasonable editor has a search function that can remove these via a regex. if it's too late to edit, ask a moderator to do it for you.

[Nyris]

It works! You're a genius Dormilich. Thanks a lot for your help!

[Dormilich]

@Nyris
no, I'm a XML god.... kidding. I'm glad I could help, that's why we're here.