X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FMethods%2FGetPersons.py;fp=PLC%2FMethods%2FGetPersons.py;h=53143e0c30bd6e91861528aa3f4e183f4d034c5d;hb=e81fb1f3d2c89f2e3951b3c0df975e174ed6545d;hp=06da9f5b5006a995ff2af3e7a53832e0a5f92ebf;hpb=70ae1fb8bcca6e47daabf31233499636234d898a;p=plcapi.git diff --git a/PLC/Methods/GetPersons.py b/PLC/Methods/GetPersons.py index 06da9f5..53143e0 100644 --- a/PLC/Methods/GetPersons.py +++ b/PLC/Methods/GetPersons.py @@ -50,7 +50,7 @@ class GetPersons(Method): for site in sites: valid_person_ids += site['person_ids'] if not valid_person_ids: - return [] + return[] # this may look suspicious; what if person_filter is not None ? # turns out the results are getting filtered again below, so we're safe @@ -70,7 +70,7 @@ class GetPersons(Method): # Must query at least person_id, site_ids, and role_ids (see # Person.can_view() and below). if return_fields is not None: - added_fields = set(['person_id', 'site_ids', 'role_ids','roles']).difference(return_fields) + added_fields = set(['person_id', 'site_ids', 'role_ids', 'roles']).difference(return_fields) return_fields += added_fields else: added_fields = []