Re: Comments on bugs marked bogus

Hi, Joshua Eichorn wrote:

Being in outsider in this just seems like a stupid argument to be having. PHP5 is case sensitive, so we should start using case when calling is_a, etc to be forward compatible. If thats too big of an issue you could just say that quickforms isn't supported under PHP5. Anyhow its a less then one line fix that only invloves changing case.

You are missing the point: QuickForm *is* PHP5 compatible. We are talking about incompatibility with third-party code, I am insisting that this code should be fixed since it may be incompatible with more PEAR packages, my opponents insist that QuickForm should be "fixed", but this leads to "fixing" of all these other packages.

Also the initial comment of "bullshit" makes me think there is some sort of hate for this user going on, and thats really not the type of thing that should be happening in PEAR.

The initial comment of "bullshit" is targeted only at the initial report saying the following: ========== The code in RuleRegistry.php at line 217 reads

       if (is_a($element, 'html_quickform_group')) {

it should read

       if (is_a($element, 'HTML_QuickForm_group')) {

otherwise it will always evaulate to false. ========== which is exactly that: bullshit.