[dev] Form.php and enum fields

[Marko]

sorry chuck i'm not arguing to get rid of this, but rather to clear up my ideas
about how horde_forms work and improve both midas and horde_forms where/if i can.

yes, if this is the reason behind isset() then there are holes in other field
types. i'll try to get a patch in for them then.

i'll see what i can do about a patch to allow some form of "label" within the
select field. i think it would be useful, what do you reckon?
doing a custom form renderer just to get one "Choose one:" label is too much.

marko


Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Marko <marko at oblo.com>:
> 
> > but it's the reason why the isset is being checked that i don't
> > understand. if i follow the logic right, the only situation where the !
> > isset test will return true and hence trigger the "field required"
> > message would be if the $value submitted is not in the <select> values
> > list - trapping for a manually altered GET submission?
> 
> Or POST. Would you rather leave that hole open?
> 
> > what's more, monthyear and monthdayyear types don't do this check...
> 
> If it could be exploited there (instead of just getting a different date),
> then it should be. Omission elsewhere isn't an argument.
> 
> -chuck
> 
> --
> Charles Hagenbuch, <chuck at horde.org>
> "Block Island Ferry - Fastest Route to Downtown"
> 
> --
> Horde developers mailing list
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe at lists.horde.org
>