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

Bug #348 No updateUser() method for LiveUser_Admin_Perm_Common
Submitted: 2003-12-03 22:31 UTC
From: anon at anon dot com Assigned: arnaud
Status: Closed Package: LiveUser
PHP Version: 4.3.3 OS: ANY
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 : 3 + 20 = ?

 
 [2003-12-03 22:31 UTC] anon at anon dot com
Description: ------------ The class LiveUser_Admin_Perm_Common has methods to add/remove users to/from the table LIVEUSER_PERM_USERS, but no method to update a user entry. If a user's auth_user_id gets changed via LiveUser_Admin_Auth_Common::changeUser() or LiveUser_Admin_Auth_Container_DB::updateUser(), then the mapping between the perm_user_id and the auth_user_id will be lost since there is no way to update the auth_user_id in LIVEUSER_PERM_USERS. LiveUser_Admin_Perm_Common::updateUser() should be created to update the auth_user_id in LIVEUSER_PERM_USERS. Also LiveUser_Admin_Auth_Common::changeUser() should be renamed to LiveUser_Admin_Auth_Common::updateUser() for naming convention compliance.

Comments

 [2003-12-04 20:43 UTC] arnaud
some of these issues have been fixed in cvs. We have yet to come up with the updateUser method.
 [2003-12-04 20:59 UTC] arnaud
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. the method updateAuthUserId was added to [M]DB_Common to update the mapping. Since it is only in cvs it can be changed if necessary.