node-red-contrib-knx-ultimate 2.2.25 → 2.2.26

package/CHANGELOG.md

@@ -6,6 +6,15 @@
 
 # CHANGELOG
 
+**Version 2.2.26** - November 2023<br/>
+This is an interim version, to quick fix some issues. Please report any issue with HUE Nodes, on gitHub.<br/>
+- HUE Light: fixed some spurious node status errors.<br/>
+- HUE Light: now the brightness status is ever transmitted over the KNX bus, whenever the light is switched on.<br/>
+- HUE Battery: the node can respond to KNX read request, by sending the stored value as response to the KNX bus.<br/>
+- HUE light level: the node can respond to KNX read request, by sending the stored value as response to the KNX bus.<br/>
+- HUE temperature sensor: the node can respond to KNX read request, by sending the stored value as response to the KNX bus.<br/>
+- WARNING: the new HUE Light options are to be considered **BETA (= in testing with user feedback)**.<br/>
+
 **Version 2.2.25** - November 2023<br/>
 - HUE Light: fixed settings of some behaviour options.<br/>
 - You can now query the HUE Light node stati, via a "read" telegram sent to the KNX stati group address. ("Status" is Latin, not English, so the plural is "stati" and not "statuses" nor "status"). Other HUE nodes will follow asap.<br/>

package/nodes/hue-config.html

@@ -18,7 +18,7 @@
             }
             $("#getinfocam").click(function () {
 
-                var myNotification = RED.notify("Please press the Link button on the HUE Bridge",
+                var myNotification = RED.notify("Please press the Link button on the HUE Bridge. Once pressed, click OK.",
                     {
                         modal: true,
                         fixed: true,
@@ -125,17 +125,15 @@
             <label for="node-config-input-clientkey"> Bridge Key</label>
             <input type="password" id="node-config-input-clientkey" placeholder="" disabled>
         </div>
+        <div class="form-row">
+            <label for="node-config-input-debug"> Bridge Key</label>
+            <input rows="20" style="width:100%" type="textarea" id="node-config-input-debug" placeholder="" disabled>
+        </div>
     </div>
     
     
 </script>
-<script type="text/markdown" data-help-name="hue-config"
-This node registers to the Hue Bridge.  
-
-Just set the Bridge's IP and click **CONNECT** button.
-
-[Find it useful?](https://www.paypal.me/techtoday)
-
-<br/>
+<script type="text/markdown" data-help-name="hue-config" This node registers to the Hue Bridge. Just set the Bridge's IP
+    and click **CONNECT** button. [Find it useful?](https://www.paypal.me/techtoday) <br />
 
 </script>

package/nodes/hue-config.js

@@ -174,6 +174,24 @@ module.exports = (RED) => {
       // °°°°°° Load ALL resources
       try {
         node.hueAllResources = await node.hueManager.hueApiV2.get("/resource");
+
+        // // DEBUG
+        // try {
+        //   const fs = require('fs');
+        //   const { resolve } = require('path');
+        //   const content = JSON.stringify(node.hueAllResources);
+        //   try {
+        //     fs.writeFileSync('resources.json', content);
+        //     RED.log.info("******************************* FILE WROTE IN resources.json " + resolve("resources.json"))
+        //     // file written successfully
+        //   } catch (error) {
+        //     RED.log.error("********************************************* const content = JSON.stringify(node.hueAllResources)2222: " + error.message)
+        //   }
+        // } catch (error) {
+        //   RED.log.error("********************************************* const content = JSON.stringify(node.hueAllResources): " + error.message)
+        // }
+
+
         if (node.hueAllResources !== undefined) {
           node.hueAllRooms = node.hueAllResources.filter((a) => a.type === "room");
           // Update all KNX State of the nodes with the new hue device values
@@ -206,21 +224,25 @@ module.exports = (RED) => {
     };
 
     node.getFirstLightInGroup = function getFirstLightInGroup(_groupID) {
-      if (node.hueAllResources === undefined) return;
-      // Find the group
-      const group = node.hueAllResources.filter((a) => a.id === _groupID)[0];
-      if (group === null || group === undefined) return;
-      const owner = node.hueAllResources.filter((a) => a.id === group.owner.rid)[0];
-      if (owner.children !== undefined && owner.children.length > 0) {
-        const firstLightId = owner.children.find((a) => a.rtype === "light").rid;
-        if (firstLightId !== undefined && firstLightId !== null) {
-          const firstLight = node.hueAllResources.find((a) => a.id === firstLightId);
-          if (firstLight !== null && firstLight !== undefined) {
-            return firstLight;
-          } else {
-            return;
+      if (node.hueAllResources === undefined || node.hueAllResources === null) return;
+      try {
+        // Find the group
+        const group = node.hueAllResources.filter((a) => a.id === _groupID)[0];
+        if (group === null || group === undefined) return;
+        const owner = node.hueAllResources.filter((a) => a.id === group.owner.rid)[0];
+        if (owner.children !== undefined && owner.children.length > 0) {
+          const firstLightId = owner.children.find((a) => a.rtype === "light").rid;
+          if (firstLightId !== undefined && firstLightId !== null) {
+            const firstLight = node.hueAllResources.find((a) => a.id === firstLightId);
+            if (firstLight !== null && firstLight !== undefined) {
+              return firstLight;
+            } else {
+              return;
+            }
           }
         }
+      } catch (error) {
+
       }
     };
 
@@ -443,7 +465,7 @@ module.exports = (RED) => {
         // °°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°
         const serverNode = RED.nodes.getNode(req.query.nodeID); // Retrieve node.id of the config node.
         if (serverNode === null) {
-          RED.log.error(`Warn KNXUltimateGetResourcesHUE serverNode is null`);
+          RED.log.warn(`Warn KNXUltimateGetResourcesHUE serverNode is null`);
           res.json({ devices: `serverNode not set` });
           return;
         }
@@ -457,6 +479,7 @@ module.exports = (RED) => {
       } catch (error) {
         //RED.log.error(`Errore KNXUltimateGetResourcesHUE non gestito ${error.message}`);
         res.json({ devices: error.message });
+        RED.log.error(`Err KNXUltimateGetResourcesHUE: ${error.message}`);
         // (async () => {
         //   await node.initHUEConnection();
         // })();

package/nodes/knxUltimate-config.html

@@ -21,7 +21,7 @@
             ignoreTelegramsWithRepeatedFlag: { value: false, required: false },
             keyringFileXML: { value: "" },
             knxSecureSelected: { value: false },
-            autoReconnect: { value: true }
+            autoReconnect: { value: "yes" }
         },
         credentials: {
             keyringFilePassword: { type: "password" }

package/nodes/knxUltimateHueBattery.html

@@ -13,13 +13,13 @@
             namebatterysensor: { value: "" },
             GAbatterysensor: { value: "" },
             dptbatterysensor: { value: "" },
-            readStatusAtStartup: { value: "no" },
+            readStatusAtStartup: { value: "yes" },
 
 
             hueDevice: { value: "" }
         },
         inputs: 0,
-        outputs: 0,
+        outputs: 1,
         icon: "node-hue-icon.svg",
         label: function () {
             return (this.name);

package/nodes/knxUltimateHueBattery.js

@@ -7,7 +7,7 @@ module.exports = function (RED) {
     node.topic = node.name;
     node.name = config.name === undefined ? 'Hue' : config.name;
     node.dpt = '';
-    node.notifyreadrequest = false;
+    node.notifyreadrequest = true;
     node.notifyreadrequestalsorespondtobus = 'false';
     node.notifyreadrequestalsorespondtobusdefaultvalueifnotinitialized = '';
     node.notifyresponse = false;
@@ -23,7 +23,9 @@ module.exports = function (RED) {
     node.formatnegativevalue = 'leave';
     node.formatdecimalsvalue = 2;
     node.hueDevice = config.hueDevice;
-    node.initializingAtStart = !((config.readStatusAtStartup === undefined || config.readStatusAtStartup === "no"));
+    node.initializingAtStart = !((config.readStatusAtStartup === undefined || config.readStatusAtStartup === "yes"));
+    node.currentDeviceValue = 0;
+
     // Used to call the status update from the config node.
     node.setNodeStatus = ({
       fill, shape, text, payload,
@@ -42,13 +44,22 @@ module.exports = function (RED) {
 
     // This function is called by the knx-ultimate config node, to output a msg.payload.
     node.handleSend = (msg) => {
+      // Respond to KNX read telegram, by sending the current value as response telegram.
+      if (msg.knx.event === "GroupValue_Read") {
+        switch (msg.knx.destination) {
+          case config.GAbatterysensor:
+            // To the KNX bus wires
+            node.sendResponseToKNX(node.currentDeviceValue);
+            break;
+          default:
+            break;
+        }
+      }
     };
 
     node.handleSendHUE = (_event) => {
       try {
         if (_event.id === config.hueDevice) {
-
-          // IMPORTANT: exit if no event presen.
           if (!_event.hasOwnProperty("power_state") || _event.power_state.battery_level === undefined) return;
 
           const knxMsgPayload = {};
@@ -62,6 +73,7 @@ module.exports = function (RED) {
               grpaddr: knxMsgPayload.topic, payload: knxMsgPayload.payload, dpt: knxMsgPayload.dpt, outputtype: 'write', nodecallerid: node.id,
             });
           }
+          node.currentDeviceValue = knxMsgPayload.payload;
 
           // Setup the output msg
           knxMsgPayload.name = node.name;
@@ -73,13 +85,27 @@ module.exports = function (RED) {
           node.setNodeStatusHue({
             fill: 'blue', shape: 'ring', text: 'HUE->KNX', payload: knxMsgPayload.payload,
           });
-
         }
       } catch (error) {
         node.status({ fill: 'red', shape: 'dot', text: `HUE->KNX error ${error.message} (${new Date().getDate()}, ${new Date().toLocaleTimeString()})` });
       }
     };
 
+
+    node.sendResponseToKNX = (_level) => {
+      const knxMsgPayload = {};
+      knxMsgPayload.topic = config.GAbatterysensor;
+      knxMsgPayload.dpt = config.dptbatterysensor;
+
+      knxMsgPayload.payload = _level;
+      // Send to KNX bus
+      if (knxMsgPayload.topic !== '' && knxMsgPayload.topic !== undefined) {
+        node.server.writeQueueAdd({
+          grpaddr: knxMsgPayload.topic, payload: knxMsgPayload.payload, dpt: knxMsgPayload.dpt, outputtype: 'response', nodecallerid: node.id,
+        });
+      }
+    };
+
     // On each deploy, unsubscribe+resubscribe
     if (node.server) {
       node.server.removeClient(node);

package/nodes/knxUltimateHueButton.js

@@ -194,8 +194,6 @@ module.exports = function (RED) {
         node.server.writeQueueAdd({
           grpaddr: knxMsgPayload.topic, payload: knxMsgPayload.payload, dpt: knxMsgPayload.dpt, outputtype: 'write', nodecallerid: node.id,
         });
-      }
-      if (knxMsgPayload.topic !== '' && knxMsgPayload.topic !== undefined) {
         node.setNodeStatusHue({
           fill: 'grey', shape: 'ring', text: 'HUE->KNX STOP DIM', payload: knxMsgPayload.payload,
         });

package/nodes/knxUltimateHueLight.html

@@ -92,6 +92,7 @@
       inputs: { value: 0 },
 
       hueDevice: { value: "" },
+      hueDeviceObject: { value: {} },
     },
     inputs: 0,
     outputs: 0,
@@ -99,7 +100,7 @@
     label: function () {
       return this.name;
     },
-    paletteLabel: "Hue Light",
+    paletteLabel: "Hue Light (beta)",
     oneditprepare: function () {
       var node = this;
       var oNodeServer = RED.nodes.node($("#node-input-server").val()); // Store the config-node
@@ -126,8 +127,10 @@
       }
       // ################################################################
 
+
       $("#tabs").tabs(); // Tabs gestione KNX
 
+
       // 19/02/2020 Used to get the server sooner als deploy.
       $("#node-input-server").change(function () {
         try {
@@ -169,7 +172,7 @@
             }
           });
           // Eval
-          const format = "node." + _destinationWidget.replace("#node-input-", "");
+          const format = "this." + _destinationWidget.replace("#node-input-", "");
           try {
             if (format !== undefined) $(_destinationWidget).val(eval(format).toString());
           } catch (error) { }
@@ -264,6 +267,52 @@
       getGroupAddress("#node-input-GALightKelvinState", "#node-input-nameLightKelvinState", "#node-input-dptLightKelvinState", " 9.002");
 
 
+
+      // Autocomplete suggestion with HUE Lights
+      $("#node-input-name").autocomplete({
+        minLength: 1,
+        source: function (request, response) {
+          $.getJSON("KNXUltimateGetResourcesHUE?rtype=light&nodeID=" + oNodeServerHue.id, (data) => {
+            response(
+              $.map(data.devices, function (value, key) {
+                //alert(JSON.stringify(value) + " "+ key)
+                var sSearch = value.name;
+                if (!value.name.includes("I'm still connecting")) {
+                  if (fullSearch(sSearch, request.term)) {
+                    return {
+                      hueDevice: value.id,
+                      value: value.name,
+                    };
+                  } else {
+                    return null;
+                  }
+                } else {
+                  return {
+                    hueDevice: value.id,
+                    value: value.name,
+                  };
+                }
+              })
+            );
+          });
+        },
+        select: function (event, ui) {
+          // Distinguish between group of lights an single light.
+          if (ui.item.value.toLowerCase().startsWith("grouped_light")) {
+            $("#node-input-hueDevice").val(ui.item.hueDevice + "#grouped_light");
+            $("#getColorAtSwitchOnDayTimeButton").hide();
+            $("#getColorAtSwitchOnNightTimeButton").hide();
+          } else {
+            $("#node-input-hueDevice").val(ui.item.hueDevice + "#light");
+            $("#getColorAtSwitchOnDayTimeButton").show();
+            $("#getColorAtSwitchOnNightTimeButton").show();
+          }
+          $("#tabs").show();
+        },
+      });
+
+
+
       // Show/Hide the div of the color at swich on
       if (this.specifySwitchOnBrightness === "yes") {
         $("#divColorsAtSwitchOn").show();
@@ -313,7 +362,7 @@
           $("#divColorsAtSwitchOnNightTime").show();
           $("#divTemperatureAtSwitchOnNightTime").hide();
           blinkBackground("#colorPickerNight")
-          $("#getColorAtSwitchOnDayTimeButton").text("Get again");
+          $("#getColorAtSwitchOnDayTimeButton").text("Get current");
         } else if ($("#node-input-enableDayNightLighting").val() === "temperature") {
           $("#divEnableDayNightLighting").show();
           $("#divCCSBoxAtNightLighting").css({ border: "1px solid dimgrey", "border-radius": "12px", padding: "5px" }); // Add little box to better understand the property page
@@ -326,67 +375,65 @@
       });
 
 
-
-      // Autocomplete suggestion with HUE Lights
-      $("#node-input-name").autocomplete({
-        minLength: 1,
-        source: function (request, response) {
-          $.getJSON("KNXUltimateGetResourcesHUE?rtype=light&nodeID=" + oNodeServerHue.id, (data) => {
-            response(
-              $.map(data.devices, function (value, key) {
-                //alert(JSON.stringify(value) + " "+ key)
-                var sSearch = value.name;
-                if (!value.name.includes("I'm still connecting")) {
-                  if (fullSearch(sSearch, request.term)) {
-                    return {
-                      hueDevice: value.id,
-                      value: value.name,
-                    };
-                  } else {
-                    return null;
-                  }
-                } else {
-                  return {
-                    hueDevice: value.id,
-                    value: value.name,
-                  };
-                }
-              })
-            );
-          });
-        },
-        select: function (event, ui) {
-          // Distinguish between group of lights an single light.
-          if (ui.item.value.toLowerCase().startsWith("grouped_light")) {
-            $("#node-input-hueDevice").val(ui.item.hueDevice + "#grouped_light");
-            $("#getColorAtSwitchOnDayTimeButton").hide();
-            $("#getColorAtSwitchOnNightTimeButton").hide();
-          } else {
-            $("#node-input-hueDevice").val(ui.item.hueDevice + "#light");
-            $("#getColorAtSwitchOnDayTimeButton").show();
-            $("#getColorAtSwitchOnNightTimeButton").show();
-          }
-          $("#tabs").show();
-        },
-      });
-
       // Get the HUE capabilities to enable/disable UI parts
       $.getJSON("KNXUltimateGetResourcesHUE?rtype=light&nodeID=" + oNodeServerHue.id, (data) => {
         data.devices.forEach((element) => {
           if (element.id === this.hueDevice.split("#")[0] && element.deviceObject !== undefined) {
-            // Check dimming
-            if (element.deviceObject.dimming === undefined) {
-              $("#tabs").tabs("disable", "#tabs-2");
-              $("#divColorsAtSwitchOn").hide();
-              $("#divColorCycle").hide();
-            }
+
             // Check color
             if (element.deviceObject.color_temperature === undefined) $("#tabs").tabs("disable", "#tabs-3");
-            if (element.deviceObject.color === undefined) {
+            if (element.deviceObject.color === undefined || JSON.stringify(deviceObject.color) === "{}") {
               $("#tabs").tabs("disable", "#tabs-4");
               $("#divColorsAtSwitchOn").hide();
               $("#divColorCycle").hide();
+              $("#node-input-specifySwitchOnBrightness").empty().append(
+                $("<option>")
+                  .val("no")
+                  .text("Last status")
+              ).append(
+                $("<option>")
+                  .val("temperature")
+                  .text("Select temperature and brightness")
+              );
+              $("#node-input-specifySwitchOnBrightness").val(this.specifySwitchOnBrightness).trigger('change');
+
+              $("#node-input-enableDayNightLighting").empty().append(
+                $("<option>")
+                  .val("no")
+                  .text("Last status")
+              ).append(
+                $("<option>")
+                  .val("temperature")
+                  .text("Select temperature and brightness")
+              );
+              $("#node-input-enableDayNightLighting").val(this.enableDayNightLighting).trigger('change');
+            }
+
+            // Check dimming
+            if (element.deviceObject.dimming === undefined) {
+              $("#tabs").tabs("disable", "#tabs-2");
+              $("#divColorsAtSwitchOn").hide();
+              $("#divColorCycle").hide();
+              $("#divUpdateKNXBrightnessStatusOnHUEOnOff").hide();
+              $("#divCCSBoxAtNightLighting").hide();
+              $("#node-input-specifySwitchOnBrightness").val("no");
+              $("#node-input-enableDayNightLighting").val("no");
+              $("#divMinMaxBrightness").hide();
+              $("#node-input-specifySwitchOnBrightness").empty().append(
+                $("<option>")
+                  .val("no")
+                  .text("Last status")
+              )
+              $("#node-input-specifySwitchOnBrightness").val(this.specifySwitchOnBrightness).trigger('change');
+
+              $("#node-input-enableDayNightLighting").empty().append(
+                $("<option>")
+                  .val("no")
+                  .text("Last status")
+              )
             }
+            $("#node-input-enableDayNightLighting").val(this.enableDayNightLighting).trigger('change');
+
             // Check if grouped, to hide/show the "Get current" buttons
             if (element.deviceObject.type === "grouped_light") {
               $("#getColorAtSwitchOnDayTimeButton").hide();
@@ -476,7 +523,9 @@
       this.colorAtSwitchOnNightTime = this.colorAtSwitchOnNightTime.replace("geen", "green"); // Old bug in "geen" property
       try {
         json = JSON.parse(this.colorAtSwitchOnDayTime);
-      } catch (error) { }
+      } catch (error) {
+        console.log("json = JSON.parse(this.colorAtSwitchOnDayTime) in HTML: " + error.message)
+      }
       if (json !== undefined && json.kelvin !== undefined) {
         // Kelvin
         $("#comboTemperatureAtSwitchOn").val(json.kelvin);
@@ -566,6 +615,9 @@
     }
   });
 
+
+
+
   function rgbHex(red, green, blue, alpha) {
     const toHex = (red, green, blue, alpha) => ((blue | green << 8 | red << 16) | 1 << 24).toString(16).slice(1) + alpha;
     const parseCssRgbString = (input) => {
@@ -937,7 +989,7 @@
         <option value="yes">Yes, and emit KNX telegrams.</option>
       </select>
     </div>
-    <div class="form-row">
+    <div class="form-row" id="divUpdateKNXBrightnessStatusOnHUEOnOff">
       <label style="width:260px;" for="node-input-updateKNXBrightnessStatusOnHUEOnOff">
         <i class="fa fa-tag"></i> KNX Brightness Status
       </label>
@@ -1027,19 +1079,21 @@
       </label>
       <input type="text" id="node-input-dimSpeed" placeholder='Default is 5000' style="width:260px">
     </div>
-    <div class="form-row">
-      <label for="node-input-minDimLevelLight" style="width:260px;">
-        <i class="fa fa-clone"></i> Min Dim Brightness
-      </label>
-      <select id="node-input-minDimLevelLight">
-        <option value="useHueLightLevel">Use minimum brightness specified in the HUE light</option>
-      </select>
-    </div>
-    <div class="form-row">
-      <label for="node-input-maxDimLevelLight" style="width:260px;">
-        <i class="fa fa-clone"></i> Max Dim Brightness
-      </label>
-      <select id="node-input-maxDimLevelLight"></select>
+    <div id ="divMinMaxBrightness">
+      <div class="form-row">
+        <label for="node-input-minDimLevelLight" style="width:260px;">
+          <i class="fa fa-clone"></i> Min Dim Brightness
+        </label>
+        <select id="node-input-minDimLevelLight">
+          <option value="useHueLightLevel">Use minimum brightness specified in the HUE light</option>
+        </select>
+      </div>
+      <div class="form-row">
+        <label for="node-input-maxDimLevelLight" style="width:260px;">
+          <i class="fa fa-clone"></i> Max Dim Brightness
+        </label>
+        <select id="node-input-maxDimLevelLight"></select>
+      </div>
     </div>
     <div class="form-row">
       <label for="node-input-enableNodePINS" style="width:260px;">

package/nodes/knxUltimateHueLight.js

@@ -481,7 +481,7 @@ module.exports = function (RED) {
     node.handleSendHUE = (_event) => {
       try {
         if (_event.id === node.hueDevice) {
-          if (node.currentHUEDevice === undefined) {
+          if (node.currentHUEDevice === undefined || node.serverHue === null || node.serverHue === undefined) {
             node.setNodeStatusHue({
               fill: "red",
               shape: "ring",
@@ -494,9 +494,15 @@ module.exports = function (RED) {
           // Output the msg to the flow
           node.send(_event);
 
+          // // DEBUG
+          //delete _event.dimming;
+          //delete _event.color;
+          //delete _event.color_temperature;
+          //delete _event.color_temperature_delta;
+
           // As grouped_light doesn't contain all requested properties, i find the first light in the group, and use this below in the code
           // If the event type is grouped light, and there are missing properties, i infer these missing properties from the first light in the group!
-          if ((_event.hasOwnProperty("color") || _event.hasOwnProperty("dimming")) && _event.type === 'grouped_light') {
+          if ((_event.color !== undefined || _event.dimming !== undefined || _event.color_temperature !== undefined) && _event.type === 'grouped_light') {
             try {
               const firstLightInGroup = node.serverHue.getFirstLightInGroup(_event.id);
               if (firstLightInGroup !== null && firstLightInGroup !== undefined) {
@@ -513,17 +519,19 @@ module.exports = function (RED) {
           if (_event.hasOwnProperty("on")) {
             node.updateKNXLightState(_event.on.on);
             // In case of switch off, set the dim to zero
-            if (
-              _event.on.on === false
-              && (config.updateKNXBrightnessStatusOnHUEOnOff === undefined || config.updateKNXBrightnessStatusOnHUEOnOff === "onhueoff")
-            ) {
+            if (_event.on.on === false && (config.updateKNXBrightnessStatusOnHUEOnOff === undefined || config.updateKNXBrightnessStatusOnHUEOnOff === "onhueoff")) {
               node.updateKNXBrightnessState(0);
               //node.currentHUEDevice.dimming.brightness = 0;
+            } else if (_event.on.on === true && node.currentHUEDevice.on.on === false) {
+              // Turn on always update the dimming KNX Status value as well.
+              let brightVal = 100;
+              if (node.currentHUEDevice.dimming !== undefined && node.currentHUEDevice.dimming.brightness !== undefined) brightVal = node.currentHUEDevice.dimming.brightness;
+              node.updateKNXBrightnessState(brightVal);
             }
             node.currentHUEDevice.on.on = _event.on.on;
           }
 
-          if (_event.hasOwnProperty("color")) {
+          if (_event.color !== undefined) { // fixed https://github.com/Supergiovane/node-red-contrib-knx-ultimate/issues/287
             node.updateKNXLightColorState(_event.color);
             node.currentHUEDevice.color = _event.color;
           }
@@ -658,33 +666,36 @@ module.exports = function (RED) {
 
     node.updateKNXLightColorState = function updateKNXLightColorState(_value, _outputtype = "write") {
       if (config.GALightColorState !== undefined && config.GALightColorState !== "") {
+        if (!_value.hasOwnProperty('xy') || _value.xy.x === undefined) return;
         const knxMsgPayload = {};
         knxMsgPayload.topic = config.GALightColorState;
         knxMsgPayload.dpt = config.dptLightColorState;
-        knxMsgPayload.payload = hueColorConverter.ColorConverter.xyBriToRgb(
-          _value.xy.x,
-          _value.xy.y,
-          node.currentHUEDevice !== undefined ? node.currentHUEDevice.dimming.brightness : 100,
-        );
-        knxMsgPayload.payload = { red: knxMsgPayload.payload.r, green: knxMsgPayload.payload.g, blue: knxMsgPayload.payload.b };
-        // Send to KNX bus
-        if (knxMsgPayload.topic !== "" && knxMsgPayload.topic !== undefined) {
-          if (node.server !== null && node.server !== undefined) {
-            node.server.writeQueueAdd({
-              grpaddr: knxMsgPayload.topic,
-              payload: knxMsgPayload.payload,
-              dpt: knxMsgPayload.dpt,
-              outputtype: _outputtype,
-              nodecallerid: node.id,
-            });
+        try {
+          knxMsgPayload.payload = hueColorConverter.ColorConverter.xyBriToRgb(
+            _value.xy.x,
+            _value.xy.y,
+            node.currentHUEDevice !== undefined && node.currentHUEDevice.dimming !== undefined && node.currentHUEDevice.dimming.brightness === undefined ? node.currentHUEDevice.dimming.brightness : 100,
+          );
+          knxMsgPayload.payload = { red: knxMsgPayload.payload.r, green: knxMsgPayload.payload.g, blue: knxMsgPayload.payload.b };
+          // Send to KNX bus
+          if (knxMsgPayload.topic !== "" && knxMsgPayload.topic !== undefined) {
+            if (node.server !== null && node.server !== undefined) {
+              node.server.writeQueueAdd({
+                grpaddr: knxMsgPayload.topic,
+                payload: knxMsgPayload.payload,
+                dpt: knxMsgPayload.dpt,
+                outputtype: _outputtype,
+                nodecallerid: node.id,
+              });
+            }
           }
-        }
-        node.setNodeStatusHue({
-          fill: "blue",
-          shape: "ring",
-          text: "HUE->KNX Color",
-          payload: knxMsgPayload.payload,
-        });
+          node.setNodeStatusHue({
+            fill: "blue",
+            shape: "ring",
+            text: "HUE->KNX Color",
+            payload: knxMsgPayload.payload,
+          });
+        } catch (error) { }
       }
     };
 

package/nodes/knxUltimateHueLightSensor.html

@@ -13,7 +13,7 @@
             namelightsensor: { value: "" },
             GAlightsensor: { value: "" },
             dptlightsensor: { value: "" },
-            readStatusAtStartup: { value: "no" },
+            readStatusAtStartup: { value: "yes" },
 
             hueDevice: { value: "" }
         },

package/nodes/knxUltimateHueLightSensor.js

@@ -7,7 +7,7 @@ module.exports = function (RED) {
     node.topic = node.name;
     node.name = config.name === undefined ? 'Hue' : config.name;
     node.dpt = '';
-    node.notifyreadrequest = false;
+    node.notifyreadrequest = true;
     node.notifyreadrequestalsorespondtobus = 'false';
     node.notifyreadrequestalsorespondtobusdefaultvalueifnotinitialized = '';
     node.notifyresponse = false;
@@ -23,7 +23,8 @@ module.exports = function (RED) {
     node.formatnegativevalue = 'leave';
     node.formatdecimalsvalue = 2;
     node.hueDevice = config.hueDevice;
-    node.initializingAtStart = !((config.readStatusAtStartup === undefined || config.readStatusAtStartup === "no"));
+    node.initializingAtStart = !((config.readStatusAtStartup === undefined || config.readStatusAtStartup === "yes"));
+    node.currentDeviceValue = 0;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({ fill, shape, text, payload }) => {
@@ -39,6 +40,19 @@ module.exports = function (RED) {
 
     // This function is called by the knx-ultimate config node, to output a msg.payload.
     node.handleSend = msg => {
+
+      // Respond to KNX read telegram, by sending the current value as response telegram.
+      if (msg.knx.event === "GroupValue_Read") {
+        switch (msg.knx.destination) {
+          case config.GAlightsensor:
+            // To the KNX bus wires
+            node.sendResponseToKNX(node.currentDeviceValue);
+            break;
+          default:
+            break;
+        }
+      }
+
     };
 
     node.handleSendHUE = _event => {
@@ -57,6 +71,8 @@ module.exports = function (RED) {
             knxMsgPayload.payload = _event.light.light_level === 0 ? 0 : Math.round(Math.pow(10, (_event.light.light_level - 1) / 10000));
             // Send to KNX bus
             if (knxMsgPayload.topic !== '' && knxMsgPayload.topic !== undefined) node.server.writeQueueAdd({ grpaddr: knxMsgPayload.topic, payload: knxMsgPayload.payload, dpt: knxMsgPayload.dpt, outputtype: 'write', nodecallerid: node.id });
+            node.currentDeviceValue = knxMsgPayload.payload;
+
             node.status({ fill: 'green', shape: 'dot', text: 'HUE->KNX ' + JSON.stringify(knxMsgPayload.payload) + ' (' + new Date().getDate() + ', ' + new Date().toLocaleTimeString() + ')' });
 
             // Setup the output msg
@@ -74,6 +90,20 @@ module.exports = function (RED) {
       }
     };
 
+    node.sendResponseToKNX = (_level) => {
+      const knxMsgPayload = {};
+      knxMsgPayload.topic = config.GAlightsensor;
+      knxMsgPayload.dpt = config.dptlightsensor;
+
+      knxMsgPayload.payload = _level;
+      // Send to KNX bus
+      if (knxMsgPayload.topic !== '' && knxMsgPayload.topic !== undefined) {
+        node.server.writeQueueAdd({
+          grpaddr: knxMsgPayload.topic, payload: knxMsgPayload.payload, dpt: knxMsgPayload.dpt, outputtype: 'response', nodecallerid: node.id,
+        });
+      }
+    };
+
     // On each deploy, unsubscribe+resubscribe
     if (node.server) {
       node.server.removeClient(node);

package/nodes/knxUltimateHueTemperatureSensor.html

@@ -13,7 +13,7 @@
             nametemperaturesensor: { value: "" },
             GAtemperaturesensor: { value: "" },
             dpttemperaturesensor: { value: "" },
-            readStatusAtStartup: { value: "no" },
+            readStatusAtStartup: { value: "yes" },
 
             hueDevice: { value: "" }
         },

package/nodes/knxUltimateHueTemperatureSensor.js

@@ -7,7 +7,7 @@ module.exports = function (RED) {
     node.topic = node.name;
     node.name = config.name === undefined ? 'Hue' : config.name;
     node.dpt = '';
-    node.notifyreadrequest = false;
+    node.notifyreadrequest = true;
     node.notifyreadrequestalsorespondtobus = 'false';
     node.notifyreadrequestalsorespondtobusdefaultvalueifnotinitialized = '';
     node.notifyresponse = false;
@@ -23,7 +23,8 @@ module.exports = function (RED) {
     node.formatnegativevalue = 'leave';
     node.formatdecimalsvalue = 2;
     node.hueDevice = config.hueDevice;
-    node.initializingAtStart = !((config.readStatusAtStartup === undefined || config.readStatusAtStartup === "no"));
+    node.initializingAtStart = !((config.readStatusAtStartup === undefined || config.readStatusAtStartup === "yes"));
+    node.currentDeviceValue = 0;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({
@@ -43,6 +44,17 @@ module.exports = function (RED) {
 
     // This function is called by the knx-ultimate config node, to output a msg.payload.
     node.handleSend = (msg) => {
+      // Respond to KNX read telegram, by sending the current value as response telegram.
+      if (msg.knx.event === "GroupValue_Read") {
+        switch (msg.knx.destination) {
+          case config.GAtemperaturesensor:
+            // To the KNX bus wires
+            node.sendResponseToKNX(node.currentDeviceValue);
+            break;
+          default:
+            break;
+        }
+      }
     };
 
     node.handleSendHUE = (_event) => {
@@ -60,6 +72,8 @@ module.exports = function (RED) {
                 grpaddr: knxMsgPayload.topic, payload: knxMsgPayload.payload, dpt: knxMsgPayload.dpt, outputtype: 'write', nodecallerid: node.id,
               });
             }
+            node.currentDeviceValue = knxMsgPayload.payload;
+
             node.status({ fill: 'green', shape: 'dot', text: `HUE->KNX ${JSON.stringify(knxMsgPayload.payload)} (${new Date().getDate()}, ${new Date().toLocaleTimeString()})` });
 
             // Setup the output msg
@@ -79,6 +93,22 @@ module.exports = function (RED) {
       }
     };
 
+
+    node.sendResponseToKNX = (_level) => {
+      const knxMsgPayload = {};
+      knxMsgPayload.topic = config.GAtemperaturesensor;
+      knxMsgPayload.dpt = config.dpttemperaturesensor;
+
+      knxMsgPayload.payload = _level;
+      // Send to KNX bus
+      if (knxMsgPayload.topic !== '' && knxMsgPayload.topic !== undefined) {
+        node.server.writeQueueAdd({
+          grpaddr: knxMsgPayload.topic, payload: knxMsgPayload.payload, dpt: knxMsgPayload.dpt, outputtype: 'response', nodecallerid: node.id,
+        });
+      }
+    };
+
+
     // On each deploy, unsubscribe+resubscribe
     if (node.server) {
       node.server.removeClient(node);

package/package.json

@@ -3,7 +3,7 @@
   "engines": {
     "node": ">=16.0.0"
   },
-  "version": "2.2.25",
+  "version": "2.2.26",
   "description": "Control your KNX intallation via Node-Red! Single Node KNX IN/OUT with optional ETS group address importer. Easy to use and highly configurable. With integrated Philips HUE devices control.",
   "dependencies": {
     "binary-parser": "2.2.1",