[RESOLVED] checkbox array issue...

fpdan

Hello all! I'm trying to pass a checkbox value and item name through to a results page using an array.

My html form says this:

<input name="hullcolor[]" type="checkbox" value="575" onclick="addUp()">

My php results page says this:

//Hull Color Array
$hullcolors_array = array("575" => "<strong>Hull Color</strong>  ",);
$hullcolor_value = $_POST['hullcolor'];
//Hull Color Output
echo "<p>Hull Color: " . $hullcolors_array[$hullcolor_value]; 
echo ".... $<b>".$_POST["hullcolor"]."</b>.";

When I the results page I don't see the hull color. Anyone got any ideas?

rsmith

echo ".... $<b>".$_POST["hullcolor"]."</b>.";

should be something like this:

 foreach($hullcolor_value AS $hullcolor){
            echo $hullcolor."<br /> ";
          }

fpdan

Thanks for the quick reply rsmith. It outputs the hull color now but gives me this also:

Warning: Illegal offset type in /hsphere/local/home/captaint/swiftda.com/buildaboat/SENDMAIL_HTML.php on line 73


Hull Color: 575

This is my current code:

//Hull Color Array
$hullcolors_array = array("575" => "<strong>Hull Color</strong>  ",);
$hullcolor_value = $_POST['hullcolor'];
//Hull Color Output
echo "<p>Hull Color: " . $hullcolors_array[$hullcolor_value]; 
foreach($hullcolor_value AS $hullcolor){ 
            echo $hullcolor."<br /> "; 
          }

Line 73 is:

echo "<p>Hull Color: " . $hullcolors_array[$hullcolor_value];

Any ideas on that?

rsmith

I had a similar issue a couple months back.

echo "<p>Hull Color: " .implode(', ', $hullcolor_value)."";

fpdan

This is getting interesting. Now when the check box is selected, it seems to work except on the results page is displays the value twice:

Hull Color: 575575

When the box remains unchecked, I now get these:

Warning: implode(): Bad arguments. in /hsphere/local/home/captaint/swiftda.com/buildaboat/SENDMAIL_HTML.php on line 73


Hull Color: 
Warning: Invalid argument supplied for foreach() in /hsphere/local/home/captaint/swiftda.com/buildaboat/SENDMAIL_HTML.php on line 74

Line 73 and 74 are:

echo "<p>Hull Color: " .implode(', ', $hullcolor_value).""; 
foreach($hullcolor_value AS $hullcolor){ 
            echo $hullcolor."<br /> "; 
          }

I'm really lost at this point...

rsmith

If $hullcolor_value is blank then set the variable array to a blank, that way it'll just echo a blank character. So put this in right after you pull the $hullcolor_value variable (or wherever you want, just make sure you define it somewhere before the mail part of your script:

if(empty($hullcolor_value)){
     $hullcolor_value = array(" ");
}

fpdan

okay, I think I put that in the right spot but it seems to have no effect on it. When I output, I still get the following when the checkbox is checked:

Warning: Illegal offset type in /hsphere/local/home/captaint/swiftda.com/buildaboat/SENDMAIL_HTML.php on line 73

And this when it is not:

Warning: Invalid argument supplied for foreach() in /hsphere/local/home/captaint/swiftda.com/buildaboat/SENDMAIL_HTML.php on line 74

Hull Color: 575

My HTML code looks as such:

<input name="hullcolor[]" type="checkbox" value="575" onclick="addUp()">

And my php code looks like this:

<html>
<head>
<title>Sending the Simple Feedback Form – HTML Version</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"><style type="text/css">
<!--
body {
	background-color: #FFFFFF;
}
-->
</style></head>
<body>
   <div align="center">
     <table width="728" border="1" cellpadding="0" cellspacing="0" bordercolor="#2A0000">
    <tr>
      <td width="724"><div align="center"><strong>Thanks for using Canyon Bay's Boat Configurator!</strong> </div></td>
      </tr>
    <tr>
      <td><?php
echo "<p><b>".$_POST["First_Name"]."</b>, here is your configured Canyon Bay!</p>";
echo "<p>We have sent this info to your email address at: <b>".$_POST["EmailFrom"]."</b>.</p>";
echo "<p>You configured a <strong>Canyon Bay 2475</strong></p>";


//Start Engine Type Output Arrays
//E-Tech Engine Array
$engines_array = array("12946" => "<strong>E 150DPXSC</strong>  ",
  "14348" => "<strong>E 175DPXSC</strong>  ",
  "15558" => "<strong>E 200DPXSC</strong>  ",
  "16914" => "<strong>E 200DHXSC</strong>  ",
  "17276" => "<strong>E 225DPXSC</strong>  ",
  "18655" => "<strong>E 250DPXSC</strong>  ",
  "12583" => "<strong>150XL OptiMax</strong>  ", 
  "14616" => "<strong>175XL OptiMax</strong>  ",
  "16016" => "<strong>200XL OptiMax</strong>  ", 
  "17146" => "<strong>225 XL OptiMax</strong>  ",   

  "19094" => "<strong>225 Pro XS XL OptiMax</strong>  ",
  "20438" => "<strong>250 Pro XS XL OptiMax</strong>  ",  

  "14905" => "<strong>F150TXR</strong>  ",
  "19813" => "<strong>F200TXR</strong>  ",
  "21407" => "<strong>F225TXR</strong>  ",
  "21866" => "<strong>F250TXR</strong>  ",); 
$engine_value = $_POST['Engine'];
?>
        <br>      <br>        <?php
//Call Engine Output
echo "Engine Type: " . $engines_array[$engine_value]; 
echo ".... $<b>".$_POST["Engine"]."</b>.";

//Engine Rigging Array
$riggings_array = array("1117" => "<strong>E-Tech Std Pre-Rig</strong>  ",
 "1848" => "<strong>E-Tech I-Command Rigging</strong>  ", 
 "2513" => "<strong>E-Tech I-Command Rigging Dlx.</strong>  ",
 "1200" => "<strong>Mercury Std Pre-Rig</strong>  ",
 "1800" => "<strong>Mercury Smartcraft Rigging</strong>  ",
 "2300" => "<strong>Mercury Smartcraft Rigging Dlx.</strong>  ",
 "1205" => "<strong>Yamaha Std Pre-Rig</strong>  ",
 "1805" => "<strong>Yamaha Std Digital Rigging</strong>  ",
 "2305" => "<strong>Yamaha Digital Rigging Dlx.</strong>  ",);
$rigging_value = $_POST['Rigging'];


//Call Rigging Output
echo "<p>Rigging Option: " . $riggings_array[$rigging_value]; 
echo ".... $<b>".$_POST["Rigging"]."</b>.";

//Hull Color Array
$hullcolors_array = array("575" => "<strong>Hull Color</strong>  ",);
$hullcolor_value = $_POST['hullcolor'];
} 
//Hull Color Output
echo "<p>Hull Color: " .implode(', ', $hullcolor_value).""; 
foreach($hullcolor_value AS $hullcolor){ 
            echo $hullcolor."<br /> "; 
          } 
if(empty($hullcolor_value)){ 
     $hullcolor_value = array(" "); 
}



//Call Total Price Output
echo "<p>Your Total Amount is $<b>".$_POST["total"]."</b>.</p>";





//start building the mail string
$msg = "<p><strong>Name:</strong>    ".$_POST["First_Name"]."</p>";
$msg .= "<p><strong>E-Mail:</strong>  ".$_POST["EmailFrom"]."</p>";
$msg .= "<p><strong>Engine:</strong>  ".$_POST["Engine"]."</p>";

//set up the mail
$recipient = "";
$subject = "Canyon Bay Configuration";
$mailheaders = "MIME-Version: 1.0\r\n";
$mailheaders .= "Content-type: text/html; charset=ISO-8859-1\r\n";
$mailheaders .= "From: ".$_POST["EmailFrom"]." \n";
$mailheaders .= "Reply-To: ".$_POST["danny@swiftda.com"];
//send the mail
mail($recipient, $subject, $msg, $mailheaders);
?>        <br>        </td>
      </tr>
        </table>
     <br>
   </div>
</body>
</html>

Here is line 71-77 as I think this is where the issue lies:

echo "<p>Hull Color: " .implode(', ', $hullcolor_value).""; 
foreach($hullcolor_value AS $hullcolor){ 
            echo $hullcolor."<br /> "; 
          } 
if(empty($hullcolor_value)){ 
     $hullcolor_value = array(" "); 
}

thanks again for all your assistance thus far!

rsmith

You need to put the if(empty) ABOVE the echo and ABOVE the foreach(), like this:

if(empty($hullcolor_value)){
     $hullcolor_value = array(" ");
}

echo "<p>Hull Color: " .implode(', ', $hullcolor_value)."";

foreach($hullcolor_value AS $hullcolor){
            echo $hullcolor."<br /> ";
          }

fpdan

hmmm....I tried that but with no luck. Here's my code:

//Hull Color Output
if(empty($hullcolor_value)){ 
     $hullcolor_value = array(" "); 
} 

echo "<p>Hull Color: " .implode(', ', $hullcolor_value).""; 

foreach($hullcolor_value AS $hullcolor){ 
            echo $hullcolor."<br /> "; 
          }

Sill same errors as before

rsmith

take out the echo part, that part was only for listing the hullcolor_values from the array in an e-mail. Just use the foreach()

fpdan

Okay, so with this code,

66    //Hull Color Array
67     $hullcolors_array = array("575" => "<strong>Hull Color</strong>  ",);
68     $hullcolor_value = $_POST['hullcolor'];
69  } 
70      //Hull Color Output
71      if(empty($hullcolor_value)){ 
72     $hullcolor_value = array(" "); 
73  } 
74
75      echo "<p>Hull Color: " .implode(', ', $hullcolor_value).""; 

76      

77      foreach($hullcolor_value AS $hullcolor){ 
78            echo $hullcolor."<br /> "; 
79   }

I'm now getting:

Parse error: parse error, unexpected '}' in /hsphere/local/home/captaint/swiftda.com/buildaboat/SENDMAIL_HTML.php on line 69

fpdan

okay, I took out the "}"

and now it's working except....

it displays the hull color value twice...????

Here's a working example link:

http://www.swiftda.com/buildaboat/buildaboat.htm

rsmith

Take out line 75.

fpdan

Thanks so much rsmith. I'm really enjoying php. I'm really green behing the ears at this as I'm a front end designer.

With my radio selections, it only echos the selected radios. Is that possible to do at all with checkboxes?

Also, do you have any suggestions on any good php books. I recently picked up the SAMS Teach Yourself PHP & Mysql. Thanks again for all your assistance.

fpdan

Actually, it does only display when it is checked but it only displays the value. Can it display the "Hull Color" as well as the value like my radios?

rsmith

Yeah I tried 5 different books and always ended up giving up on PHP. But then, for basic understanding, I got this book: [URL="http://www.amazon.com/PHP-blueprint-creating-server-side-Blueprint/dp/0764583328"[/URL] and it helped make it click, now I'm working on refining my weak areas and moving into advanced stuff.

As for your question I'm not understanding exactly what you want. If you want it to display the value of a checkbox, the value is something you set like this:

<input type="checkbox" name="colors[]" value="red">You've Chosen Red
<input type="checkbox" name="colors[]" value="blue">You've Chosen Blue

As you can see the value is defined in the properties of the input attribute but the text appearing on the page can be more descriptive. I hope this is what you meant.

fpdan

Thanks for the heads up on the book, I'll give it a shot. I'm really trying to grasp the over basic concept of php and it's proving to be a difficult task. It seems there are a lot of ways to do many things.

As for my question,

on the html (form) page, this infact does display the value.

<input name="hullcolor[]" type="checkbox" value="575" onclick="addUp()">Hull Color....$575

Now when it is submited to the processing page (which is also my "results" page) it only displays the value "575". I was wondering how I could display:

Hull Color...$575

if it was selected and if not, it will not display.

Due to the help from stolzboy, for my radio selections, we:

on the processing page that "maps" to the engine type based on the value send through Engines... so then you could echo it out

so... something like the following, it's untested, but they theory is there.

$engines_array = array("12946" => "E 150DPXSC", "14348" => "E 175DPXSC"); 

$engine_value = $_POST['Engines']; 

echo "Engine Type: " .$engines_array[$engine_value];

rsmith

You need to set the value of the html form entity to be exactly what you want it to display when the form is submitted. 575 is only displayed because that's what you have set as the value of the input item. If you want it to say Hull Color 575 then put that into your value, like this:

<input name="hullcolor[]" type="checkbox" value="Hull Color.... 575" onclick="addUp()">Hull Color....$575

fpdan

On my form I have total box which updates from the value. If I put anything other than numbers on the value it will not calculate and update the total.

rsmith

Ah, then you might be out of luck.