Help!? Php errors...

[pemdas21]

Here is the code for the php:

<?PHP
$email = $HTTP_POST_VARS[email];
$Person = $HTTP_POST_VARS[Person];
if ($Person == "John Smith") {
$mailto = "[email protected]";
}
if ($Person == "George Smith") {
$mailto = "[email protected]";
}
// repeat the above If statement as many times as the persons
$mailsubj = $HTTP_POST_VARS[Subject];
$Message = $HTTP_POST_VARS[Message];
$mailhead = "From: $email\n";
$mailbody = "$Message\n"
. "\n"
. "Email sent through our Web Mail form\n" ;
mail($mailto, $mailsubj, $mailbody, $mailhead);
?>

Here are the errors:

Notice: Use of undefined constant email - assumed 'email' in C:\Sites\www.orrita.com\new\emails.php on line 2

Notice: Undefined index: email in C:\Sites\www.orrita.com\new\emails.php on line 2

Notice: Use of undefined constant Person - assumed 'Person' in C:\Sites\www.orrita.com\new\emails.php on line 3

Notice: Use of undefined constant Subject - assumed 'Subject' in C:\Sites\www.orrita.com\new\emails.php on line 89

Notice: Use of undefined constant Message - assumed 'Message' in C:\Sites\www.orrita.com\new\emails.php on line 90

Notice: Undefined variable: mailto in C:\Sites\www.orrita.com\new\emails.php on line 95

Warning: mail(): SMTP server response: 554 Error: no valid recipients in C:\Sites\www.orrita.com\new\emails.php on line 95

[fert]

simple enough $mailto is empty and so the email isn't being sent

[trq]

Your missing quotes. eg;

[code=php:0]
$email = $HTTP_POST_VARS[email];
[/code]

should be...

[code=php:0]
$email = $HTTP_POST_VARS['email'];
[/code]

Also be aware that $HTTP_POST_VARS has long been depricated in favour of $_POST.

[pemdas21]

wait, so what does the script look like completely fixed?

[trq]

I dunno, rewrite it and see.

[pemdas21]

<?PHP
$email = $HTTP_POST_VARS['email'];
$Person = $HTTP_POST_VARS['Person'];
if ($Person == "John Smith") {
$mailto = "[email protected]";
}
if ($Person == "George Smith") {
$mailto = "[email protected]";
}
// repeat the above If statement as many times as the persons
$mailsubj = $HTTP_POST_VARS['Subject'];
$Message = $HTTP_POST_VARS['Message'];
$mailhead = "From: $email\n";
$mailbody = "$Message\n"
. "\n"
. "Email sent through our Web Mail form\n" ;
mail($mailto, $mailsubj, $mailbody, $mailhead);
?>

[trq]

If I must...

[code]
<?php
  if (isset($_POST['email') && isset($_POST['Person']) && isset($_POST['Message']) && isset($_POST['Subject'])) {
    $email  = $_POST['email'];
    $Person = $_POST['Person'];
    if ($Person == "John Smith") {
      $mailto = "[email protected]";
    }
    if ($Person == "George Smith") {
      $mailto = "[email protected]";
    }
    $mailsubj = $_POST['Subject'];
    $Message = $_POST['Message'];
    $mailhead = "From: $email\n";
    $mailbody = "$Message\n\nEmail sent through our Web Mail form\n" ;
    mail($mailto, $mailsubj, $mailbody, $mailhead);
  }
?>
[/code]

[pemdas21]

Parse error: parse error, unexpected $end in C:\Sites\www.orrita.com\emails.php

what does that mean?

[Nhoj]

That usually means you are missing a closing brace or ';' somewhere in your coding.

[chronister]

Missing ] in line 1.

[code]if (isset($_POST['email']) [/code]