[SOLVED] PHP Form Help

[NathanS]

Hi guys,

 

I've created a web based form which emails the results. However, I've now got a multiple select form, and I'm trying to make the email echo the output of that (ie, however many have been selected)

 

$msg .= "<b>Modified Yes Details</b>: "foreach($_POST['MODIFIEDYES'] as $name) { echo $name; }" "\n";

 

however, PHP returns "unexpected T_FOREACH". Any way I can achieve this?

 

Thanks!

[rajivgonsalves]

should be

 

$msg .= "Modified Yes Details: ".implode("\n",$_POST['MODIFIEDYES'])."\n";

[NathanS]

thanks for your fast reply, much appreciated!

 

i get:

 

Warning: implode() [function.implode]: Bad arguments. in C:\Websites\www.hertsinsurance.com\epasend.php on line 225

 

when I try this now - any ideas?

[rajivgonsalves]

is $_POST['MODIFIEDYES'] an array ?

[NathanS]

yes it is, my html code is:

 

<select name="MODIFIEDYES[]" size="10" multiple>
<?php
while($noticia = mysql_fetch_array($vehiclemod)) { 
echo  "<option value='$noticia[VEHICLE_MOD_DEBUG]'>$noticia[VEHICLE_MOD_DEBUG]";
}
?>
</select>

 

thanks

[chigley]

Well your HTML is invalid for a start. You need a </option> closing tag.

[NathanS]

Really? For multiple I've been informed no closing tag is required.

[redarrow]

and here a quick example of implode and explode

 

<?php

$msg = "Modified Yes Details:";

$x=explode(' ' ,$msg);

$x=implode('',$x);

echo $x;

?>

[redarrow]

you always need a closing tag?

 

ok.....

[adam291086]

You always need closing tag otherwise you are relaying on the browser to predict what you are doing. Some older browser wont do this and therefore mess up your code. It's best practice to end all html tags.

[NathanS]

ok, I was just getting that from http://www.faqts.com/knowledge_base/view.phtml/aid/578 where they use no closing tags on the options. I shall add one in.

[redarrow]

see what we mean by copy and past programmers and ones that study ha....

[NathanS]

Indeed. Am doing my best!