X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FMethods%2FGetPersons.py;h=e9ffcb8ae75f389a603424b559895ec3145db54a;hb=c3fc031363ac794e6b1245c6ed1a05329cba69c9;hp=263c66359429dd72013a5c790c489cff6a417de7;hpb=fdc1bf78189188df147a882721805e7ed5654459;p=plcapi.git

diff --git a/PLC/Methods/GetPersons.py b/PLC/Methods/GetPersons.py
index 263c663..e9ffcb8 100644
--- a/PLC/Methods/GetPersons.py
+++ b/PLC/Methods/GetPersons.py
@@ -35,8 +35,7 @@ class GetPersons(Method):
         ]
 
     # Filter out password field
-    return_fields = dict(filter(lambda (field, value): field not in hidden_fields,
-                                Person.fields.items()))
+    return_fields = dict([field_value for field_value in list(Person.fields.items()) if field_value[0] not in hidden_fields])
     returns = [return_fields]
 
     def call(self, auth, person_filter = None, return_fields = None):
@@ -67,10 +66,9 @@ class GetPersons(Method):
 
         # Filter out password field
         if return_fields:
-            return_fields = filter(lambda field: field not in hidden_fields,
-                                   return_fields)
+            return_fields = [field for field in return_fields if field not in hidden_fields]
         else:
-            return_fields = self.return_fields.keys()
+            return_fields = list(self.return_fields.keys())
 
         # Must query at least person_id, site_ids, and role_ids (see
         # Person.can_view() and below).
@@ -85,7 +83,7 @@ class GetPersons(Method):
         # Filter out accounts that are not viewable
         if isinstance(self.caller, Person) and \
            'admin' not in self.caller['roles']:
-            persons = filter(self.caller.can_view, persons)
+            persons = list(filter(self.caller.can_view, persons))
 
         # Remove added fields if not specified
         if added_fields: