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

Bug #1560 LiveUser\Admin\Admin.php, function:updateUser, Line 472
Submitted: 2004-06-04 16:24 UTC Modified: 2004-06-08 15:42 UTC
From: waxdipped at yahoo dot co dot uk Assigned: lsmith
Status: Closed Package: LiveUser
PHP Version: Irrelevant OS: Linux (SuSE 9.0)
Roadmaps: (Not assigned)    
Subscription  


 [2004-06-04 16:24 UTC] waxdipped at yahoo dot co dot uk
Description: ------------ auth_user_id being set to nothing in table liveuser_perm_users after calling updateUser LiveUser\Admin\Admin.php, function:updateUser, Line 472 from current CVS version: $authId has not been initialised or set. $perm = $this->perm->updateUser($permId, $authId, $this->authContainerName, $type); should almost certainly be $perm = $this->perm->updateUser($permId, $authData['auth_user_id'], $this->authContainerName, $type); Reproduce code: --------------- just call updateUser Expected result: ---------------- select * from liveuser_perm_users; +--------------+--------------+-----------+---------------------+ | perm_user_id | auth_user_id | perm_type | auth_container_name | +--------------+--------------+-----------+---------------------+ | 17 | 40 | 1 | 0 | +--------------+--------------+-----------+---------------------+ Actual result: -------------- select * from liveuser_perm_users; +--------------+--------------+-----------+---------------------+ | perm_user_id | auth_user_id | perm_type | auth_container_name | +--------------+--------------+-----------+---------------------+ | 17 | | 1 | 0 | +--------------+--------------+-----------+---------------------+

Comments

 [2004-06-04 16:35 UTC] smith at backendmedia dot com
I have a fix pending for this issue.
 [2004-06-08 15:42 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.