avoid as much as possible accessing logger through class instances, whenever that...

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

diff --git a/sfa/methods/List.py b/sfa/methods/List.py
index 83d7a6e..b487285 100644 (file)
--- a/sfa/methods/List.py
+++ b/sfa/methods/List.py
@@ -1,37 +1,42 @@
 
 from sfa.util.xrn import urn_to_hrn
 from sfa.util.method import Method
+from sfa.util.sfalogging import logger
 
 from sfa.trust.credential import Credential
 
 from sfa.storage.parameter import Parameter, Mixed
 
+
 class List(Method):
     """
-    List the records in an authority. 
+    List the records in an authority.
 
     @param cred credential string specifying the rights of the caller
     @param hrn human readable name of authority to list (hrn or urn)
-    @return list of record dictionaries         
+    @return list of record dictionaries
     """
     interfaces = ['registry']
-    
+
     accepts = [
         Parameter(str, "Human readable name (hrn or urn)"),
         Mixed(Parameter(str, "Credential string"),
               Parameter(type([str]), "List of credentials")),
-        ]
+    ]
 
     # xxx used to be [SfaRecord]
     returns = [Parameter(dict, "registry record")]
-    
+
     def call(self, xrn, creds, options=None):
-        if options is None: options={}
+        if options is None:
+            options = {}
         hrn, type = urn_to_hrn(xrn)
         valid_creds = self.api.auth.checkCredentials(creds, 'list')
 
-        #log the call
-        origin_hrn = Credential(string=valid_creds[0]).get_gid_caller().get_hrn()
-        self.api.logger.info("interface: %s\tcaller-hrn: %s\ttarget-hrn: %s\tmethod-name: %s"%(self.api.interface, origin_hrn, hrn, self.name))
-       
-        return self.api.manager.List(self.api, xrn, options=options) 
+        # log the call
+        origin_hrn = Credential(
+            string=valid_creds[0]).get_gid_caller().get_hrn()
+        logger.info("interface: %s\tcaller-hrn: %s\ttarget-hrn: %s\tmethod-name: %s" %
+                    (self.api.interface, origin_hrn, hrn, self.name))
+
+        return self.api.manager.List(self.api, xrn, options=options)