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

Bug #3101 Wrong variable names in Auth/Container/File.php
Submitted: 2005-01-05 02:13 UTC
From: herojoker at nexgo dot de Assigned: mike
Status: Closed Package: Auth
PHP Version: Irrelevant OS: Irrelevant
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


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 : 36 - 21 = ?

 
 [2005-01-05 02:13 UTC] herojoker at nexgo dot de
Description: ------------ In Auth/Container/File.php on line 188 in the method changePassword wrong variable names are used. Therefore an error occurs when using this method. Reproduce code: --------------- The error causing method: function changePassword($username, $password) { $pw_obj = &$this->_load(); if (PEAR::isError($pw_obj)) { return false; } $res = $pw_obj->changePasswd($user, $pass); if (PEAR::isError($res)) { return false; } $res = $pw_obj->save(); if (PEAR::isError($res)) { return false; } return true; } Use the following code instead: function changePassword($username, $password) { $pw_obj = &$this->_load(); if (PEAR::isError($pw_obj)) { return false; } $res = $pw_obj->changePasswd($username, $password); if (PEAR::isError($res)) { return false; } $res = $pw_obj->save(); if (PEAR::isError($res)) { return false; } return true; } Expected result: ---------------- Thx for the Auth package! Hero Wanders

Comments

 [2005-01-17 12:55 UTC] mike
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.