X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=sfa%2Fmanagers%2Fmanagerwrapper.py;h=326b6b63147b7a1409cf06dc6a7c45273ec133f9;hb=4a9e6751f9f396f463932133b9d62fc925a99ef6;hp=da8c98f689fc4096601e175a03ddedc390abf6eb;hpb=ecc85e0b923922cf7117d29b380f5284edb88f21;p=sfa.git

diff --git a/sfa/managers/managerwrapper.py b/sfa/managers/managerwrapper.py
index da8c98f6..326b6b63 100644
--- a/sfa/managers/managerwrapper.py
+++ b/sfa/managers/managerwrapper.py
@@ -1,37 +1,42 @@
-from types import ModuleType, ClassType
+from types import ModuleType
 
 from sfa.util.faults import SfaNotImplemented, SfaAPIError
 from sfa.util.sfalogging import logger
 
 ####################
+
+
 class ManagerWrapper:
     """
     This class acts as a wrapper around an SFA interface manager module, but
-    can be used with any python module. The purpose of this class is raise a 
-    SfaNotImplemented exception if someone attempts to use an attribute 
+    can be used with any python module. The purpose of this class is raise a
+    SfaNotImplemented exception if someone attempts to use an attribute
     (could be a callable) thats not available in the library by checking the
-    library using hasattr. This helps to communicate better errors messages 
-    to the users and developers in the event that a specifiec operation 
+    library using hasattr. This helps to communicate better errors messages
+    to the users and developers in the event that a specifiec operation
     is not implemented by a libarary and will generally be more helpful than
-    the standard AttributeError         
+    the standard AttributeError
     """
+
     def __init__(self, manager, interface, config):
-        if isinstance (manager, ModuleType):
+        if isinstance(manager, ModuleType):
             # old-fashioned module implementation
             self.manager = manager
-        elif isinstance (manager, ClassType):
-            # create an instance; we don't pass the api in argument as it is passed 
+        # if it's a class
+        elif isinstance(manager, type):
+            # create an instance; we don't pass the api in argument as it is passed
             # to the actual method calls anyway
             self.manager = manager(config)
         else:
             # that's what happens when there's something wrong with the db
             # or any bad stuff of that kind at startup time
-            logger.log_exc("Failed to create a manager, startup sequence is broken")
-            raise SfaAPIError("Argument to ManagerWrapper must be a module or class")
+            logger.log_exc(
+                "Failed to create a manager, startup sequence is broken")
+            raise SfaAPIError(
+                "Argument to ManagerWrapper must be a module or class")
         self.interface = interface
-        
+
     def __getattr__(self, method):
         if not hasattr(self.manager, method):
             raise SfaNotImplemented(self.interface, method)
         return getattr(self.manager, method)
-