Persons.can_view() requires the 'roles' fields to be set
authorMarc Fiuczynski <mef@cs.princeton.edu>
Thu, 16 Jul 2009 15:28:26 +0000 (15:28 +0000)
committerMarc Fiuczynski <mef@cs.princeton.edu>
Thu, 16 Jul 2009 15:28:26 +0000 (15:28 +0000)
PLC/Methods/GetPersons.py

index 461c04b..cfcab35 100644 (file)
@@ -66,7 +66,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']).difference(return_fields)
+            added_fields = set(['person_id', 'site_ids', 'role_ids','roles']).difference(return_fields)
             return_fields += added_fields
         else:
             added_fields = []