Tell me more ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite

so as the title states, using the following code I have got it populating the dropbox with a single result from the query, that result being the latest added in the table.

here is my code:

<?php 
$query = "SELECT * FROM units_tb WHERE user_id='$userid'";
$result = mysql_query($query) or die (mysql_error());
while($row = mysql_fetch_assoc($result)){
$aa = "<option value='{$row['unit_id']}'>{$row['unit_code']}</option>";
}
?>
<select name="t_unit"><? echo $aa; ?></select>

The odd thing is, I use this same code for another field, and it works, populating the dropdown with all the results, however in this case it only fills in the last unit code in the table and not all of which are attached to the particular user id.

I would appreciate anyones thoughts :D

thanks

share|improve this question
1  
as bensiu points out below, you are not appending to your string, you're overwriting each time. But if you're doing this multiple places, why isn't it a function that you pass an array to and get the HTML back from? – Erik Mar 21 '11 at 13:34
Do a sanity check... how many rows are being returned by the query? Run the exact query against mysql directly -- do you get the results you expect? – ybakos Mar 21 '11 at 13:36
Is the user_id attribute of the units_tb table an int or a string? – ybakos Mar 21 '11 at 13:37

2 Answers

up vote 4 down vote accepted 
$aa .= "<option value='{$row['unit_id']}'>{$row['unit_code']}</option>";

add . before = and initiate $aa = ''; before while loop

share|improve this answer
Good eye... a classic syntax mistake. – ybakos Mar 21 '11 at 13:39
apologies, i do now feel like a fool. it was just the . thanks though! – buymypies Mar 21 '11 at 13:41
up vote 0 down vote 
<?php 
$query = "SELECT * FROM units_tb WHERE user_id='$userid'";
$result = mysql_query($query) or die (mysql_error());
$options = "";
while($row = mysql_fetch_assoc($result)){
    $options .= "<option value='{$row['unit_id']}'>{$row['unit_code']}</option>";
}
?>
<select name="t_unit"><? echo $options; ?></select>

should work. You forgot a . in your while loop

share|improve this answer
and $aa has not been initiated – bensiu Mar 21 '11 at 13:36

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.