OpcUaEndpoint2 adjust UI

node-opcua · Aug 18, 2023 · 350779f · 350779f
1 parent 4cfb4ae
commit 350779f
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 7 deletions.
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -1,5 +1,6 @@
 {
     "cSpell.words": [
-        "oneditprepare"
+        "oneditprepare",
+        "oneditsave"
     ]
 }
diff --git a/source/nodes/OpcUa-Endpoint2/OpcUa-Endpoint2.html/editor.html b/source/nodes/OpcUa-Endpoint2/OpcUa-Endpoint2.html/editor.html
@@ -5,7 +5,7 @@
 </style>
 
 <div class="form-row">
-    <label for="node-config-input-name"><i class="icon-tasks"></i> Name</label>
+    <label for="node-config-input-name"><i class="icon-tasks"></i>Name</label>
     <input type="text" id="node-config-input-name" placeholder="enter the friendly name of the connection" />
 </div>
 
@@ -57,8 +57,8 @@
     </div>
     <div class="section-userIdentityType-UserName">
         <div class="form-row">
-            <label for="node-config-input-user"><i class="icon-user"></i> User</label>
-            <input type="text" id="node-config-input-user" />
+            <label for="node-config-input-userName"><i class="icon-user"></i> User</label>
+            <input type="text" id="node-config-input-userName" />
         </div>
         <div class="form-row">
             <label for="node-config-input-password"><i class="icon-password"></i> Password</label>

diff --git a/source/nodes/OpcUa-Endpoint2/OpcUa-Endpoint2.html/index.ts b/source/nodes/OpcUa-Endpoint2/OpcUa-Endpoint2.html/index.ts
@@ -23,6 +23,9 @@ function isValidOPC_UA_URI(str: string) {
 RED.nodes.registerType<OpcUaEndpoint2NodeProperties, CRED>("OpcUa-Endpoint2", {
     category: "config",
     defaults: {
+
+        name: { value: "" },
+
         endpoint: {
             value: "",
             required: true,
@@ -71,6 +74,7 @@ RED.nodes.registerType<OpcUaEndpoint2NodeProperties, CRED>("OpcUa-Endpoint2", {
             _securityMode.text("None");
         }
     },
+
     oneditprepare() {
         const node = this;
         // -----------------------------------------------------------------------
@@ -110,7 +114,7 @@ RED.nodes.registerType<OpcUaEndpoint2NodeProperties, CRED>("OpcUa-Endpoint2", {
             const updateIdentityControls = function (userIdentityType: string) {
                 switch (userIdentityType) {
                     case "Anonymous":
-                        $("#node-input-user").val("");
+                        $("#node-input-userName").val("");
                         $("#node-input-password").val("");
                         authRow.hide();
                         certRow.hide();
@@ -120,7 +124,7 @@ RED.nodes.registerType<OpcUaEndpoint2NodeProperties, CRED>("OpcUa-Endpoint2", {
                         certRow.hide();
                         break;
                     case "Certificate":
-                        $("#node-input-user").val("");
+                        $("#node-input-userName").val("");
                         $("#node-input-password").val("");
                         certRow.show();
                         authRow.hide();

diff --git a/source/nodes/tools/connection_tools.ts b/source/nodes/tools/connection_tools.ts
@@ -94,7 +94,8 @@ export async function get_opcua_endpoint(endpointNode: OpcUaEndpoint2Node, reque
     pendingConnection.set(endpointId, []);
 
     const securityMode = MessageSecurityMode[(endpointNode.securityMode as any) || "None"] || MessageSecurityMode.None;
-    const securityPolicy = coerceSecurityPolicy(endpointNode.securityPolicy);
+    const securityPolicy =
+        securityMode === MessageSecurityMode.None ? SecurityPolicy.None : coerceSecurityPolicy(endpointNode.securityPolicy);
     nodeToNotify.warn("securityMode   = " + securityMode);
     nodeToNotify.warn("securityPolicy = " + securityPolicy);