homebridge-smartsystem 6.2.2 → 6.2.3

package/README.md

@@ -271,6 +271,7 @@ Homebridge UI version
  ### v6.2.x - 11-08-2022 - P1 release
  - 1: always write config to file too (next to sending through the HB-UI API)
  - 2: added headers to http calls from switches, dimmers and up/down's (including OH stuff)
+ - 3: fixed bug in Sanitizer of UnitConfig preventing from publishing units to homebridge
  
 
 ## Hardware 

package/duotecno/types.js

@@ -380,31 +380,32 @@ exports.Sanitizers = {
         if (!config) {
             config = {};
         }
-        config = exports.Sanitizers.unitDef(config);
-        config.active = config.active || 'N';
-        config.used = config.used || config.active;
+        const cfg = exports.Sanitizers.unitDef(config);
+        cfg.active = config.active || 'N';
+        cfg.used = config.used || config.active;
         if (typeof config.group === 'string') {
             config.group = parseInt(config.group);
         }
-        config.group = config.group || 0;
-        return config;
+        cfg.group = config.group || 0;
+        return cfg;
     },
     unitScene: function (config) {
         // change + create new clean record for writing to config files
         if (!config) {
             config = {};
         }
-        config = exports.Sanitizers.unitDef(config);
+        const cfg = exports.Sanitizers.unitDef(config);
         if (typeof config.value === "undefined")
-            config.value = 0;
+            cfg.value = 0;
         if (typeof config.value === "string")
-            config.value = parseInt(config.value);
-        return { logicalNodeAddress: config.logicalNodeAddress,
-            logicalAddress: config.logicalAddress,
-            masterAddress: config.masterAddress,
-            masterPort: config.masterPort,
-            value: config.value,
-            name: config.name };
+            cfg.value = parseInt(config.value);
+        cfg.name = config.name || "New scene";
+        return { logicalNodeAddress: cfg.logicalNodeAddress,
+            logicalAddress: cfg.logicalAddress,
+            masterAddress: cfg.masterAddress,
+            masterPort: cfg.masterPort,
+            value: cfg.value,
+            name: cfg.name };
     },
     ////////////
     // Scenes //

package/duotecno/types.ts

@@ -570,31 +570,32 @@ export const Sanitizers = {
 
   unitConfig: function(config?: UnitConfig) {  // unitDef + active + group
     if (!config) { config = <UnitConfig>{}; }
-    config = <UnitConfig>Sanitizers.unitDef(config);
+    const cfg = <UnitConfig>Sanitizers.unitDef(config);
 
-    config.active = config.active || 'N';
-    config.used = config.used || config.active;
+    cfg.active = config.active || 'N';
+    cfg.used = config.used || config.active;
 
     if (typeof config.group === 'string') { config.group = parseInt(config.group); }
-    config.group = config.group || 0;
+    cfg.group = config.group || 0;
 
-    return config;
+    return cfg;
   },
 
   
   unitScene: function(config?: UnitScene) {  // unitDef + value 
     // change + create new clean record for writing to config files
     if (!config) { config = <UnitScene>{}; }
-    config = <UnitScene>Sanitizers.unitDef(config);
-    if (typeof config.value === "undefined") config.value = 0;
-    if (typeof config.value === "string") config.value = parseInt(config.value);
-
-    return { logicalNodeAddress: config.logicalNodeAddress,
-             logicalAddress: config.logicalAddress,
-             masterAddress: config.masterAddress,
-             masterPort: config.masterPort,
-             value: config.value,
-             name: config.name };
+    const cfg = <UnitScene>Sanitizers.unitDef(config);
+    if (typeof config.value === "undefined") cfg.value = 0;
+    if (typeof config.value === "string") cfg.value = parseInt(config.value);
+    cfg.name = config.name || "New scene";
+
+    return { logicalNodeAddress: cfg.logicalNodeAddress,
+             logicalAddress: cfg.logicalAddress,
+             masterAddress: cfg.masterAddress,
+             masterPort: cfg.masterPort,
+             value: cfg.value,
+             name: cfg.name };
   },
 
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-smartsystem",
   "displayname": "Duotecno Bridge",
-  "version": "6.2.2",
+  "version": "6.2.3",
   "description": "SmartServer (Proxy Websockets to TCP sockets, Smappee MQTT, Duotecno IP Nodes, Homekit interface)",
   "main": "index.js",
   "author": "Johan Coppieters",