a little nicer wrt pep8

[sfa.git] / sfa / methods / GetGids.py

diff --git a/sfa/methods/GetGids.py b/sfa/methods/GetGids.py
index f67f5d4..1e2349f 100644 (file)
--- a/sfa/methods/GetGids.py
+++ b/sfa/methods/GetGids.py
@@ -5,6 +5,7 @@ from sfa.trust.credential import Credential
 
 from sfa.storage.parameter import Parameter, Mixed
 
+
 class GetGids(Method):
     """
     Get a list of record information (hrn, gid and type) for 
@@ -16,30 +17,31 @@ class GetGids(Method):
     """
 
     interfaces = ['registry']
-    
+
     accepts = [
-        Mixed(Parameter(str, "Human readable name (hrn or xrn)"), 
+        Mixed(Parameter(str, "Human readable name (hrn or xrn)"),
               Parameter(type([str]), "List of Human readable names (hrn or xrn)")),
         Mixed(Parameter(str, "Credential string"),
-              Parameter(type([str]), "List of credentials")), 
-        ]
+              Parameter(type([str]), "List of credentials")),
+    ]
 
     returns = [Parameter(dict, "Dictionary of gids keyed on hrn")]
-    
+
     def call(self, xrns, creds):
         # validate the credential
         valid_creds = self.api.auth.checkCredentials(creds, 'getgids')
         # xxxpylintxxx origin_hrn is unused..
-        origin_hrn = Credential(string=valid_creds[0]).get_gid_caller().get_hrn()
-        
+        origin_hrn = Credential(
+            string=valid_creds[0]).get_gid_caller().get_hrn()
+
         # resolve the record
-        records = self.api.manager.Resolve(self.api, xrns, full = False)
+        records = self.api.manager.Resolve(self.api, xrns, details=False)
         if not records:
             raise RecordNotFound(xrns)
 
-        allowed_fields =  ['hrn', 'type', 'gid']
+        allowed_fields = ['hrn', 'type', 'gid']
         for record in records:
-            for key in record.keys():
+            for key in list(record.keys()):
                 if key not in allowed_fields:
                     del(record[key])
-        return records    
+        return records