php form

randyw · August 14, 2009

i have a php from collects the data from an array and places it in a database

the form has the different sections

shows_Selected

additional_Info

further_information

form proccess

<?php

include_once("include/dbc.php");

// Verify that all form elements are completed

if (empty($_POST['first_name'])){

//die ("ERROR: Please add your First Name..");

header("Location: form.php?msg=ERROR: Please add your First Name..");

exit();

}

if (empty($_POST['last_name'])){

//die ("ERROR: Please add your Last Name..");

header("Location: form.php?msg=ERROR: Please add your Last Name..");

exit();

}

if (empty($_POST['email_address'])){

//die ("ERROR: Please add your Email..");

header("Location: form.php?msg=ERROR: Please add your Email..");

exit();

}

if (empty($_POST['mailling_address'])){

//die ("ERROR: Please add your Address..");

header("Location: form.php?msg=ERROR: Please add your Address..");

exit();

}

if (empty($_POST['city'])){

//die ("ERROR: Please add your City..");

header("Location: form.php?msg=ERROR: Please add your City..");

exit();

}

if (empty($_POST['zip'])){

//die ("ERROR: Please add your City..");

header("Location: form.php?msg=ERROR: Please add your City..");

exit();

}

if (empty($_POST['home_phone'])){

//die ("ERROR: Please add your Home Phone..");

header("Location: form.php?msg=ERROR: Please add your Home Phone..");

exit();

}

if (empty($_POST['work_phone'])){

//die ("ERROR: Please add your WorkPhone..");

header("Location: form.php?msg=ERROR: Please add your WorkPhone..");

exit();

}

else{

$media_array = $_POST['shows_Selected'];

foreach ($media_array as $one_media) {

$source .= $one_media."<br>";

}

$shows_Selected = substr($source, 0, -4);

{

// Now process the Addition Information array from POST

$addlInfo_array = $_POST['additional_Info'];

foreach ($addlInfo_array as $one_addl) {

$addlInfo .= $one_addl."<br>";

}

$additional_Info = substr($addlInfo, 0, -4);

{

// Now process the Addition Information array from POST

$furtherInfo_array = $_POST['further_information'];

foreach ($furtherInfo_array as $one_futher) {

$furtherInfo .= $one_further."<br>";

}

$further_information = substr($furtherInfo, 0, -4);

{

// update data in mysql database

$sql = "INSERT INTO newportweddingExpoForm

(shows_Selected, additional_Info, further_information, first_name, last_name, Role, email_address, home_phone, work_phone, city, state, zip, mailling_address, apt_number, wedding_month, wedding_day, wedding_year, Wedding_Tentative, wedding_Region, wedding_city, wedding_state, budget, find_aboutUs)VALUES('$shows_Selected', '$additional_Info', '$further_information', '$first_name', '$last_name', '$role', '$email_address', '$home_phone', '$work_phone', '$city', '$state', '$zip', '$mailling_address', '$apt_number', '$wedding_month', '$wedding_day', '$wedding_year', '$Wedding_Tentative', '$wedding_Region', '$wedding_city', '$wedding_state', '$budget', '$find_aboutUs')";

$result = @mysql_query($sql);

}

if($result){

//die ("Your form information has been successfully sent..");

header("Location: form.php?msg=Your form information has been successfully sent..");

exit();

}

else {

//die ("Iam sorry there was a problem, duplicate form name allready sent..");

header("Location: form.php?msg=Iam sorry there was a problem, duplicate form name allready sent..");

exit();

}

mysql_close();

?>

<?php

}

?>

<?php

}

?>

<?php

}

?>

this works great

however if the dont check any of the boxes i get this error

Warning: Invalid argument supplied for foreach() in /home/va010021/public_html/newportweddingExpo/form_procces.php on line 73

mikesta707 · August 14, 2009

there are multiple foreach statements. which line is line 73. also wrap your code in code tags next time, especially if you have that much.

also, the following area of code


if (empty($_POST['work_phone'])){
   //die ("ERROR: Please add your WorkPhone..");
         header("Location: form.php?msg=ERROR: Please add your WorkPhone..");
         exit();


}
else{

might not be doing what you think its doing. That else statement is only connected to the if statement above it, and is not connected to any other if statement in that code. If you want it to be connected to the rest, you have to use else if statements. This won't make a huge difference, or really any difference the way your code is now, but may cause some problems in the future.

randyw · August 14, 2009

i have a php from collects the data from an array and places it in a database

the form has the different sections

shows_Selected

<input type="checkbox" name="shows_Selected[]" value="January 10, 2010 Seaport World Trade Center, Boston, MA" tabindex="5">

additional_Info

<input type="checkbox" name="additional_Info[]" value="Yes, I'd like to receive info on wedding related specials via "snail" mail." checked tabindex="150">

further_information

<input type="checkbox" name="further_information[]" value="Alterations">

form proccess

<?php

include_once("include/dbc.php");

// Verify that all form elements are completed

if (empty($_POST['first_name'])){

//die ("ERROR: Please add your First Name..");

header("Location: form.php?msg=ERROR: Please add your First Name..");

exit();

}

if (empty($_POST['last_name'])){

//die ("ERROR: Please add your Last Name..");

header("Location: form.php?msg=ERROR: Please add your Last Name..");

exit();

}

if (empty($_POST['email_address'])){

//die ("ERROR: Please add your Email..");

header("Location: form.php?msg=ERROR: Please add your Email..");

exit();

}

if (empty($_POST['mailling_address'])){

//die ("ERROR: Please add your Address..");

header("Location: form.php?msg=ERROR: Please add your Address..");

exit();

}

if (empty($_POST['city'])){

//die ("ERROR: Please add your City..");

header("Location: form.php?msg=ERROR: Please add your City..");

exit();

}

if (empty($_POST['zip'])){

//die ("ERROR: Please add your City..");

header("Location: form.php?msg=ERROR: Please add your City..");

exit();

}

if (empty($_POST['home_phone'])){

//die ("ERROR: Please add your Home Phone..");

header("Location: form.php?msg=ERROR: Please add your Home Phone..");

exit();

}

if (empty($_POST['work_phone'])){

//die ("ERROR: Please add your WorkPhone..");

header("Location: form.php?msg=ERROR: Please add your WorkPhone..");

exit();

}

else{

this is line 73

$media_array = $_POST['shows_Selected'];

foreach ($media_array as $one_media) {

$source .= $one_media."<br>";

}

$shows_Selected = substr($source, 0, -4);

{

// Now process the Addition Information array from POST

$addlInfo_array = $_POST['additional_Info'];

foreach ($addlInfo_array as $one_addl) {

$addlInfo .= $one_addl."<br>";

}

$additional_Info = substr($addlInfo, 0, -4);

{

// Now process the Addition Information array from POST

$furtherInfo_array = $_POST['further_information'];

foreach ($furtherInfo_array as $one_futher) {

$furtherInfo .= $one_further."<br>";

}

$further_information = substr($furtherInfo, 0, -4);

{

// update data in mysql database

$sql = "INSERT INTO newportweddingExpoForm

(shows_Selected, additional_Info, further_information, first_name, last_name, Role, email_address, home_phone, work_phone, city, state, zip, mailling_address, apt_number, wedding_month, wedding_day, wedding_year, Wedding_Tentative, wedding_Region, wedding_city, wedding_state, budget, find_aboutUs)VALUES('$shows_Selected', '$additional_Info', '$further_information', '$first_name', '$last_name', '$role', '$email_address', '$home_phone', '$work_phone', '$city', '$state', '$zip', '$mailling_address', '$apt_number', '$wedding_month', '$wedding_day', '$wedding_year', '$Wedding_Tentative', '$wedding_Region', '$wedding_city', '$wedding_state', '$budget', '$find_aboutUs')";

$result = @mysql_query($sql);

}

if($result){

//die ("Your form information has been successfully sent..");

header("Location: form.php?msg=Your form information has been successfully sent..");

exit();

}

else {

//die ("Iam sorry there was a problem, duplicate form name allready sent..");

header("Location: form.php?msg=Iam sorry there was a problem, duplicate form name allready sent..");

exit();

}

mysql_close();

?>

<?php

}

?>

<?php

}

?>

<?php

}

?>

this works great

however if the dont check any of the boxes i get this error

Warning: Invalid argument supplied for foreach() in /home/va010021/public_html/newportweddingExpo/form_procces.php on line 73

mikesta707 · August 14, 2009

... thats not how you wrap code in code tags but whatever...

so line 73 is an empty line? I still don't quite understand. Is like 73 the one below where you indicated?

ill assume that line 73 is the foreach two lines below where you indicated.

do a print_r on the $media_array variable and see what you get

randyw · August 14, 2009

how do you do a print ri have never heard of it im new to php

mikesta707 · August 14, 2009

there is a function called print_r().

print_r($media_array);

what happens when you run that

ziegel · August 14, 2009

print_r($media_array);

Edit: Oops someone beat me to it!

randyw · August 14, 2009

there is a function called print_r().
print_r($media_array);
what happens when you run that

i got this

Warning: Invalid argument supplied for foreach() in /home/va010021/public_html/newportweddingExpo/form_procces.php on line 73

Warning: Invalid argument supplied for foreach() in /home/va010021/public_html/newportweddingExpo/form_procces.php on line 82

Warning: Invalid argument supplied for foreach() in /home/va010021/public_html/newportweddingExpo/form_procces.php on line 90

Warning: Cannot modify header information - headers already sent by (output started at /home/va010021/public_html/newportweddingExpo/form_procces.php:73) in /home/va010021/public_html/newportweddingExpo/form_procces.php on line 111

mikesta707 · August 14, 2009

none of those errors have anything to do with the print_r function. it is clear that none of the foreach statements have an array as their supplied argument. It appears that the post data is not an array.

post the part of the form that shows the checkboxes. if you only have 1 check box.. then it won't be an array.

randyw · August 14, 2009

none of those errors have anything to do with the print_r function. it is clear that none of the foreach statements have an array as their supplied argument. It appears that the post data is not an array.

post the part of the form that shows the checkboxes. if you only have 1 check box.. then it won't be an array.

<tr>

<h1>Select the show(s) of your choice:</h1>

</strong></td>

</tr>

<tr>

</tr>

<tr>

<td colspan="2"><h3> <strong>Fall Shows</strong></h3></td>

</tr>

<tr>

October 25, 2009 – Westin Boston Waterfront, Boston, MA </font></td>

</tr>

<tr>

<h3><strong>Winter Shows</strong></h3>

</font></td>

</tr>

<tr>

<!--input type="checkbox" name="Event17" value="Yes" tabindex="35">

February 22, 2009 - Burlington Marriott, MA-->

</font></td>

January 10, 2010 - Seaport World Trade Center, Boston, MA <br>

January 17, 2010 - Wyndham Andover, MA<br>

January 24, 2010 - Granite Links Golf Club, Quincy, MA<br>

January 31, 2010 - Sheraton Ferncroft, Danvers, MA<br>

February 7, 2010 - Crowne Plaza, Natick, MA<br>

February 14, 2010 - Indian Pond Country Club, Kingston, MA<br>

February 21, 2010 - Burlington Marriott, MA</font> <font color="#666666"><br>

</font></td>

</tr>

</table>

i have multiple check boxes

mikesta707 · August 14, 2009

ok so it is an array. Again, WRAP YOUR CODE IN CODE TAGS.

now before you go into any of the foreach statements, write print_r($_POST['shows_Selected'])

and see what it says. You are making sure that you have something selected right?

ziegel · August 14, 2009

What do you have for the method in your form tag? Are you sure it's post and not get?

Oh I see...

You are making sure that you have something selected right?

This:

however if the dont check any of the boxes i get this error

Warning: Invalid argument supplied for foreach() in /home/va010021/public_html/newportweddingExpo/form_procces.php on line 73

randyw · August 14, 2009

ok so it is an array. Again, WRAP YOUR CODE IN CODE TAGS.

now before you go into any of the foreach statements, write print_r($_POST['shows_Selected'])

and see what it says. You are making sure that you have something selected right?

i have post

this is the error

Parse error: syntax error, unexpected T_VARIABLE in /home/va010021/public_html/newportweddingExpo/form_procces.php on line 72

this is line 72

$source .= $one_media."<br>";

ziegel · August 14, 2009

@randyw:

Change this:

$media_array = $_POST['shows_Selected'];
foreach ($media_array as $one_media) {
$source .= $one_media."<br>";
}

To this:

if (is_array($_POST['shows_Selected'])) {
   $media_array = $_POST['shows_Selected'];
   foreach ($media_array as $one_media) {
      $source .= $one_media."<br>";
   }
}

That will check first if they checked 1 of the boxes, if they didn't, it will skip over the foreach.

randyw · August 14, 2009

<?php

include_once("include/dbc.php");

// Verify that all form elements are completed

if (empty($_POST['first_name'])){

//die ("ERROR: Please add your First Name..");

header("Location: form.php?msg=ERROR: Please add your First Name..");

exit();

}

if (empty($_POST['last_name'])){

//die ("ERROR: Please add your Last Name..");

header("Location: form.php?msg=ERROR: Please add your Last Name..");

exit();

}

if (empty($_POST['email_address'])){

//die ("ERROR: Please add your Email..");

header("Location: form.php?msg=ERROR: Please add your Email..");

exit();

}

if (empty($_POST['mailling_address'])){

//die ("ERROR: Please add your Address..");

header("Location: form.php?msg=ERROR: Please add your Address..");

exit();

}

if (empty($_POST['city'])){

//die ("ERROR: Please add your City..");

header("Location: form.php?msg=ERROR: Please add your City..");

exit();

}

if (empty($_POST['zip'])){

//die ("ERROR: Please add your City..");

header("Location: form.php?msg=ERROR: Please add your City..");

exit();

}

if (empty($_POST['home_phone'])){

//die ("ERROR: Please add your Home Phone..");

header("Location: form.php?msg=ERROR: Please add your Home Phone..");

exit();

}

if (empty($_POST['work_phone'])){

//die ("ERROR: Please add your WorkPhone..");

header("Location: form.php?msg=ERROR: Please add your WorkPhone..");

exit();

}

if (is_array($_POST['shows_Selected'])) {

$media_array = $_POST['shows_Selected'];

foreach ($media_array as $one_media) {

$source .= $one_media."<br>";

}

// Now process the Addition Information array from POST

if (is_array($_POST['additional_Info'])) {

$addlInfo_array = $_POST['additional_Info'];

foreach ($addlInfo_array as $one_addl) {

$addlInfo .= $one_addl."<br>";

}

$additional_Info = substr($addlInfo, 0, -4);

// Now process the Addition Information array from POST

if (is_array($_POST['further_information'])) {

$furtherInfo_array = $_POST['further_information'];

foreach ($furtherInfo_array as $one_futher) {

$furtherInfo .= $one_further."<br>";

}

$further_information = substr($furtherInfo, 0, -4);

{

// update data in mysql database

$sql = "INSERT INTO newportweddingExpoForm

(shows_Selected, additional_Info, further_information, first_name, last_name, Role, email_address, home_phone, work_phone, city, state, zip, mailling_address, apt_number, wedding_month, wedding_day, wedding_year, Wedding_Tentative, wedding_Region, wedding_city, wedding_state, budget, find_aboutUs)VALUES('$shows_Selected', '$additional_Info', '$further_information', '$first_name', '$last_name', '$role', '$email_address', '$home_phone', '$work_phone', '$city', '$state', '$zip', '$mailling_address', '$apt_number', '$wedding_month', '$wedding_day', '$wedding_year', '$Wedding_Tentative', '$wedding_Region', '$wedding_city', '$wedding_state', '$budget', '$find_aboutUs')";

$result = @mysql_query($sql);

}

if($result){

//die ("Your form information has been successfully sent..");

header("Location: form.php?msg=Your form information has been successfully sent..");

exit();

}

else {

//die ("Iam sorry there was a problem, duplicate form name allready sent..");

header("Location: form.php?msg=Iam sorry there was a problem, duplicate form name allready sent..");

}

mysql_close();

?>

<?php

}

?>

<?php

}

?>

<?php

}

this is still not doing anything i thought it looked right

if (is_array($_POST['shows_Selected'])) {

$media_array = $_POST['shows_Selected'];

foreach ($media_array as $one_media) {

$source .= $one_media."<br>";

}

// Now process the Addition Information array from POST

if (is_array($_POST['additional_Info'])) {

$addlInfo_array = $_POST['additional_Info'];

foreach ($addlInfo_array as $one_addl) {

$addlInfo .= $one_addl."<br>";

}

$additional_Info = substr($addlInfo, 0, -4);

// Now process the Addition Information array from POST

if (is_array($_POST['further_information'])) {

$furtherInfo_array = $_POST['further_information'];

foreach ($furtherInfo_array as $one_futher) {

$furtherInfo .= $one_further."<br>";

}

$further_information = substr($furtherInfo, 0, -4);

{

// update data in mysql database

$sql = "INSERT INTO newportweddingExpoForm

(shows_Selected, additional_Info, further_information, first_name, last_name, Role, email_address, home_phone, work_phone, city, state, zip, mailling_address, apt_number, wedding_month, wedding_day, wedding_year, Wedding_Tentative, wedding_Region, wedding_city, wedding_state, budget, find_aboutUs)VALUES('$shows_Selected', '$additional_Info', '$further_information', '$first_name', '$last_name', '$role', '$email_address', '$home_phone', '$work_phone', '$city', '$state', '$zip', '$mailling_address', '$apt_number', '$wedding_month', '$wedding_day', '$wedding_year', '$Wedding_Tentative', '$wedding_Region', '$wedding_city', '$wedding_state', '$budget', '$find_aboutUs')";

$result = @mysql_query($sql);

}

ziegel · August 14, 2009

You left out a bracket after the foreach

if () {

foreach () {

}

You're missing that bracket

randyw · August 14, 2009

You left out a bracket after the foreach

if () {

foreach () {

}

}

You're missing that bracket

thank you you rock

randyw · August 14, 2009

You left out a bracket after the foreach

if () {

foreach () {

}

}

You're missing that bracket

thank you you rock

Sign In

php form

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation

Important Information