Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.1.2

Bug #895 PEAR QA: improvement for get_class()-usage
Submitted: 2004-02-26 14:18 UTC
From: pear-qa at lists dot php dot net Assigned: jmcastagnetto
Status: Closed Package: Science_Chemistry
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes. If this is not your bug, you can add a comment by following this link. If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
2004-04-10 02:18 UTC
Package:
Bug Type:
Summary:
From: pear-qa at lists dot php dot net
New email:
PHP Version: Package Version: OS:

 

 [2004-02-26 14:18 UTC] pear-qa at lists dot php dot net
Description: ------------ ## from the PEAR QA team ## ## quality-assurance related ## Please note that if you don't react to this issue within 4 weeks the PEAR QA-team will take care of it automatically. This is not "rude" but necessary for QA. If this is a "false positive" and your package is not affected please apologize. This package uses get_class()-calls without care for case-sensitivity of the returned classnames. In PHP 4.x the default for getclass() was that lowercase classnames were returned. However, due to extended DOM functionality, PHP 5.x returns classnames with correct upper-and-lowercase. If you intend to run your package under both PHP4 and PHP5 please consider using case-independent comparisons or apply case-fixes. Found in: FILE: '/cvs/php/pear/Science_Chemistry/Chemistry/Atom.php' : /cvs/php/pear/Science_Chemistry/Chemistry/Atom.php(136): (strtolower(get_class($obj)) == strtolower("Science_Chemistry_Atom") ---------------------------------------- FILE: '/cvs/php/pear/Science_Chemistry/Chemistry/Coordinates.php' : /cvs/php/pear/Science_Chemistry/Chemistry/Coordinates.php(81): (strtolower(get_class($obj)) == strtolower("Science_Chemistry_Coordinates") ---------------------------------------- FILE: '/cvs/php/pear/Science_Chemistry/Chemistry/Element.php' : /cvs/php/pear/Science_Chemistry/Chemistry/Element.php(120): (get_class($obj) == "Science_Chemistry_Element" || ---------------------------------------- FILE: '/cvs/php/pear/Science_Chemistry/Chemistry/Macromolecule.php' : /cvs/php/pear/Science_Chemistry/Chemistry/Macromolecule.php(137): (strtolower(get_class($obj)) == strtolower("Science_Chemistry_Macromolecule") || ---------------------------------------- FILE: '/cvs/php/pear/Science_Chemistry/Chemistry/Molecule.php' : /cvs/php/pear/Science_Chemistry/Chemistry/Molecule.php(180): (strtolower(get_class($obj)) == strtolower("Science_Chemistry_Molecule") || Reproduce code: --------------- Affected are constructs like: if (get_class($foo) == 'foo') { ... } Possible solutions for correct class-checks: strtolower(get_class($foo)) == 'foo' is_a($foo, 'foo') !strcasecmp(get_class($foo), 'foo')

Comments

 [2004-04-10 02:18 UTC] jmcastagnetto
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.