Form Vaildating

[monkeyhead]

Hi!
Heres My Script:

Feedback Form

<title>Contact Me</title>
<form method="post" action="mailfeedback.php">
<h2 align="left">Contact Me</h2>
<p>Email:
<input name="email" type="text" />
</p>
<p>Subject:
<input name="subject" type="text" id="subject" />
</p>
<p> Message:<br />
<textarea name="message" rows="12" cols="40">
</textarea>
<br />
<input name="submit" type="submit" />
<input name="reset" type="reset" id="reset" value="Reset" />
</p>
</form>

---

Script

<?
//Contact form script made by Matt Seymour
//Email: [e-mail]
//Website: http://www.mattseymour.co.uk
$email= $REQUEST['email'] ;
$message= $REQUEST['message'] ;

If (empty ($email)){
print'Please Enter Your Email Address!';
} else {

If (empty ($message)){
print'Please Enter Your Message!';
}else{
mail( "[e-mail]", "Someone Has Left Feedback At Your Site!",
$message, "From: $email" );
}
}
?>

<META HTTP-EQUIV="refresh" CONTENT="2; URL=thanks.html">

---

Heres my problem, when you do not fill in a field it displays the error i want but it then redirects to the thank you page.I only want the form to be redirceted if the form has been fillled in

Please Help me

Thanks for your time
Monkeyhead

[leatherback]

Hi,

Here a --part of-- some error trapping I do on user input:

$errors = '';

if($nights < 0) {$errors .= "You cannot leave before you arrive<br>";}
if(empty($initials)) {$errors .= "You have to fill out your initials<br>";}
if(empty($name)) {$errors .= "You have to fill out your name<br>";}
if(empty($city)) {$errors .= "You have to fill out your city<br>";}
if(empty($country)) {$errors .= "You have to fill out your country<br>";}
if(isset($_SESSION[form_processed])) {$errors .= "You have already stored your profile in this session. Please close all the browser windows, login again, and update again<br>";}


if(isset($_POST[formid]))                                    // Form was submitted
  {
  if(empty($errors))                                         // Input is OK
    {
      if(! isset($_SESSION[name]))                           // User is not logged in: No update of data
        {

    }                                       // End of new submission

 else                                       // Update

    {
    }                                       // End of update
//    echo $errors;

}                                          // end of no errors
else
{
  $body = "<b>The following errors occurred when processing your form:</b><br><hr>".$errors."<hr>Please close this window, correct the mistakes, and submit again";  
}
 

[marky2coats]

Hi,

Instead of using

<META HTTP-EQUIV="refresh" CONTENT="2; URL=thanks.html">

use
header("Location: thanks.html");
exit;

put it just after the mail command and delete the META refresh tag

Hope this helps

Mark.