Hi,

My apologies if this is already posted, I've searched everywhere (google, etc) but just can't seem to get it right.

My form has a drop down in it where people may select a city ... and each city has it's own email.

For example:

If city 'Mathis' is selected, then an email will go to x@mathis.com
If city 'Orange Grove' is selected, then an email will go to x@orangegrove.com

and so on ...

Below is where I'm at with he form, but I can't get this to work.

I'd really appreciate any help with this, thank you so much ...

<?php
$EmailFrom = "xxx";
$EmailTo = "xxx";
if (Locations===Marthis) {
     mailto = "x@mathis.com";
} else if (Locations===Orange Grove) {
     mailto = "x@og.com";
}
$Subject = "Customer Request";
$locations= htmlspecialchars(Trim(stripslashes($_POST['Locations'])));
$Name = htmlspecialchars(Trim(stripslashes($_POST['Name'])));
$Phone = htmlspecialchars(Trim(stripslashes($_POST['Phone'])));
$Email = htmlspecialchars(Trim(stripslashes($_POST['Email'])));
$Message = htmlspecialchars(Trim(stripslashes($_POST['Message'])));
// validation
$validationOK=true;
if (!$validationOK) {
  print "<meta http-equiv=\"refresh\" content=\"0;URL=error.htm\">";
  exit;
}
// prepare email body text
$Body = "";
$Body .= "Name: ";
$Body .= $Name;
$Body .= "\n";
$Body .= "Phone: ";
$Body .= $Phone;
$Body .= "\n";
$Body .= "Email: ";
$Body .= $Email;
$Body .= "\n";
$Body .= "Location: ";
$Body .= $Locations;
$Body .= "\n";
$Body .= "Request: ";
$Body .= $Request;
$Body .= "\n";
// send email
$success = mail($EmailTo, $Subject, $Body, "From: <$EmailFrom>");
// redirect to success page
if ($success){
  print "<meta http-equiv=\"refresh\" content=\"0;URL=submit.htm\">";
}
else{
  print "<meta http-equiv=\"refresh\" content=\"0;URL=error.htm\">";
}
?>

Jennifer ~

Oh, pretty, pretty please go get some elementary training in coding in PHP. You're going to get yourself in trouble with such coding.

A couple of pointers:

1) Assume case matters! "locations" is not always equal to "Locations".
2) Learn what a variable is and then learn what a constant is. You won't be leaving "$" symbols missing anymore.

I could give you more suggestions, but the most important is to go learn before you create something live!

Here's adjusted code:

PHP Code:

<?php
$locations_to_emails = array('Mathis'=>'mathis@mydomain.com',
                             'Orange Grove'=>'orange_grove@mydomain.com'
                            );
$Subject = "Customer Request";
$Locations = htmlspecialchars(trim(stripslashes($_POST['Locations'])));
$Name = htmlspecialchars(trim(stripslashes($_POST['Name'])));
$Phone = htmlspecialchars(trim(stripslashes($_POST['Phone'])));
$Email = htmlspecialchars(trim(stripslashes($_POST['Email'])));
$Message = htmlspecialchars(trim(stripslashes($_POST['Message'])));

$EmailFrom = "xxx";
$EmailTo = isset($locations_to_emails[$Locations])?$locations_to_emails[$Locations]:'YOUR_DEFAULT_EMAIL_ADDRESS';

// validation
$validationOK=true;
if (!$validationOK) {
  print "<meta http-equiv=\"refresh\" content=\"0;URL=error.htm\">";
  exit;
}
// prepare email body text
$Body = "";
$Body .= "Name: ";
$Body .= $Name;
$Body .= "\n";
$Body .= "Phone: ";
$Body .= $Phone;
$Body .= "\n";
$Body .= "Email: ";
$Body .= $Email;
$Body .= "\n";
$Body .= "Location: ";
$Body .= $Locations;
$Body .= "\n";
$Body .= "Request: ";
$Body .= $Request;
$Body .= "\n";
// send email
$success = mail($EmailTo, $Subject, $Body, "From: <$EmailFrom>");
// redirect to success page
if ($success){
  print "<meta http-equiv=\"refresh\" content=\"0;URL=submit.htm\">";
}
else{
  print "<meta http-equiv=\"refresh\" content=\"0;URL=error.htm\">";
}
?>

Originally Posted by JeremyMiller

Oh, pretty, pretty please go get some elementary training in coding in PHP. You're going to get yourself in trouble with such coding.

A couple of pointers:

1) Assume case matters! "locations" is not always equal to "Locations".
2) Learn what a variable is and then learn what a constant is. You won't be leaving "$" symbols missing anymore.

I could give you more suggestions, but the most important is to go learn before you create something live!

Here's adjusted code:

PHP Code:

<?php
$locations_to_emails = array('Mathis'=>'mathis@mydomain.com',
                             'Orange Grove'=>'orange_grove@mydomain.com'
                            );
$Subject = "Customer Request";
$Locations = htmlspecialchars(trim(stripslashes($_POST['Locations'])));
$Name = htmlspecialchars(trim(stripslashes($_POST['Name'])));
$Phone = htmlspecialchars(trim(stripslashes($_POST['Phone'])));
$Email = htmlspecialchars(trim(stripslashes($_POST['Email'])));
$Message = htmlspecialchars(trim(stripslashes($_POST['Message'])));

$EmailFrom = "xxx";
$EmailTo = isset($locations_to_emails[$Locations])?$locations_to_emails[$Locations]:'YOUR_DEFAULT_EMAIL_ADDRESS';

// validation
$validationOK=true;
if (!$validationOK) {
  print "<meta http-equiv=\"refresh\" content=\"0;URL=error.htm\">";
  exit;
}
// prepare email body text
$Body = "";
$Body .= "Name: ";
$Body .= $Name;
$Body .= "\n";
$Body .= "Phone: ";
$Body .= $Phone;
$Body .= "\n";
$Body .= "Email: ";
$Body .= $Email;
$Body .= "\n";
$Body .= "Location: ";
$Body .= $Locations;
$Body .= "\n";
$Body .= "Request: ";
$Body .= $Request;
$Body .= "\n";
// send email
$success = mail($EmailTo, $Subject, $Body, "From: <$EmailFrom>");
// redirect to success page
if ($success){
  print "<meta http-equiv=\"refresh\" content=\"0;URL=submit.htm\">";
}
else{
  print "<meta http-equiv=\"refresh\" content=\"0;URL=error.htm\">";
}
?>

Learning = screwing up.

How are we supposed to continue learning without practicing and/or asking questions?

I'm assuming you knew it all before ever starting then ...

Originally Posted by jennifer420

How are we supposed to continue learning without practicing and/or asking questions?

learning and practicing is two separate things... jeremy suggests you learn first then practice not vice versa...

you should be reading up PHP from a few tutorial sites then attempting to code because when you just practice you might get a full 20 page/script site working like you want... but most people don't use sites like you would expect therefore with a strong knowledge of the language you're able to cover all the loopholes and all the exception cases...

Originally Posted by orionoreo

learning and practicing is two separate things... jeremy suggests you learn first then practice not vice versa...

you should be reading up PHP from a few tutorial sites then attempting to code because when you just practice you might get a full 20 page/script site working like you want... but most people don't use sites like you would expect therefore with a strong knowledge of the language you're able to cover all the loopholes and all the exception cases...

Well said.

@jennifer: As an enticement, consider that the first step academics will tell you to coding has nothing to do with sitting down in front of a computer.