diff options
author | Rob Crittenden <rcritten@redhat.com> | 2007-10-31 10:08:16 -0400 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2007-10-31 10:08:16 -0400 |
commit | 303d5ebad9cea79c1e67e1be73ab629846b8121a (patch) | |
tree | aa59f523ad13aa6d84128a1428e6e6a99c18afd1 /ipa-server | |
parent | 1d6e88565c48ee0799ac1df58d4a56444388f0c8 (diff) | |
download | freeipa-303d5ebad9cea79c1e67e1be73ab629846b8121a.tar.gz freeipa-303d5ebad9cea79c1e67e1be73ab629846b8121a.tar.xz freeipa-303d5ebad9cea79c1e67e1be73ab629846b8121a.zip |
Have the GUI use memberOf() instead of looping through the member DNs
Fix a bug in the local transport version of memberOf()
Diffstat (limited to 'ipa-server')
-rw-r--r-- | ipa-server/ipa-gui/ipagui/subcontrollers/group.py | 30 |
1 files changed, 5 insertions, 25 deletions
diff --git a/ipa-server/ipa-gui/ipagui/subcontrollers/group.py b/ipa-server/ipa-gui/ipagui/subcontrollers/group.py index d4663f7ae..c3048f0de 100644 --- a/ipa-server/ipa-gui/ipagui/subcontrollers/group.py +++ b/ipa-server/ipa-gui/ipagui/subcontrollers/group.py @@ -183,20 +183,8 @@ class GroupController(IPAController): # # convert members to users, for easier manipulation on the page # - member_dns = [] - if group_dict.has_key('uniquemember'): - member_dns = group_dict.get('uniquemember') - # remove from dict - it's not needed for update - # and we are storing the members in a different form - del group_dict['uniquemember'] - if not(isinstance(member_dns,list) or isinstance(member_dns,tuple)): - member_dns = [member_dns] - - # TODO: convert this into an efficient (single) function call - members = map( - lambda dn: client.get_entry_by_dn(dn, ['dn', 'givenname', 'sn', - 'uid', 'cn']), - member_dns) + + members = client.memberOf(group.dn, ['dn', 'givenname', 'sn', 'uid', 'cn']) members.sort(self.sort_group_member) # Map users into an array of dicts, which can be serialized @@ -360,17 +348,9 @@ class GroupController(IPAController): # # convert members to users, for display on the page # - member_dns = [] - if group_dict.has_key('uniquemember'): - member_dns = group_dict.get('uniquemember') - if not(isinstance(member_dns,list) or isinstance(member_dns,tuple)): - member_dns = [member_dns] - - # TODO: convert this into an efficient (single) function call - members = map( - lambda dn: client.get_entry_by_dn(dn, ['dn', 'givenname', 'sn', - 'uid', 'cn']), - member_dns) + + members = client.memberOf(group.dn, ['dn', 'givenname', 'sn', 'uid', 'cn']) + members = members[1:] members.sort(self.sort_group_member) member_dicts = map(lambda member: member.toDict(), members) |