node-red-contrib-knx-ultimate 2.2.40 → 2.3.0

package/CHANGELOG.md

@@ -6,6 +6,9 @@
 
 # CHANGELOG
 
+**Version 2.3.0** - Jan 2024<br/>
+- HUE Light: partially rewrote code to cloneDeep(oHUEDevice) not to duplicate the brightness status of a single ligh, at node-red restart. https://github.com/Supergiovane/node-red-contrib-knx-ultimate/issues/312#issue-2064480332 <br/>
+
 **Version 2.2.40** - Jan 2024<br/>
 - HUE Light: fixed an issue with initial status read from HUE Bridge (on/off status was incorrectly set).<br/>
 

package/nodes/hue-config.js

@@ -7,6 +7,7 @@ const dptlib = require("../KNXEngine/src/dptlib");
 const HueClass = require("./utils/hueEngine").classHUE;
 const loggerEngine = require("./utils/sysLogger");
 const hueColorConverter = require("./utils/hueColorConverter");
+const cloneDeep = require("lodash/cloneDeep");
 
 function getRandomIntInclusive(min, max) {
   min = Math.ceil(min);
@@ -206,7 +207,7 @@ module.exports = (RED) => {
                   shape: "ring",
                   text: "Ready :-)",
                 });
-                _node.currentHUEDevice = oHUEDevice;
+                _node.currentHUEDevice = cloneDeep(oHUEDevice);
                 if (_node.initializingAtStart === true) _node.handleSendHUE(oHUEDevice);
               }
             }
@@ -417,7 +418,7 @@ module.exports = (RED) => {
         if (node.hueAllResources !== undefined && node.hueAllResources !== null && _Node.initializingAtStart === true) {
           const oHUEDevice = node.hueAllResources.filter((a) => a.id === _Node.hueDevice)[0];
           if (oHUEDevice !== undefined) {
-            _Node.currentHUEDevice = oHUEDevice;
+            _Node.currentHUEDevice = cloneDeep(oHUEDevice);
             _Node.handleSendHUE(oHUEDevice);
             // Add _Node to the clients array
             _Node.setNodeStatusHue({

package/nodes/knxUltimateHueLight.js

@@ -667,7 +667,7 @@ module.exports = function (RED) {
             // In case of switch off, set the dim to zero
             if (_event.on.on === false && (config.updateKNXBrightnessStatusOnHUEOnOff === undefined || config.updateKNXBrightnessStatusOnHUEOnOff === "onhueoff")) {
               node.updateKNXBrightnessState(0);
-              //node.currentHUEDevice.dimming.brightness = 0;
+              if (_event.dimming !== undefined) delete _event.dimming; // Remove event.dimming, because has beem handled by this function and i don't want the function below to take care of it.
             } else if (_event.on.on === true && node.currentHUEDevice.on.on === false) {
               // Turn on always update the dimming KNX Status value as well.
               let brightVal = 50;
@@ -715,7 +715,7 @@ module.exports = function (RED) {
           shape: "dot",
           text: `HUE->KNX error ${node.id} ${error.message}. Seee Log`,
         });
-        RED.log.error(`knxUltimateHueLight: node.handleSendHUE = (_event): ${error.message}`);
+        RED.log.error(`knxUltimateHueLight: node.handleSendHUE = (_event): ${error.stack}`);
       }
     };
 

package/package.json

@@ -3,7 +3,7 @@
   "engines": {
     "node": ">=16.0.0"
   },
-  "version": "2.2.40",
+  "version": "2.3.0",
   "description": "Control your KNX intallation via Node-Red! A bunch of KNX nodes, with integrated Philips HUE control and ETS group address importer. Easy to use and highly configurable.",
   "dependencies": {
     "binary-parser": "2.2.1",