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

Request #6517 $lu->getproperty('allgroups')
Submitted: 2006-01-18 15:27 UTC
From: murphy at spreekonzept dot de Assigned: lsmith
Status: Closed Package: LiveUser
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  


 [2006-01-18 15:27 UTC] murphy at spreekonzept dot de
Description: ------------ Currently, the $lu->getproperty('group_ids') delivers a) MEDIUM: all group-id's of groups the user is in (because subgroups are not available in this Schema) b) COMPLEX: all group-id's of groups and subgroup-id's the user is in I have written an application with the Complex-Schema and came to a point where it would've been helpful to get all group-id's of groups the user is in, but no subgroups - because the system in my case only allowed one group per user (with as many subgroups you want) and I wanted a quick way to get the group-id of the user. ATM this is only possible by instantiating a LiveUser_Admin object. This is one of some cases I stumbled upon, where the seperation of Auth/Perm/Admin resulted in a dirty architectural decision, where it could've been so easy. This is just a suggestion and based on a (german) discussion here: http://www.pear-forum.de/ftopic919.html Greetings, Thomas

Comments

 [2006-01-18 15:33 UTC] murphy at spreekonzept dot de
Well seems I actually forgot the suggestion itself: Seperate $lu->getproperty('group_ids') into $lu->getproperty('group_ids') => get all groups, no subgroups $lu->getproperty('all_group_ids') => get all groups with subgroups
 [2006-01-23 12:39 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send
MANUAL CONFIRMATION IS NOT POSSIBLE.  Write a message to pear-dev@lists.php.net
to request the confirmation link.  All bugs/comments/patches associated with this

email address will be deleted within 48 hours if the account request is not confirmed!