node-opcua-address-space 2.130.0 → 2.131.0

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "node-opcua-address-space",
-    "version": "2.130.0",
+    "version": "2.131.0",
     "description": "pure nodejs OPCUA SDK - module address-space",
     "main": "./dist/src/index_current.js",
     "types": "./dist/source/index.d.ts",
@@ -18,41 +18,41 @@
     "dependencies": {
         "@types/lodash": "4.17.7",
         "@types/semver": "^7.5.8",
-        "async": "^3.2.5",
+        "async": "^3.2.6",
         "chalk": "4.1.2",
         "dequeue": "^1.0.5",
         "lodash": "4.17.21",
-        "node-opcua-address-space-base": "2.129.0",
+        "node-opcua-address-space-base": "2.131.0",
         "node-opcua-assert": "2.120.0",
-        "node-opcua-basic-types": "2.128.0",
+        "node-opcua-basic-types": "2.131.0",
         "node-opcua-binary-stream": "2.128.0",
-        "node-opcua-client-dynamic-extension-object": "2.130.0",
+        "node-opcua-client-dynamic-extension-object": "2.131.0",
         "node-opcua-constants": "2.125.0",
-        "node-opcua-crypto": "4.9.2",
-        "node-opcua-data-access": "2.129.0",
-        "node-opcua-data-model": "2.128.0",
-        "node-opcua-data-value": "2.129.0",
+        "node-opcua-crypto": "4.9.4",
+        "node-opcua-data-access": "2.131.0",
+        "node-opcua-data-model": "2.131.0",
+        "node-opcua-data-value": "2.131.0",
         "node-opcua-date-time": "2.128.0",
         "node-opcua-debug": "2.128.0",
         "node-opcua-enum": "2.128.0",
-        "node-opcua-extension-object": "2.128.0",
-        "node-opcua-factory": "2.128.0",
+        "node-opcua-extension-object": "2.131.0",
+        "node-opcua-factory": "2.131.0",
         "node-opcua-nodeid": "2.128.0",
-        "node-opcua-nodeset-ua": "2.129.0",
-        "node-opcua-numeric-range": "2.128.0",
+        "node-opcua-nodeset-ua": "2.131.0",
+        "node-opcua-numeric-range": "2.131.0",
         "node-opcua-object-registry": "2.128.0",
-        "node-opcua-pseudo-session": "2.130.0",
-        "node-opcua-service-browse": "2.129.0",
-        "node-opcua-service-call": "2.129.0",
-        "node-opcua-service-history": "2.129.0",
-        "node-opcua-service-translate-browse-path": "2.129.0",
-        "node-opcua-service-write": "2.129.0",
+        "node-opcua-pseudo-session": "2.131.0",
+        "node-opcua-service-browse": "2.131.0",
+        "node-opcua-service-call": "2.131.0",
+        "node-opcua-service-history": "2.131.0",
+        "node-opcua-service-translate-browse-path": "2.131.0",
+        "node-opcua-service-write": "2.131.0",
         "node-opcua-status-code": "2.128.0",
-        "node-opcua-types": "2.129.0",
+        "node-opcua-types": "2.131.0",
         "node-opcua-utils": "2.128.0",
-        "node-opcua-variant": "2.129.0",
+        "node-opcua-variant": "2.131.0",
         "node-opcua-xml2json": "2.128.0",
-        "semver": "^7.6.2",
+        "semver": "^7.6.3",
         "set-prototype-of": "^1.0.0",
         "thenify": "^3.3.1",
         "xml-writer": "^1.7.0"
@@ -61,9 +61,9 @@
         "node-opcua-benchmarker": "2.128.0",
         "node-opcua-leak-detector": "2.128.0",
         "node-opcua-nodesets": "2.121.0",
-        "node-opcua-packet-analyzer": "2.128.0",
-        "node-opcua-service-filter": "2.129.0",
-        "node-opcua-test-fixtures": "2.129.0",
+        "node-opcua-packet-analyzer": "2.131.0",
+        "node-opcua-service-filter": "2.131.0",
+        "node-opcua-test-fixtures": "2.131.0",
         "source-map-support": "^0.5.21"
     },
     "author": "Etienne Rossignon",
@@ -81,7 +81,7 @@
         "internet of things"
     ],
     "homepage": "http://node-opcua.github.io/",
-    "gitHead": "94bf9012b03dbd4325cd122ce04edfa5466875bc",
+    "gitHead": "80f430dd3044826c904f803af40297839dd809be",
     "files": [
         "dist",
         "distHelpers",

package/src/base_node_impl.ts

@@ -1791,12 +1791,10 @@ function install_components_as_object_properties(parentObj: BaseNode) {
         }
 
         // ignore reserved names
-        if (doDebug) {
-            debugLog("Installing property " + name, " on ", parentObj.browseName.toString());
-        }
+        doDebug && debugLog("Installing property " + name, " on ", parentObj.browseName.toString());
 
-        /* istanbul ignore next */
-        if (Object.prototype.hasOwnProperty.call(parentObj, name)) {
+        const hasProperty = Object.prototype.hasOwnProperty.call(parentObj, name);
+        if (hasProperty && ((parentObj as any)[name] !== null && (parentObj as any)[name] !== undefined)) {
             continue;
         }
 

package/src/ua_variable_impl.ts

@@ -32,7 +32,7 @@ import { extractRange, sameDataValue, DataValue, DataValueLike, DataValueT } fro
 import { coerceClock, getCurrentClock, PreciseClock } from "node-opcua-date-time";
 import { checkDebugFlag, make_debugLog, make_errorLog, make_warningLog } from "node-opcua-debug";
 import { ExtensionObject } from "node-opcua-extension-object";
-import { NodeId } from "node-opcua-nodeid";
+import { NodeId, NodeIdLike } from "node-opcua-nodeid";
 import { NumericRange } from "node-opcua-numeric-range";
 import { WriteValue } from "node-opcua-service-write";
 import { StatusCode, StatusCodes, CallbackT } from "node-opcua-status-code";
@@ -421,7 +421,7 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
                         this.$dataValue = dataValueOrPromise as DataValue;
                         if (this.$dataValue.statusCode.isGoodish()) {
                             this.verifyVariantCompatibility(this.$dataValue.value);
-                        } 
+                        }
                     }
                 } else {
                     errorLog(
@@ -1407,6 +1407,10 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
         }
     }
 
+    public changeDataType(newDataType: NodeIdLike, newValue?: VariantLike): void {
+        changeUAVariableDataType(this, newDataType, newValue);
+    }
+
     /**
      * @private
      * install UAVariable to exposed th
@@ -2287,3 +2291,35 @@ export interface UAVariableImplT<T, DT extends DataType> extends UAVariableImpl,
     writeValue(context: ISessionContext, dataValue: DataValueT<T, DT>, indexRange?: NumericRange | null): Promise<StatusCode>;
 }
 export class UAVariableImplT<T, DT extends DataType> extends UAVariableImpl {}
+
+function changeUAVariableDataType(uaVariable: UAVariableImpl, newDataType: NodeIdLike, valueLike?: VariantLike) {
+    let value: Variant | null = valueLike ? (valueLike instanceof Variant ? valueLike : new Variant(valueLike)) : null;
+
+    const addressSpace = uaVariable.addressSpace;
+    const newDataTypeNode = addressSpace.findNode(newDataType) as UADataType;
+    // istanbul ignore next
+    if (!newDataTypeNode || !(newDataTypeNode instanceof UADataTypeImpl)) {
+        throw new Error("Cannot find newDataTypeNode " + newDataType.toString());
+    }
+
+    const newBaseDataType: DataType = newDataTypeNode.basicDataType;
+    // istanbul ignore next
+    if (newBaseDataType === DataType.Null) {
+        throw new Error("newDataTypeNode must be a DataType");
+    }
+    if (!value) {
+        value = uaVariable.readValue().value;
+        value.dataType = newBaseDataType;
+    }
+    if (newBaseDataType !== value.dataType) {
+        throw new Error("newDataTypeNode must be of the same base dataType as the value");
+    }
+
+    // change uaVariable.dataType
+    uaVariable.dataType = newDataTypeNode.nodeId;
+    (uaVariable as any)._basicDataType = null;
+    (uaVariable as any).$dataValue.value.dataType = newDataType;
+    (uaVariable as any).$dataValue.value.value = value;
+    // also change the value to ensure that we have a default value with the correct dataType
+    uaVariable._internal_set_value(value);
+}