From 944dc68f9fe2c4a373e4be24dbff3d2a82fd96cd Mon Sep 17 00:00:00 2001 From: Marc Fiuczynski Date: Thu, 16 Jul 2009 20:09:11 +0000 Subject: [PATCH] to get 'roles' from Persons() one must also use 'role_ids' in the return fields --- PLC/Methods/GetSlivers.py | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/PLC/Methods/GetSlivers.py b/PLC/Methods/GetSlivers.py index ce43f72b..b8743ccf 100644 --- a/PLC/Methods/GetSlivers.py +++ b/PLC/Methods/GetSlivers.py @@ -250,21 +250,9 @@ class v43GetSlivers(Method): sites = Sites(self.api, site_filter, site_return_filter) site = sites[0] person_filter = {'person_id':site['person_ids'],'enabled':True} - person_return_filter = ['person_id', 'enabled', 'key_ids','role_ids'] + person_return_filter = ['person_id', 'enabled', 'key_ids','role_ids','roles'] site_persons = Persons(self.api, person_filter, person_return_filter) - # XXX This snippet below maps role names to role_ids, - # which is really DUMB. Why can't one just pass 'roles' - # as a return_filter to Persons() above. - __roles = {} - dbroles = Roles(self.api) - for dbrole in dbroles: - __roles[dbrole['name']]=dbrole['role_id'] - __theroles = [] - for role in theroles: - __theroles.append(__roles[role]) - theroles=__theroles - # collect the keys into a table to weed out duplicates site_keys = {} for site_person in site_persons: -- 2.47.0