Merge pull request #43 from iirob/Asscesslevel

Explicit type for AccessLevel and UserAccessLevel

opcua/node.py

@@ -199,11 +199,11 @@ class Node(object):
 
     def set_writable(self, writable=True):
         if writable:
-            self.set_attribute(ua.AttributeIds.AccessLevel, ua.DataValue(ua.AccessLevelMask.CurrentWrite))
-            self.set_attribute(ua.AttributeIds.UserAccessLevel, ua.DataValue(ua.AccessLevelMask.CurrentWrite))
+            self.set_attribute(ua.AttributeIds.AccessLevel, ua.DataValue(ua.Variant(ua.AccessLevelMask.CurrentWrite, ua.VariantType.Byte)))
+            self.set_attribute(ua.AttributeIds.UserAccessLevel, ua.DataValue(ua.Variant(ua.AccessLevelMask.CurrentWrite, ua.VariantType.Byte)))
         else:
-            self.set_attribute(ua.AttributeIds.AccessLevel, ua.DataValue(ua.AccessLevelMask.CurrentRead))
-            self.set_attribute(ua.AttributeIds.AccessLevel, ua.DataValue(ua.AccessLevelMask.CurrentRead))
+            self.set_attribute(ua.AttributeIds.AccessLevel, ua.DataValue(ua.Variant(ua.AccessLevelMask.CurrentRead, ua.VariantType.Byte)))
+            self.set_attribute(ua.AttributeIds.AccessLevel, ua.DataValue(ua.Variant(ua.AccessLevelMask.CurrentRead, ua.VariantType.Byte)))
 
     def set_read_only(self):
         return self.set_writable(False)

opcua/uatypes.py

@@ -741,7 +741,6 @@ class Variant(FrozenClass):
         if val is None:
             return VariantType.Null
         elif isinstance(val, bool):
-            # TDA, added this because it was missing and causes exceptions when 'bool' type is used
             return VariantType.Boolean
         elif isinstance(val, float):
             return VariantType.Double