connectors/PYRO/__init__.py
changeset 2429 15f18dc8b56a
parent 2339 48b4eba13064
parent 2418 5587c490a070
child 2430 65ff9a309ff3
--- a/connectors/PYRO/__init__.py	Thu Nov 22 23:39:54 2018 +0100
+++ b/connectors/PYRO/__init__.py	Fri Nov 23 00:33:04 2018 +0100
@@ -37,6 +37,7 @@
 from Pyro.errors import PyroError
 
 import PSKManagement as PSK
+from runtime import PlcStatus
 
 zeroconf_service_type = '_PYRO._tcp.local.'
 # this module attribute contains a list of DNS-SD (Zeroconf) service types
@@ -91,7 +92,7 @@
     # Try to get the proxy object
     try:
         RemotePLCObjectProxy = Pyro.core.getAttrProxyForURI(schemename + "://" + location + "/PLCObject")
-    except Exception,e:
+    except Exception:
         confnodesroot.logger.write_error(_("Connection to '%s' failed with exception '%s'\n") % (location, str(e)))
         #confnodesroot.logger.write_error(traceback.format_exc())
         return None
@@ -104,12 +105,12 @@
         def catcher_func(*args, **kwargs):
             try:
                 return func(*args, **kwargs)
-            except Pyro.errors.ConnectionClosedError, e:
+            except Pyro.errors.ConnectionClosedError as e:
                 confnodesroot.logger.write_error(_("Connection lost!\n"))
                 confnodesroot._SetConnector(None)
-            except Pyro.errors.ProtocolError, e:
+            except Pyro.errors.ProtocolError as e:
                 confnodesroot.logger.write_error(_("Pyro exception: %s\n") % e)
-            except Exception, e:
+            except Exception as e:
                 # confnodesroot.logger.write_error(traceback.format_exc())
                 errmess = ''.join(Pyro.util.getPyroTraceback(e))
                 confnodesroot.logger.write_error(errmess + "\n")
@@ -130,8 +131,8 @@
 
     _special_return_funcs = {
         "StartPLC": False,
-        "GetTraceVariables": ("Broken", None),
-        "GetPLCstatus": ("Broken", None),
+        "GetTraceVariables": (PlcStatus.Broken, None),
+        "GetPLCstatus": (PlcStatus.Broken, None),
         "RemoteExec": (-1, "RemoteExec script failed!")
     }
 
@@ -140,7 +141,6 @@
         A proxy proxy class to handle Beremiz Pyro interface specific behavior.
         And to put Pyro exception catcher in between caller and Pyro proxy
         """
-
         def __getattr__(self, attrName):
             member = self.__dict__.get(attrName, None)
             if member is None:
@@ -151,4 +151,3 @@
             return member
 
     return PyroProxyProxy()
-