[Resolved] Need help

youthman

Hi, I have created a simple form so visitors of my site can subscribe to my newsletter. The form contains a box for name, email, address, city/state, and zip. Below is the php script I'm using, but when I hit submit, I receive the error, "Please Fill Out All Fields." Anyone know how to fix my script to where it will work properly. Below is the php code.

php:

<?php
$email = $POST["email"];
$address = $POST["address"];
$citystate = $POST["citystate"];
$zip = $POST["zip"];
$headers = "From: " . $POST["email"];
$name = $POST["name"];
$subject = "Please Subscribe me to the DanaLittleton.com Newsletter"; // your webpage
$time = date("l d F Y @ H:i");
$to = "youthman@soleministries.org"; // your email
$message2 = "\nName: " . $name . "\nEmail: " . $email . "\nAddress: " . $address . "\nCity/State: " . $citystate . "\nZip: " . $zip ;

if ($email) {
    if (mail($to, $message2)) {
        print("Thank You for subscribing.  Click back to Return to DanaLittleton.com");
    }

    else {
        print("The server encountered an unexpected condition which prevented it from fulfilling the request.");
    }
}

else {
    print("Please fill out all fields.");
}

sid

i guess the email input field actually doesnt have the NAME="email" (case sensitive) - check your form code!

also - make sure youre using METHOD="post" in your form tag.

mtimdog

If the above doesn't work, then look at how you're testing email
if ($email)

try

if ($email != '')

because $email will always be set to whatever $_POST['email'] is.

youthman

Sorry sid, but I'm really new to php. The form I used on my site that I am trying to modify was not one I created. Unfortunately, I don't understand what you are wanting me to do.

sid

go ahead and post the code for the form.

youthman

<form action="cgi-bin/test.php" method="get" name="newsletterform">
<table width="599" border="0" cellspacing="2" cellpadding="0">
<tr>
<td width="20">.</td>
<td width="20"></td>
<td width="50"></td>
<td colspan="2">
<hr>
</td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td colspan="2">
<div align="left">
<img src="images/newsletter.jpg" alt="" height="58" width="500" border="0"></div>
</td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td colspan="2"><font color="black">DanaLittleton.com is committed to your privacy. It is our policy that all email addresses and information submitted to DanaLittleton.com</font> will be kept strictly confidential and private. Be assured our mailing list will NEVER be shared or sold as a whole or in part</td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td></td>
<td></td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td><label>NAME:</label></td>
<td><input type="text" name="name" size="24" tabindex="1"></td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td><label>EMAIL ADDRESS:</label></td>
<td><input type="text" name="email" size="24" tabindex="2"></td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td><label>ADDRESS:</label></td>
<td><input type="text" name="address" size="24" tabindex="3"></td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td><label>CITY,STATE</label></td>
<td><input type="text" name="citystate" size="24" tabindex="4"></td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td><label>ZIP CODE</label></td>
<td><input type="text" name="zip" size="24" tabindex="5"></td>
</tr>
<tr>
<td width="20"></td>
<td width="20"></td>
<td width="50"></td>
<td>

							</td>
							<td>
								<div align="left">
									<input type="submit" name="submitButtonName" value="Submit" tabindex="6"></div>
							</td>
						</tr>
						<tr>
							<td width="20"></td>
							<td width="20"></td>
							<td width="50"></td>
							<td colspan="2">
								<hr>
							</td>
						</tr>
					</table>
				</form>

youthman

Sid, I changed in GoLive the name for the form to email as you suggested. You can see the actual form at www.danalittleton.com/newsletter.html Hope that helps ya. Thank you so much for your help too.

youthman

Sid, YOU ROCK MAN! Thank you so much for helping me fix the problem. I went into the code and when i changed it from get to post, it worked. is there a simple way have it go to a thankyou.html page instead of just "Thank You for subscribing. Click back to Return to DanaLittleton.com"? I don't mind doing the research myself, but don't know what to search for in the forums.

sid

no problem - im glad i could help...

is there a simple way have it go to a thankyou.html page instead of just "Thank You for subscribing. Click back to Return to DanaLittleton.com"?

try changing the ode to this:

if ($email) { 
if (mail($to, $message2)) { 
header("Location:thankyou.html"); 
}

make sure there is no output (also no white spaces) before this.

youthman

Cool it works great. I have finished the form and it now takes you to a thank you page. I also updated my youth ministry website with a similar form using the same script (with a few modifications to it) and it got it up and running in about 10 minutes. You can check it out at www.soleministries.org/contact_us.html My next venture is to learn how to modify the script to where it checks the syntax of the email address and phone number in case they key it in wrong, it will ask them to go back and correct it. Thanks again for all of your help bro.