PHP echoed into Javascript

[dannyboy]

Hi

I'm trying to echo some php into a piece of javascript that checks that the selectedIndex in a given select name is not 0, and gives an alert if this is so:
////
<script language="JavaScript"><!--
function validate(form) {
if (form2.pmethod.selectedIndex == 0) {
alert('You have not selected a postage method. Please select one.');
return false;
}
////
This all works fine, but when I try to echo some php into this from a foreach post loop, it doesn't work. From playing around with it, it is the squae brackets that cause the script not to run properly:

<?
foreach ($_POST['problem'] as $postName=>$postValue)
{
?>
if (form2.[<?echo''.$postValue.'';?>].selectedIndex == 0) {
alert('You have not selected a replacement type for <?echo''.$postValue.'';?>. Please select one.');
return false;
}
<?
}
?>

Does anyone know a way around the square brackets, or an alternative way to check the form, given that the name of the select option is echoed out through a foreach loop in the following format:

name="replacement_type['.$postValue.']"

Thanks.

[b1randon]

Hi

I'm trying to echo some php into a piece of javascript that checks that the selectedIndex in a given select name is not 0, and gives an alert if this is so:
////
<script language="JavaScript"><!--
function validate(form) {
if (form2.pmethod.selectedIndex == 0) {
alert('You have not selected a postage method. Please select one.');
return false;
}
////
This all works fine, but when I try to echo some php into this from a foreach post loop, it doesn't work. From playing around with it, it is the squae brackets that cause the script not to run properly:

<?
foreach ($_POST['problem'] as $postName=>$postValue)
{
?>
if (form2.[<?echo''.$postValue.'';?>].selectedIndex == 0) {
alert('You have not selected a replacement type for <?echo''.$postValue.'';?>. Please select one.');
return false;
}
<?
}
?>

Does anyone know a way around the square brackets, or an alternative way to check the form, given that the name of the select option is echoed out through a foreach loop in the following format:

name="replacement_type['.$postValue.']"

Thanks.

That's the way you do it in javascript. The real problem is your php. You should most this post to the php section where they can help you figure out why your php is not working properly.

[dannyboy]

Yeah, but if I change the name of the select name to

replacement_type'.$postValue.'

instead of:

replacement_type['.$postValue.']

and echo the javascript out as below:
*********
<?
foreach ($_POST['problem'] as $postName=>$postValue)
{
?>
if (form2.<?echo''.$postValue.'';?>.selectedIndex == 0) {
alert('You have not selected a replacement type for <?echo''.$postValue.'';?>. Please select one.');
return false;
}
<?
}
?>

**********
i.e. - removing the square brackets from the javascript, it works perfect. It seems that the square brackets in the name cause the javascript not to work properly.

ANy ideas? Is there a different way I can write the javascript, or php?

[dannyboy]

I've fixed it:

I used getElementById instead, so the php can work and the javascript can work:

<?
foreach ($_POST['problem'] as $postName=>$postValue)
{
?>
var replacement_type = document.getElementById ('replacement_type<?echo''.$postValue.'';?>');
if ((replacement_type).selectedIndex == 0)
{
alert('You have not selected a replacement type for <?echo''.$postValue.'';?>. Please select one.');
return false;
}
<?
}
?>

[AricC]

Danny is your problem fixed or should I move this to the PHP forum?