Home » Authentication » LiveUser » Bug #1672
'auth_user_id' naming inconsistency
Details
| Submitted | 2004-06-17 18:04 UTC |
|---|---|
| From | xav at spotk dot net |
| Assigned | lsmith |
| Status | Closed |
| Package | LiveUser |
| PHP Version | 4.3.7 |
| OS | linux debian sarge |
| Roadmaps | (Not assigned) |
Comments
[2004-06-17 18:04 UTC] xav at spotk dot net
Description:
------------
According to the different config arrays founds in example3,4 and 5
AuthTableCols array format is:
'authTableCols' => array(
'required' => array(
'user_id' => array('type' => '', 'name' => 'auth_user_id'),
'handle' => array('type' => '', 'name' => 'handle'),
'passwd' => array('type' => '', 'name' => 'passwd')
),
'optional' => array(
'lastlogin' => array('type' => '', 'name' => 'lastlogin'),
'is_active' => array('type' => '', 'name' => 'is_active')
),
'custom' => array()
)
In configs the auth_user_id is called: ['required']['user_id']
but, Admin::getUser doesn't use the ['required'] key to find ['user_id'] field, so generated SQL doesn't contain any reference to the auth_user_id field.
in the same time, there a problem in Auth containers
for each container the getUsers() method search for
$this->authTableCols['required']['auth_user_id']['name']
[2004-06-17 18:12 UTC] smith at backendmedia dot com
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.