Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.0.5
Bug #17669 typo: key_exists instead of array_key_exists
Submitted: 2010-08-08 19:28 UTC
From: olleolleolle Assigned: olleolleolle
Status: Closed Package: Net_Nmap (version SVN)
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 19 + 1 = ?

 
 [2010-08-08 19:28 UTC] olleolleolle (Olle Jonsson)
Description: ------------ While looking at a linting problem, I found an anomalous function call to "key_exists". The attached patch makes the __get and __set methods public, as they should be, and use array_key_exists on the arrays. I was inspired to do this via the report page http://test.pear.php.net/unit-test-results/deprecated.txt

Comments

 [2010-08-08 19:30 UTC] olleolleolle (Olle Jonsson)
Added patch magic_methods_public.diff.
 [2010-08-08 19:32 UTC] olleolleolle (Olle Jonsson)
Added patch nmap_key_exists_typo.
 [2010-08-08 19:33 UTC] olleolleolle (Olle Jonsson)
Added patches separately, according to the instruction. Thank science for instructions.
 [2010-08-09 05:43 UTC] doconnor (Daniel O'Connor)
-Status: Open +Status: Closed -Assigned To: +Assigned To: olleolleolle
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. Cheers for the patches!