Drop Down Boxes selecting selected option

[iceblox]

Hi Guys how would i make a PHP Query select the one that has been selected in a drop down box?

 

This is query i tried which didnt work

 

 

Can anyone please tell me what is wrong with it?

 

 

$query = "SELECT * FROM deals WHERE FreeGift <> 'Offer.FreeGift' AND FreeGift <> ' 'GROUP BY FreeGift ASC";

$result = mysql_query($query);

if (mysql_num_rows($result) > 
                            0)
{

while($row = mysql_fetch_row($result))     

if($row[4] == '$FreeGift') {
                $auto = '<option value="' . $row[4] . '">' . $row[4] . '</option>';
                }
                else {
                $auto = '<option value="' . $row[4] . '">' . $row[4] . '</option>';
	}

echo '' . $auto . '';

}
else
{
echo '';
}

[sdi126]

You are missing the keyword "selected"...both of your if statements are the same.

 

It should be

if($row[4] == '$FreeGift') {
       $auto = '<option selected="selected" value="' . $row[4] . '">' . $row[4] . '</option>';
}

 

[iceblox]

Ooops lol

 

The only issue is that when i have that if statement it only brings up one option when there should be about 20?

[sdi126]

Use mysql_fetch_array instead of row

 

while($row = mysql_fetch_array($result))

[iceblox]

Whats the difference? Sorry to be stupid

[sdi126]

It just returns a reference to one row. Using array you have access to all the rows returned by the sql statement.

 

Also it is better practice to refer to the variable by name and not by array number....ie $row[4] should be something like $row["freegift"] if your field name was called freegift of course in the table 

[iceblox]

Hmm that is still allowing returning one row..

[sdi126]

Echo out the number of rows returned by your sql statement and make sure it is more than one.

[iceblox]

OK thanks for the help ill look into it