Ok, code is mostly ok, but you made a few mistakes, which probably comeup because you're assuming an old server-config.
Your code assumes register_globals is on, in other words, you access all server and POST variables by their direct name, instead of fetching them from their respective arrays, however, few servers allow that nowadays, as it is a serious security flaw, so you should instead get them from the global arrays.
The variables you try to use are:
$PHP_SELF
$chooseCountry
they should be:
$_SERVER['PHP_SELF'] (you also forgot to echo this one)
$_POST['chooseCountry']
next, there are a few minor flaws in your HTML.
to make an <option> selected, the proper code is:
<option value="value" selected="selected">
instead of
<option value="value selected">
Finally, unless you've defined a javascript function submitpage() yourself, the proper code would be submit(this), to submit the form on the onchange.
The correct code would then read (tested this):
PHP Code:
?>
<form name="country" action="<? echo $_SERVER['PHP_SELF']; ?>" method="post">
<select name="chooseCountry" onChange="submit(this)">
<option value='0'>Choose country</option>
<?
$querydrop = "select * from countryes";
$result = mysql_query($querydrop);
while($row = mysql_fetch_array($result))
{
if($row['id']==$_POST['chooseCountry'])
{
print "<option value=\"".$row['id']."\" selected=\"selected\">".$row['country_name']."</option>\n";
}
else
{
print "<option value=\"".$row['id']."\">".$row['country_name']."</option>\n";
}
}
?>
</select>
</form>
Hope this helps.
PS: the correct plural of country is countries, not countryes
Dev Articles Community Forums Sponsor:
August 12th, 2004, 05:11 PM
Del.icio.us
Digg
Google
Spurl
Blink
Furl
Simpy
Y! MyWeb
Linear Mode
