Why is this not showing correctly?

[Navajo]

Why is it all messed up, I want two drop down menu's which show identical things.

 

Why is the first one fine and the second is messed up?

 

<FORM ACTION="calculate.php" METHOD="POST" onSubmit="return dropdown(this.display)">


<SELECT NAME="display">
<OPTION VALUE="">Choose a Product
<?php
  
while ($row = mysql_fetch_array($result)) {
extract($row);
$productID = $row['vchar_product_id'];
$assesmentLimit = $row['vchar_assessment_limit'];

?>
<OPTION VALUE=<?php "$productID, $assesmentLimit"?> > <?php echo "$productID $assesmentLimit" ?>
<?php
//}
?>
</SELECT>


<SELECT NAME="display2">
<OPTION VALUE="">Choose a Product2
<?php


?>
<OPTION VALUE=<?php "$productID, $assesmentLimit"?> > <?php echo "$productID $assesmentLimit" ?>
</SELECT>

<?php
}
?>
<br /><br /><br /><br />
<INPUT TYPE=SUBMIT VALUE="Calculate">
</FORM>

[Mchl]

Both are messed up:

<OPTION VALUE=<?php "$productID, $assesmentLimit"?> > <?php echo "$productID $assesmentLimit" ?>

 

there should be 'echo' in the first <?php tag

[Navajo]

thanks, but that makes no difference to the result?

[Mchl]

Still, this was an error. What is the HTML produced by this code?

[premiso]

Along side that here is a corrected version, you need to use " around values in html or you get errors:

 

<OPTION VALUE="<?php echo "$productID, $assesmentLimit"; ?>"> <?php echo "$productID $assesmentLimit"; ?></OPTION>

 

And it is also good practice to have the </option> Not required, just good practice.

[Mchl]

And if you're about to populate two drop downs, you will need to use two separate loops. The way you do it now, I'm surprised it shows even the first one correctly.