From: Marc Fiuczynski <mef@cs.princeton.edu>
Date: Thu, 16 Jul 2009 15:28:26 +0000 (+0000)
Subject: Persons.can_view() requires the 'roles' fields to be set
X-Git-Tag: PLCAPI-4.3-23~42
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=54a91e22ca955a67f657d8b8ce916440c45c9c5c;p=plcapi.git

Persons.can_view() requires the 'roles' fields to be set
---

diff --git a/PLC/Methods/GetPersons.py b/PLC/Methods/GetPersons.py
index 461c04b0..cfcab353 100644
--- a/PLC/Methods/GetPersons.py
+++ b/PLC/Methods/GetPersons.py
@@ -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 = []