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

Bug #7262 PDO: Warnings on wrong login credentials
Submitted: 2006-03-30 13:47 UTC
From: marc dot jakubowski at gmail dot com Assigned: lsmith
Status: Closed Package: LiveUser (version 0.16.10)
PHP Version: 5.1.1 OS: XP
Roadmaps: (Not assigned)    
Subscription  


 [2006-03-30 13:47 UTC] marc dot jakubowski at gmail dot com (Marc Jakubowski)
Description: ------------ Again an issue with the PDO driver. When submitting $lu->login() with wrong credentials, I get 4 warnings: Notice: Undefined offset: 0 in D:\Programme\xampp\php\pear\LiveUser\Auth\PDO.php on line 279 Warning: array_key_exists() [function.array-key-exists]: The second argument should be either an array or an object in D:\Programme\xampp\php\pear\LiveUser\Auth\Common.php on line 383 Warning: array_key_exists() [function.array-key-exists]: The second argument should be either an array or an object in D:\Programme\xampp\php\pear\LiveUser\Auth\Common.php on line 459 Notice: Undefined offset: 0 in D:\Programme\xampp\php\pear\LiveUser\Perm\Storage\PDO.php on line 168 When the query that looks in the database if there is a user with the given password, the result is checked in LiveUser\Auth\PDO.php line 265 with if ($row === false) but the PDO::fetchAll() method always returns an array either with the rows of the result or an empty array if no rows were found. So because of $row never being === false, the line should be changed to if (empty($row))

Comments

 [2006-03-30 14:36 UTC] marc dot jakubowski at gmail dot com
oops bug belongs to LiveUser, not to LiveUser_Admin
 [2006-03-30 19:31 UTC] lsmith (Lukas Smith)
This bug has been fixed in CVS. 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.