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

Bug #3874 removeUserFromGroup() Removes all assignments
Submitted: 2005-03-18 18:26 UTC
From: benpjohnson at gmail dot com Assigned: lsmith
Status: Closed Package: LiveUser_Admin
PHP Version: 5.0.3 OS: Windows XP
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 : 30 + 49 = ?

 
 [2005-03-18 18:26 UTC] benpjohnson at gmail dot com
Description: ------------ package version: 0.2.0 The removeUserFromGroup() function does not seem to take the group_id filter into account and all groups assignments for that the user are deleted. Reproduce code: --------------- $admin->perm->removeUserFromGroup(array('group_id' => $group_id,'perm_user_id' => $perm_user_id)); Expected result: ---------------- User with id $perm_user_id is removed from group: $group_id Actual result: -------------- All group assignments for that user are removed

Comments

 [2005-03-23 18:48 UTC] smith at backendmedia dot com
Yes we have a conceptual problem here. $filters = $this->_makeRemoveFilter($filters, 'perm_user_id', 'getGroups'); returns only the 'perm_user_id' returned by passing the filters to getGroups() ... I think we probably need to remove the entire _makeRemoveFilter() business for all cases where we are removing people from relationship tables and the user may simply only set filters on the given relation..
 [2005-03-24 16:33 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.