[dev] Re: [cvs] commit: framework/Util Util.php

[Michael M Slusarz]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Michael M Slusarz <slusarz at bigworm.colorado.edu>:
>
>> Huh.  Any reason why you are checking for an empty parameter then? I 
>> personally can't think of a reason why an empty parameter would be 
>> useful (i.e. checking whether a parameter exists vs. checking if a 
>> parameter is empty _should_ produce the same results).
>
> No. There are plenty of cases where we want to consider 0 valid, but an empty
> parameter/no parameter invalid. Horde_Form's validation checks for this, for
> instance. And the distinction *is* important. I can get into the details with
> select lists and valid input sets if you really need me to.

OK.  I'll trust other's judgment on this one and revert, but I still think it
looks unprofessional and like the software is broken when I see a hanging
'value2=' parameter out there.  But I'm anal so I should be ignored when I
start  talking about this kind of stuff.

michael

______________________________________________
Michael Slusarz [slusarz at bigworm.colorado.edu]
The University of Colorado at Boulder