| 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-04 21:13 UTC | Modified: | 2005-01-17 07:55 UTC |
| From: | herojoker at nexgo dot de | Assigned: | mike |
| Status: | Closed | Package: | Auth |
| PHP Version: | Irrelevant | OS: | Irrelevant |
| Roadmaps: | (Not assigned) | ||
[2005-01-04 21: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 07: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.