a little nicer wrt pep8

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

diff --git a/sfa/methods/Update.py b/sfa/methods/Update.py
index 7643302..5daeb65 100644 (file)
--- a/sfa/methods/Update.py
+++ b/sfa/methods/Update.py
@@ -1,39 +1,44 @@
 from sfa.util.method import Method
-from sfa.util.parameter import Parameter
+from sfa.util.sfalogging import logger
+
 from sfa.trust.credential import Credential
 
+from sfa.storage.parameter import Parameter
+
+
 class Update(Method):
     """
     Update an object in the registry. Currently, this only updates the
     PLC information associated with the record. The SFA fields (name, type,
     GID) are fixed.
-    
+
     @param cred credential string specifying rights of the caller
     @param record a record dictionary to be updated
 
-    @return 1 if successful, faults otherwise 
+    @return 1 if successful, faults otherwise
     """
 
     interfaces = ['registry']
-    
+
     accepts = [
         Parameter(dict, "Record dictionary to be updated"),
         Parameter(str, "Credential string"),
-        ]
+    ]
 
     returns = Parameter(int, "1 if successful")
-    
+
     def call(self, record_dict, creds):
         # validate the cred
         valid_creds = self.api.auth.checkCredentials(creds, "update")
-        
+
         # verify permissions
-        hrn = record_dict.get('hrn', '')  
+        hrn = record_dict.get('hrn', '')
         self.api.auth.verify_object_permission(hrn)
-    
+
         # log
-        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.Update(self.api, record_dict)
+        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.Update(self.api, record_dict)