Hi,

There are a couple of problems with your code. The first is that you have global variables enabled, which makes things easier, but also makes things less secure. You should definitely disable them in your php.ini file if possible.

Also, when you put the square brackets after gmev (gmev[]), you turn $_POST['gmev'] into an array. Yet your code just checks to see if the $_POST['gmev'] variable has any value at all, which of course, it does.


> if ($gmev) {
> $checked = ' checked="checked"';
> } else {
> $checked = '';
> }

Therefore, when you go to present the form, $checked always has the string from the first case in the if statement, since you're not specifying which part of the $gmev array you really care about.

My suggestion to everyone dealing with these form problems, which are a general pain... Use Lampshade. It's so much easier! You can do this all in three lines of code: two to set up your choices for the form element, and one to draw the element itself.

http://www.thinkcomputer.com/software/lampshade

I'd be glad to help you with it if you have any questions.

Best of luck,

Aaron

Aaron Greenspan
President & CEO
Think Computer Corporation

http://www.thinkcomputer.com

--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to