node-opcua-variant - npm Package Compare versions

Comparing version 2.0.0-alpha.12 to 2.0.0-alpha.13

dist/variant.d.ts

@@ -58,3 +58,3 @@ /**
 export declare function coerceVariantType(dataType: DataType, value: any): any;
-export declare function isValidVariant(arrayType: VariantArrayType, dataType: DataType, value: any, dimensions: number[] | null): boolean;
+export declare function isValidVariant(arrayType: VariantArrayType, dataType: DataType, value: any, dimensions?: number[] | null): boolean;
 export declare function buildVariantArray(dataType: DataType, nbElements: number, defaultValue: any): any[] | Int8Array | Uint8Array | Int16Array | Uint16Array | Int32Array | Uint32Array | Float32Array | Float64Array;
@@ -61,0 +61,0 @@ /***

dist/variant.js

@@ -422,2 +422,3 @@ "use strict";
     }
+    // istanbul ignore next
     if (arrayTypeConstructor && displayWarning && n > 10) {
@@ -595,3 +596,2 @@ // tslint:disable-next-line:no-console
             node_opcua_assert_1.assert(value !== undefined);
-            const ttt = value;
             if (isEnumerationItem(value)) {
@@ -625,2 +625,3 @@ // value is a enumeration of some sort
             value = (typeof value === "string") ? Buffer.from(value) : value;
+            // istanbul ignore next
             if (!(value === null || value instanceof Buffer)) {
@@ -762,6 +763,8 @@ throw new Error("ByteString should be null or a Buffer");
             value = new Array(nbElements);
-            for (let i = 0; i < nbElements; i++) {
-                value[i] = defaultValue;
-            }
     }
+    if (defaultValue !== undefined) {
+        for (let i = 0; i < nbElements; i++) {
+            value[i] = defaultValue;
+        }
+    }
     return value;
@@ -768,0 +771,0 @@ }

package.json

 {
  "name": "node-opcua-variant",
- "version": "2.0.0-alpha.12",
+ "version": "2.0.0-alpha.13",
  "description": "pure nodejs OPCUA SDK - module -variant",
@@ -14,8 +14,8 @@ "main": "./dist/index.js",
   "node-opcua-assert": "^2.0.0-alpha.10",
-  "node-opcua-basic-types": "^2.0.0-alpha.12",
-  "node-opcua-data-model": "^2.0.0-alpha.12",
+  "node-opcua-basic-types": "^2.0.0-alpha.13",
+  "node-opcua-data-model": "^2.0.0-alpha.13",
   "node-opcua-enum": "^2.0.0-alpha.10",
-  "node-opcua-factory": "^2.0.0-alpha.12",
-  "node-opcua-nodeid": "^2.0.0-alpha.11",
-  "node-opcua-utils": "^2.0.0-alpha.12",
+  "node-opcua-factory": "^2.0.0-alpha.13",
+  "node-opcua-nodeid": "^2.0.0-alpha.13",
+  "node-opcua-utils": "^2.0.0-alpha.13",
   "underscore": "^1.9.1"
@@ -25,8 +25,8 @@ },
   "node-opcua-benchmarker": "^2.0.0-alpha.10",
-  "node-opcua-binary-stream": "^2.0.0-alpha.12",
+  "node-opcua-binary-stream": "^2.0.0-alpha.13",
   "node-opcua-debug": "^2.0.0-alpha.10",
-  "node-opcua-generator": "^2.0.0-alpha.12",
-  "node-opcua-numeric-range": "^2.0.0-alpha.12",
-  "node-opcua-packet-analyzer": "^2.0.0-alpha.12",
-  "node-opcua-status-code": "^2.0.0-alpha.12",
+  "node-opcua-generator": "^2.0.0-alpha.13",
+  "node-opcua-numeric-range": "^2.0.0-alpha.13",
+  "node-opcua-packet-analyzer": "^2.0.0-alpha.13",
+  "node-opcua-status-code": "^2.0.0-alpha.13",
   "should": "13.2.3",
@@ -50,3 +50,3 @@ "source-map-support": "^0.5.10"
  "homepage": "http://node-opcua.github.io/",
- "gitHead": "61699a99c75c631388f2e8d8e33eb209649c5eac"
+ "gitHead": "6a0c56afb819a44d5abd453d39ce684df6fb3505"
 }

source/variant.ts

@@ -543,2 +543,3 @@ /**
     }
+    // istanbul ignore next
     if (arrayTypeConstructor && displayWarning && n > 10) {
@@ -752,3 +753,3 @@ // tslint:disable-next-line:no-console
             assert(value !== undefined);
-            const ttt = value;
+
             if (isEnumerationItem(value)) {
@@ -781,2 +782,4 @@ // value is a enumeration of some sort
             value = (typeof value === "string") ? Buffer.from(value) : value;
+
+            // istanbul ignore next
             if (!(value === null || value instanceof Buffer)) {
@@ -880,3 +883,3 @@ throw new Error("ByteString should be null or a Buffer");
     dataType: DataType,
-    value: any, dimensions: number[] | null): boolean {
+    value: any, dimensions?: number[] | null): boolean {
 
@@ -890,3 +893,3 @@ switch (arrayType) {
             assert(arrayType === VariantArrayType.Matrix);
-            return isValidMatrixVariant(dataType, value, dimensions);
+            return isValidMatrixVariant(dataType, value, dimensions!);
     }
@@ -925,6 +928,8 @@ }
             value = new Array(nbElements);
-            for (let i = 0; i < nbElements; i++) {
-                value[i] = defaultValue;
-            }
     }
+    if (defaultValue !== undefined) {
+        for (let i = 0; i < nbElements; i++) {
+            value[i] = defaultValue;
+        }
+    }
     return value;
@@ -931,0 +936,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

194689

increased by78.68%

Number of package files

22

increased by10%

Lines of code

1961

increased by0.31%

Dependency changes

• Updatednode-opcua-basic-types@^2.0.0-alpha.13

• Updatednode-opcua-data-model@^2.0.0-alpha.13

• Updatednode-opcua-factory@^2.0.0-alpha.13

• Updatednode-opcua-nodeid@^2.0.0-alpha.13

• Updatednode-opcua-utils@^2.0.0-alpha.13