node-red-contrib-knx-ultimate 2.2.4 → 2.2.5

package/CHANGELOG.md

@@ -6,6 +6,9 @@
 
 # CHANGELOG
 
+<b>Version 2.2.5</b> - October 2023<br/>
+- Fix: fixed some HUE nodes not able to register to the event notification service.<br/>
+</p>
 <b>Version 2.2.4</b> - October 2023<br/>
 - HUE Light: fixed some status hiccups and better handling of async hue bridge functions.<br/>
 </p>
@@ -765,7 +768,7 @@
 </p>
 <p>
 <b>Version 1.2.33</b> - May 2021<br/>
-- Gateway configuration -> Advanced Options -> Node list in all flows: added more infos to each node in the list, to allow more control on the nodes overview. The options are: "No Initial Read, React to Write, React to Response, No React to Read, No Autorespond to Read Requests, Output type write, No RBE on Output to Bus, No RBE on Input from Bus"<br/>
+- Gateway configuration -> Advanced Options -> Node list in all flows: added more infos to each node in the list, to allow more control on the nodes overview. The options are: "No Initial Read, React to Write, React to Response, No React to Read, No Autorespond to Read Requests, Telegram type write, No RBE on Output to Bus, No RBE on Input from Bus"<br/>
 </p>
 <p>
 <b>Version 1.2.32</b> - May 2021<br/>
@@ -1067,7 +1070,7 @@
 </p>
 <p>
 <b>Version 1.1.64</b> - April 2020<br/>
-- NEW: Added output type "Read", to issue a read by simply pass a payload to the node. Thanks @waldbaer for the suggestion.<br/>
+- NEW: Added Telegram type "Read", to issue a read by simply pass a payload to the node. Thanks @waldbaer for the suggestion.<br/>
 </p>
 <p>
 <b>Version 1.1.63</b> - April 2020<br/>
@@ -1304,7 +1307,7 @@
 </p>
 <p>
 <b>Version 1.1.10</b><br/>
-- Auto send node value as response to the KNX Bus. It works in conjunction with React to event GroupValue read. When checked, whenever the node receives a read request from bus, it sends a response to the KNX Bus with the stored payload value.<br/>
+- Auto send node value as response to the KNX Bus. It works in conjunction with React to read telegrams. When checked, whenever the node receives a read request from bus, it sends a response to the KNX Bus with the stored payload value.<br/>
 - Fixed an issue where if you have a node set to Universal mode (listen to all Group Addresses) (with ETS CSV File set) and you create a new node having a Group Addr. not in the ETS CSV file, an exception is raised but not caught and the nodes may not receive the values from KNX BUS.<br/>
 </p>
 <p>
@@ -1413,7 +1416,7 @@
 </p>
 <p>
 <b>Version 0.0.6 BETA</b><br/>
-- Fixed Output Type unable to be set<br/>
+- Fixed Telegram type unable to be set<br/>
 - Added node status for response (blue) and read (grey)<br/>
 </p>
 <p>

package/nodes/hue-config.js

@@ -122,6 +122,12 @@ module.exports = (RED) => {
             if (nodeClient.hueDevice !== undefined) {
               const oHUEDevice = node.hueAllResources.filter((a) => a.id === nodeClient.hueDevice)[0];
               if (oHUEDevice !== undefined) {
+                // Add _Node to the clients array
+                nodeClient.setNodeStatusHue({
+                  fill: "green",
+                  shape: "ring",
+                  text: "Ready :-)",
+                });
                 nodeClient.currentHUEDevice = oHUEDevice;
                 nodeClient.handleSendHUE(oHUEDevice);
               }

package/nodes/knxUltimate-config.js

@@ -74,8 +74,8 @@ module.exports = (RED) => {
         help: `// KNX-Ultimate set as UNIVERSAL NODE
 // Example of a function that sends a message to the KNX-Ultimate
 msg.destination = "0/0/1"; // Set the destination 
-msg.payload = false; // issues a write or response (based on the options Output Type above) to the KNX bus
-msg.event = "GroupValue_Write"; // "GroupValue_Write" or "GroupValue_Response", overrides the option Output Type above.
+msg.payload = false; // issues a write or response (based on the options Telegram type above) to the KNX bus
+msg.event = "GroupValue_Write"; // "GroupValue_Write" or "GroupValue_Response", overrides the option Telegram type above.
 msg.dpt = "1.001"; // for example "1.001", overrides the Datapoint option. (Datapoints can be sent as 9 , "9" , "9.001" or "DPT9.001")
 return msg;`,
         helplink: "https://github.com/Supergiovane/node-red-contrib-knx-ultimate/wiki",

package/nodes/knxUltimate.html

@@ -337,10 +337,7 @@
     <br />
 
     <label for="node-input-server">
-        <span id="divknxsecure">
-            <font color="green" size="4px"><i class="fa fa-shield" aria-hidden="true"></i></font>
-        </span>&nbsp;<span><i class="fa fa-circle-o" aria-hidden="true"></i></font></span>
-        <span data-i18n="knxUltimate.properties.node-input-server"></span>
+        <i class="fa fa-circle-o"></i> Gateway
     </label>
     <input type="text" id="node-input-server" />
 </div>
@@ -354,17 +351,16 @@
                     src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAwAAAAMCAYAAABWdVznAAAACXBIWXMAAB7CAAAewgFu0HU+AAAF6WlUWHRYTUw6Y29tLmFkb2JlLnhtcAAAAAAAPD94cGFja2V0IGJlZ2luPSLvu78iIGlkPSJXNU0wTXBDZWhpSHpyZVN6TlRjemtjOWQiPz4gPHg6eG1wbWV0YSB4bWxuczp4PSJhZG9iZTpuczptZXRhLyIgeDp4bXB0az0iQWRvYmUgWE1QIENvcmUgNS42LWMxNDggNzkuMTY0MDM2LCAyMDE5LzA4LzEzLTAxOjA2OjU3ICAgICAgICAiPiA8cmRmOlJERiB4bWxuczpyZGY9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkvMDIvMjItcmRmLXN5bnRheC1ucyMiPiA8cmRmOkRlc2NyaXB0aW9uIHJkZjphYm91dD0iIiB4bWxuczp4bXA9Imh0dHA6Ly9ucy5hZG9iZS5jb20veGFwLzEuMC8iIHhtbG5zOmRjPSJodHRwOi8vcHVybC5vcmcvZGMvZWxlbWVudHMvMS4xLyIgeG1sbnM6cGhvdG9zaG9wPSJodHRwOi8vbnMuYWRvYmUuY29tL3Bob3Rvc2hvcC8xLjAvIiB4bWxuczp4bXBNTT0iaHR0cDovL25zLmFkb2JlLmNvbS94YXAvMS4wL21tLyIgeG1sbnM6c3RFdnQ9Imh0dHA6Ly9ucy5hZG9iZS5jb20veGFwLzEuMC9zVHlwZS9SZXNvdXJjZUV2ZW50IyIgeG1wOkNyZWF0b3JUb29sPSJBZG9iZSBQaG90b3Nob3AgMjEuMSAoTWFjaW50b3NoKSIgeG1wOkNyZWF0ZURhdGU9IjIwMjAtMDMtMjNUMTY6MjE6MDkrMDE6MDAiIHhtcDpNb2RpZnlEYXRlPSIyMDIwLTAzLTIzVDE2OjI4OjI3KzAxOjAwIiB4bXA6TWV0YWRhdGFEYXRlPSIyMDIwLTAzLTIzVDE2OjI4OjI3KzAxOjAwIiBkYzpmb3JtYXQ9ImltYWdlL3BuZyIgcGhvdG9zaG9wOkNvbG9yTW9kZT0iMyIgcGhvdG9zaG9wOklDQ1Byb2ZpbGU9InNSR0IgSUVDNjE5NjYtMi4xIiB4bXBNTTpJbnN0YW5jZUlEPSJ4bXAuaWlkOjc4M2I5OWIxLTkwMjYtNGQ2OC05N2FmLTRkM2E2ZWY4Zjk2OCIgeG1wTU06RG9jdW1lbnRJRD0ieG1wLmRpZDo4N2E3YTg0NS0xMDljLTRkMTMtOGEzOS04OWVhOTMyMDQ0ZWMiIHhtcE1NOk9yaWdpbmFsRG9jdW1lbnRJRD0ieG1wLmRpZDo4N2E3YTg0NS0xMDljLTRkMTMtOGEzOS04OWVhOTMyMDQ0ZWMiPiA8eG1wTU06SGlzdG9yeT4gPHJkZjpTZXE+IDxyZGY6bGkgc3RFdnQ6YWN0aW9uPSJjcmVhdGVkIiBzdEV2dDppbnN0YW5jZUlEPSJ4bXAuaWlkOjg3YTdhODQ1LTEwOWMtNGQxMy04YTM5LTg5ZWE5MzIwNDRlYyIgc3RFdnQ6d2hlbj0iMjAyMC0wMy0yM1QxNjoyMTowOSswMTowMCIgc3RFdnQ6c29mdHdhcmVBZ2VudD0iQWRvYmUgUGhvdG9zaG9wIDIxLjEgKE1hY2ludG9zaCkiLz4gPHJkZjpsaSBzdEV2dDphY3Rpb249InNhdmVkIiBzdEV2dDppbnN0YW5jZUlEPSJ4bXAuaWlkOjc4M2I5OWIxLTkwMjYtNGQ2OC05N2FmLTRkM2E2ZWY4Zjk2OCIgc3RFdnQ6d2hlbj0iMjAyMC0wMy0yM1QxNjoyODoyNyswMTowMCIgc3RFdnQ6c29mdHdhcmVBZ2VudD0iQWRvYmUgUGhvdG9zaG9wIDIxLjEgKE1hY2ludG9zaCkiIHN0RXZ0OmNoYW5nZWQ9Ii8iLz4gPC9yZGY6U2VxPiA8L3htcE1NOkhpc3Rvcnk+IDwvcmRmOkRlc2NyaXB0aW9uPiA8L3JkZjpSREY+IDwveDp4bXBtZXRhPiA8P3hwYWNrZXQgZW5kPSJyIj8+DGyPFQAAAE9JREFUKJG1UEEOACAIgub/v2yHalnJVoe4uIEKSgccJ9jroVmjA0+ujZtWku3DgZmgBiT4egN9CmmEAAfA/5HUW0OQu7dKmOCzmM3k9YYKZv8ZEZ/YgNsAAAAASUVORK5CYII="></img>
                 <span data-i18n="knxUltimate.properties.node-input-topic"></span>
             </label>
-            <input style="width:60%" type="text" id="node-input-topic" placeholder="Ex: 1/1/1" />
+            <input type="text" id="node-input-topic" placeholder="Sample: Dinning light" />
         </div>
         <div class="form-row">
             <label for="node-input-dpt">
                 <img
                     src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAKRGlDQ1BJQ0MgUHJvZmlsZQAAeAGdlndUFNcXx9/MbC+0XZYiZem9twWkLr1IlSYKy+4CS1nWZRewN0QFIoqICFYkKGLAaCgSK6JYCAgW7AEJIkoMRhEVlczGHPX3Oyf5/U7eH3c+8333nnfn3vvOGQAoASECYQ6sAEC2UCKO9PdmxsUnMPG9AAZEgAM2AHC4uaLQKL9ogK5AXzYzF3WS8V8LAuD1LYBaAK5bBIQzmX/p/+9DkSsSSwCAwtEAOx4/l4tyIcpZ+RKRTJ9EmZ6SKWMYI2MxmiDKqjJO+8Tmf/p8Yk8Z87KFPNRHlrOIl82TcRfKG/OkfJSREJSL8gT8fJRvoKyfJc0WoPwGZXo2n5MLAIYi0yV8bjrK1ihTxNGRbJTnAkCgpH3FKV+xhF+A5gkAO0e0RCxIS5cwjbkmTBtnZxYzgJ+fxZdILMI53EyOmMdk52SLOMIlAHz6ZlkUUJLVlokW2dHG2dHRwtYSLf/n9Y+bn73+GWS9/eTxMuLPnkGMni/al9gvWk4tAKwptDZbvmgpOwFoWw+A6t0vmv4+AOQLAWjt++p7GLJ5SZdIRC5WVvn5+ZYCPtdSVtDP6386fPb8e/jqPEvZeZ9rx/Thp3KkWRKmrKjcnKwcqZiZK+Jw+UyL/x7ifx34VVpf5WEeyU/li/lC9KgYdMoEwjS03UKeQCLIETIFwr/r8L8M+yoHGX6aaxRodR8BPckSKPTRAfJrD8DQyABJ3IPuQJ/7FkKMAbKbF6s99mnuUUb3/7T/YeAy9BXOFaQxZTI7MprJlYrzZIzeCZnBAhKQB3SgBrSAHjAGFsAWOAFX4Al8QRAIA9EgHiwCXJAOsoEY5IPlYA0oAiVgC9gOqsFeUAcaQBM4BtrASXAOXARXwTVwE9wDQ2AUPAOT4DWYgSAID1EhGqQGaUMGkBlkC7Egd8gXCoEioXgoGUqDhJAUWg6tg0qgcqga2g81QN9DJ6Bz0GWoH7oDDUPj0O/QOxiBKTAd1oQNYSuYBXvBwXA0vBBOgxfDS+FCeDNcBdfCR+BW+Bx8Fb4JD8HP4CkEIGSEgeggFggLYSNhSAKSioiRlUgxUonUIk1IB9KNXEeGkAnkLQaHoWGYGAuMKyYAMx/DxSzGrMSUYqoxhzCtmC7MdcwwZhLzEUvFamDNsC7YQGwcNg2bjy3CVmLrsS3YC9ib2FHsaxwOx8AZ4ZxwAbh4XAZuGa4UtxvXjDuL68eN4KbweLwa3gzvhg/Dc/ASfBF+J/4I/gx+AD+Kf0MgE7QJtgQ/QgJBSFhLqCQcJpwmDBDGCDNEBaIB0YUYRuQRlxDLiHXEDmIfcZQ4Q1IkGZHcSNGkDNIaUhWpiXSBdJ/0kkwm65KdyRFkAXk1uYp8lHyJPEx+S1GimFLYlESKlLKZcpBylnKH8pJKpRpSPakJVAl1M7WBep76kPpGjiZnKRcox5NbJVcj1yo3IPdcnihvIO8lv0h+qXyl/HH5PvkJBaKCoQJbgaOwUqFG4YTCoMKUIk3RRjFMMVuxVPGw4mXFJ0p4JUMlXyWeUqHSAaXzSiM0hKZHY9O4tHW0OtoF2igdRzeiB9Iz6CX07+i99EllJWV75RjlAuUa5VPKQwyEYcgIZGQxyhjHGLcY71Q0VbxU+CqbVJpUBlSmVeeoeqryVYtVm1Vvqr5TY6r5qmWqbVVrU3ugjlE3VY9Qz1ffo35BfWIOfY7rHO6c4jnH5tzVgDVMNSI1lmkc0OjRmNLU0vTXFGnu1DyvOaHF0PLUytCq0DqtNa5N03bXFmhXaJ/RfspUZnoxs5hVzC7mpI6GToCOVGe/Tq/OjK6R7nzdtbrNug/0SHosvVS9Cr1OvUl9bf1Q/eX6jfp3DYgGLIN0gx0G3QbThkaGsYYbDNsMnxipGgUaLTVqNLpvTDX2MF5sXGt8wwRnwjLJNNltcs0UNnUwTTetMe0zg80czQRmu836zbHmzuZC81rzQQuKhZdFnkWjxbAlwzLEcq1lm+VzK32rBKutVt1WH60drLOs66zv2SjZBNmstemw+d3W1JZrW2N7w45q52e3yq7d7oW9mT3ffo/9bQeaQ6jDBodOhw+OTo5ixybHcSd9p2SnXU6DLDornFXKuuSMdfZ2XuV80vmti6OLxOWYy2+uFq6Zroddn8w1msufWzd3xE3XjeO2323Ineme7L7PfchDx4PjUevxyFPPk+dZ7znmZeKV4XXE67m3tbfYu8V7mu3CXsE+64P4+PsU+/T6KvnO9632fein65fm1+g36e/gv8z/bAA2IDhga8BgoGYgN7AhcDLIKWhFUFcwJTgquDr4UYhpiDikIxQODQrdFnp/nsE84by2MBAWGLYt7EG4Ufji8B8jcBHhETURjyNtIpdHdkfRopKiDke9jvaOLou+N994vnR+Z4x8TGJMQ8x0rE9seexQnFXcirir8erxgvj2BHxCTEJ9wtQC3wXbF4wmOiQWJd5aaLSwYOHlReqLshadSpJP4iQdT8YmxyYfTn7PCePUcqZSAlN2pUxy2dwd3Gc8T14Fb5zvxi/nj6W6pZanPklzS9uWNp7ukV6ZPiFgC6oFLzICMvZmTGeGZR7MnM2KzWrOJmQnZ58QKgkzhV05WjkFOf0iM1GRaGixy+LtiyfFweL6XCh3YW67hI7+TPVIjaXrpcN57nk1eW/yY/KPFygWCAt6lpgu2bRkbKnf0m+XYZZxl3Uu11m+ZvnwCq8V+1dCK1NWdq7SW1W4anS1/+pDa0hrMtf8tNZ6bfnaV+ti13UUahauLhxZ77++sUiuSFw0uMF1w96NmI2Cjb2b7Dbt3PSxmFd8pcS6pLLkfSm39Mo3Nt9UfTO7OXVzb5lj2Z4tuC3CLbe2emw9VK5YvrR8ZFvottYKZkVxxavtSdsvV9pX7t1B2iHdMVQVUtW+U3/nlp3vq9Orb9Z41zTv0ti1adf0bt7ugT2ee5r2au4t2ftun2Df7f3++1trDWsrD+AO5B14XBdT1/0t69uGevX6kvoPB4UHhw5FHupqcGpoOKxxuKwRbpQ2jh9JPHLtO5/v2pssmvY3M5pLjoKj0qNPv0/+/tax4GOdx1nHm34w+GFXC62luBVqXdI62ZbeNtQe395/IuhEZ4drR8uPlj8ePKlzsuaU8qmy06TThadnzyw9M3VWdHbiXNq5kc6kznvn487f6Iro6r0QfOHSRb+L57u9us9ccrt08rLL5RNXWFfarjpebe1x6Gn5yeGnll7H3tY+p772a87XOvrn9p8e8Bg4d93n+sUbgTeu3px3s//W/Fu3BxMHh27zbj+5k3Xnxd28uzP3Vt/H3i9+oPCg8qHGw9qfTX5uHnIcOjXsM9zzKOrRvRHuyLNfcn95P1r4mPq4ckx7rOGJ7ZOT437j154ueDr6TPRsZqLoV8Vfdz03fv7Db56/9UzGTY6+EL+Y/b30pdrLg6/sX3VOhU89fJ39ema6+I3am0NvWW+738W+G5vJf49/X/XB5EPHx+CP92ezZ2f/AAOY8/xJsCmYAAAACXBIWXMAAAsTAAALEwEAmpwYAAAAfklEQVQ4EaWTwQnAIAxFU3AYnclVHNQtFPTgrcWC8JNL2sSL+fjz1BiJnOM6+THG+8R/5oDmUgpKyjkzLUVKiRhgrSU9qmaAOaeaIA0MMMaQ66pmgN67miANAatvAiCxtYbyUxxqrW8v7JOYALiN+wpugPsZLY3k/kxYQ1P8AA+YKsRd6CkwAAAAAElFTkSuQmCC"></img>
-                <span data-i18n="knxUltimate.properties.node-input-dpt"></span>
+                Datapoint
             </label>
-            <select style="width:40%" id="node-input-dpt"></select>
+            <select id="node-input-dpt"></select>
             <span id="sampleCodeEditor" style="color:red;" />
-
         </div>
     </div>
     <div class="form-row">
@@ -395,45 +391,30 @@
 
     <div id="tabs">
         <ul>
-            <li><a href="#tabs-1"><i class="fa fa-code"></i> Sample in Function</a></li>
-            <li><a href="#tabs-2"><i class="fa fa-braille"></i> Advanced options</a></li>
-            <li><a href="#tabs-3"><i class="fa fa-list-ol"></i> Format input</a></li>
+            <li><a href="#tabs-1"><i class="fa fa-braille"></i> Advanced options</a></li>
+            <li><a href="#tabs-2"><i class="fa fa-list-ol"></i> Format input</a></li>
+            <li><a href="#tabs-3"><i class="fa fa-code"></i> Payload Sample</a></li>
         </ul>
         <div id="tabs-1">
-            <p>
-            <div style="height:200px;min-height:150px;padding:0px;">
-                <div style="height:100%;width:100%;padding:0px;" class="node-text-editor" id="example-editor"></div>
-            </div>
-            </p>
-        </div>
-
-        <div id="tabs-2">
-            <p>
-
-
             <div class="form-row" style="padding:10px">
                 <div class="form-row">
-                    <label style="width:230px" for="node-input-passthrough">
-                        <i class="fa fa-long-arrow-right"></i>
-                        <span data-i18n="knxUltimate.properties.node-input-passthrough"></span>
+                    <label style="width:180px" for="node-input-passthrough">
+                        <i class="fa fa-long-arrow-right"></i> Msg passthrough
                     </label>
-                    <select style="width:92px" id="node-input-passthrough">
+                    <select id="node-input-passthrough">
                         <option value="no" data-i18n="knxUltimate.selectlists.passthrough_No"></option>
                         <option value="yes" data-i18n="knxUltimate.selectlists.passthrough_Leave"></option>
                         <option value="yesownprop" data-i18n="knxUltimate.selectlists.passthrough_OwnProp"></option>
                     </select>
                 </div>
-                <br />
                 <div class="form-row">
                     <dt>
-                        <i class="fa fa-arrow-right"></i>|
-                        <span data-i18n="knxUltimate.advanced.OUTPUT"></span>
+                        <i class="fa fa-arrow-right"></i> CONTROL (payload to KNX bus)
                     </dt>
                 </div>
                 <div class="form-row">
-                    &nbsp;&nbsp;<label for="node-input-outputtype">
-                        <i class="fa fa-paper-plane-o"></i>
-                        <span data-i18n="knxUltimate.properties.node-input-outputtype"></span>
+                    <label style="width:180px" for="node-input-outputtype">
+                        <i class="fa fa-paper-plane-o"></i> Telegram type
                     </label>
                     <select id="node-input-outputtype">
                         <option value="write" data-i18n="knxUltimate.selectlists.Output_write"></option>
@@ -443,25 +424,23 @@
                     </select>
                 </div>
                 <div class="form-row" id="divOutputRBE">
-                    &nbsp;&nbsp;<input type="checkbox" id="node-input-outputRBE"
+                    <input type="checkbox" id="node-input-outputRBE"
                         style="display:inline-block; width:auto; vertical-align:top;" />
                     <label style="width:85%" for="node-input-outputRBE">
-                        <i class="fa fa-filter"></i>
-                        <span data-i18n="knxUltimate.properties.node-input-outputRBE"></span>
+                        <i class="fa fa-filter"></i> Send payload to KNX only if changed (RBE filter)
                     </label>
                 </div>
                 <br />
                 <div class="form-row">
                     <dt>
-                        |<i class="fa fa-arrow-right"></i>
-                        <span data-i18n="knxUltimate.advanced.INPUT"></span>
+                        |<i class="fa fa-arrow-right"></i> STATUS (payload from KNX BUS)
                     </dt>
                 </div>
                 <div class="form-row" id="divNode-input-initialread">
-                    &nbsp;&nbsp;<label style="width:50%" for="node-input-initialread">
-                        <i class="fa fa-question-circle-o"></i> Read status upon connection
+                    <label style="width:180px" for="node-input-initialread">
+                        <i class="fa fa-question-circle-o"></i> Read status on start
                     </label>
-                    <select style="width:100px" id="node-input-initialread">
+                    <select id="node-input-initialread">
                         <option value=0 data-i18n="knxUltimate.properties.node-input-initialread0"></option>
                         <option value=1 data-i18n="knxUltimate.properties.node-input-initialread1"></option>
                         <option value=2 data-i18n="knxUltimate.properties.node-input-initialread2"></option>
@@ -470,42 +449,38 @@
 
                 </div>
                 <div class="form-row" id="divInputRBE">
-                    &nbsp;&nbsp;<input type="checkbox" id="node-input-inputRBE"
+                    <input type="checkbox" id="node-input-inputRBE"
                         style="display:inline-block; width:auto; vertical-align:top;" />
                     <label style="width:85%" for="node-input-inputRBE">
-                        <i class="fa fa-filter"></i>
-                        <span data-i18n="knxUltimate.properties.node-input-inputRBE"></span>
+                        <i class="fa fa-filter"></i> React only by changed payload (RBE filter)
                     </label>
                 </div>
                 <div class="form-row">
-                    &nbsp;&nbsp;<input type="checkbox" id="node-input-notifywrite"
+                    <input type="checkbox" id="node-input-notifywrite"
                         style="display:inline-block; width:auto; vertical-align:top;" />
                     <label style="width:85%" for="node-input-notifywrite">
-                        <i class="fa fa-sign-in"></i>
-                        <span data-i18n="knxUltimate.properties.node-input-notifywrite"></span>
+                        <i class="fa fa-sign-in"></i> React to write telegrams
                     </label>
                 </div>
                 <div class="form-row">
-                    &nbsp;&nbsp;<input type="checkbox" id="node-input-notifyresponse"
+                    <input type="checkbox" id="node-input-notifyresponse"
                         style="display:inline-block; width:auto; vertical-align:top;" />
                     <label style="width:85%" for="node-input-notifyresponse">
-                        <i class="fa fa-sign-in"></i>
-                        <span data-i18n="knxUltimate.properties.node-input-notifyresponse"></span>
+                        <i class="fa fa-sign-in"></i> React to response telegrams
                     </label>
                 </div>
                 <div class="form-row">
-                    &nbsp;&nbsp;<input type="checkbox" id="node-input-notifyreadrequest"
+                    <input type="checkbox" id="node-input-notifyreadrequest"
                         style="display:inline-block; width:auto; vertical-align:top;" />
                     <label style="width:85%" for="node-input-notifyreadrequest">
-                        <i class="fa fa-sign-in"></i>
-                        <span data-i18n="knxUltimate.properties.node-input-notifyreadrequest"></span>
+                        <i class="fa fa-sign-in"></i> React to read telegrams
                     </label>
                 </div>
 
                 <div id="divnotifyreadrequestautoreact">
                     <dd>
                         <div class="form-row">
-                            &nbsp;&nbsp;<input type="checkbox" id="node-input-notifyreadrequestalsorespondtobus"
+                            <input type="checkbox" id="node-input-notifyreadrequestalsorespondtobus"
                                 style="display:inline-block; width:auto; vertical-align:top;" />
                             <label style="width:85%" for="node-input-notifyreadrequestalsorespondtobus">
                                 <span
@@ -513,7 +488,7 @@
                             </label>
                         </div>
                         <div class="form-row">
-                            &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<label style="width:auto">
+                            &nbsp;<label style="width:auto">
                                 <span
                                     data-i18n="knxUltimate.properties.node-input-notifyreadrequestalsorespondtobusdefaultvalueifnotinitialized"></span>
                             </label>
@@ -524,11 +499,9 @@
                     </dd>
                 </div>
             </div>
-
-            </p>
         </div>
-        <div id="tabs-3">
-            <p>
+
+        <div id="tabs-2">
             <div class="form-row" style="padding:10px">
                 <div class="form-row">
                     <dt>
@@ -577,7 +550,12 @@
                     </select>
                 </div>
             </div>
-            </p>
+
+        </div>
+        <div id="tabs-3">
+            <div style="height:200px;min-height:150px;padding:0px;">
+                <div style="height:100%;width:100%;padding:0px;" class="node-text-editor" id="example-editor"></div>
+            </div>
         </div>
     </div>
 
@@ -607,13 +585,13 @@
 | Universal mode (listen to all Group Addresses) | By setting this, the node will read and control ALL group addresses. |
 | Sample Function | This will give it a hint on what to write in a function, if you want to control the node via a Node-Red function node. |
 | Input msg passthrough | If set, you can pass the input mgs to the output msg. |
-| Output Type | *write* to send write telegram (usually, you want that), otherwise you can choose the telegram's type to react to. |
-| Send payload to BUS only if changed (RBE filter) | *Report by change* filter. If set, only the msg input (from the Flow) having different values each time, will be sent to the KNX bus. If you intend to send everytime the same value, turn it off. |
-| Read status upon connection | Read group address status, every time Node-Red starts and at every reconnection to the KNX Gateway. The node stores all group address values to a file, so you can choose wether to read from file or from the KNX bus. |
-| React only if payload from BUS is changed (RBE filter) | *Report by change* filter. If set, only the msg output (to KNX bus) having different values each time, will be sent to the msg output's flow. If you intend to send everytime the same value, leave it off. |
-| React to event GroupValue write | The node will react (will send a msg to the flow) each time it receives a *write* telegram from the KNX bus. Usually, you want that. |
-| React to event GroupValue response | The node will react (will send a msg to the flow) each time it receives a *response* telegram from the KNX bus. Usually, you want that for particular scenarios. |
-| React to event GroupValue read | The node will react (will send a msg to the flow) each time it receives a *read* telegram from the KNX bus. Usually, you want that if you're want to send a custom value to the KNX BUS. |
+| Telegram type | *write* to send write telegram (usually, you want that), otherwise you can choose the telegram's type to react to. |
+| Send payload to KNX only if changed (RBE filter) | *Report by change* filter. If set, only the msg input (from the Flow) having different values each time, will be sent to the KNX bus. If you intend to send everytime the same value, turn it off. |
+| Read status on start | Read group address status, every time Node-Red starts and at every reconnection to the KNX Gateway. The node stores all group address values to a file, so you can choose wether to read from file or from the KNX bus. |
+| React only by changed payload (RBE filter) | *Report by change* filter. If set, only the msg output (to KNX bus) having different values each time, will be sent to the msg output's flow. If you intend to send everytime the same value, leave it off. |
+| React to write telegrams | The node will react (will send a msg to the flow) each time it receives a *write* telegram from the KNX bus. Usually, you want that. |
+| React to response telegrams | The node will react (will send a msg to the flow) each time it receives a *response* telegram from the KNX bus. Usually, you want that for particular scenarios. |
+| React to read telegrams | The node will react (will send a msg to the flow) each time it receives a *read* telegram from the KNX bus. Usually, you want that if you're want to send a custom value to the KNX BUS. |
 | Format input | Allow you to format numeric values received from the KNX Bus. |
 
 <br/>

package/nodes/knxUltimate.js

@@ -146,7 +146,7 @@ module.exports = function (RED) {
         node.inputmessage = RED.util.cloneMessage(msg); // 28/03/2020 Store the message to be passed through.
       }
 
-      // 25/07/2019 if payload is read or the output type is set to "read", do a read, otherwise, write to the bus
+      // 25/07/2019 if payload is read or the Telegram type is set to "read", do a read, otherwise, write to the bus
       if ((msg.hasOwnProperty('readstatus') && msg.readstatus === true) || node.outputtype === 'read') {
         // READ: Send a Read request to the bus
         let grpaddr = '';
@@ -156,7 +156,7 @@ module.exports = function (RED) {
           // 29/12/2020 Protection over circular references (for example, if you link two Ultimate Nodes toghether with the same group address), to prevent infinite loops
           if (msg.hasOwnProperty('knx')) {
             if (msg.knx.destination == grpaddr && ((msg.knx.event === 'GroupValue_Response' || msg.knx.event === 'GroupValue_Read'))) {
-              if (node.sysLogger !== undefined && node.sysLogger !== null) node.sysLogger.error(`knxUltimate: Circular reference protection during READ. The node ${node.id} has been temporary disabled. Two nodes with same group address and reaction/output type are linked. See the FAQ in the Wiki. Msg:${JSON.stringify(msg)}`);
+              if (node.sysLogger !== undefined && node.sysLogger !== null) node.sysLogger.error(`knxUltimate: Circular reference protection during READ. The node ${node.id} has been temporary disabled. Two nodes with same group address and reaction/Telegram type are linked. See the FAQ in the Wiki. Msg:${JSON.stringify(msg)}`);
               const t = setTimeout(() => { // 21/03/2022 fixed possible memory leak. Previously was setTimeout without "let t = ".
                 node.setNodeStatus({
                   fill: 'red', shape: 'ring', text: `DISABLED due to a circulare reference while READ (${grpaddr}).`, payload: '', GA: '', dpt: '', devicename: '',
@@ -178,7 +178,7 @@ module.exports = function (RED) {
             // 29/12/2020 Protection over circular references (for example, if you link two Ultimate Nodes toghether with the same group address), to prevent infinite loops
             if (msg.hasOwnProperty('knx')) {
               if (msg.knx.destination == grpaddr && ((msg.knx.event === 'GroupValue_Response' || msg.knx.event === 'GroupValue_Read'))) {
-                if (node.sysLogger !== undefined && node.sysLogger !== null) node.sysLogger.error(`knxUltimate: Circular reference protection during READ-2. The node ${node.id} has been temporary disabled. Two nodes with same group address and reaction/output type are linked. See the FAQ in the Wiki. Msg:${JSON.stringify(msg)}`);
+                if (node.sysLogger !== undefined && node.sysLogger !== null) node.sysLogger.error(`knxUltimate: Circular reference protection during READ-2. The node ${node.id} has been temporary disabled. Two nodes with same group address and reaction/Telegram type are linked. See the FAQ in the Wiki. Msg:${JSON.stringify(msg)}`);
                 const t = setTimeout(() => { // 21/03/2022 fixed possible memory leak. Previously was setTimeout without "let t = ".
                   node.setNodeStatus({
                     fill: 'red', shape: 'ring', text: `DISABLED due to a circulare reference while READ-2 (${grpaddr}).`, payload: '', GA: '', dpt: '', devicename: '',
@@ -201,7 +201,7 @@ module.exports = function (RED) {
                 // 29/12/2020 Protection over circular references (for example, if you link two Ultimate Nodes toghether with the same group address), to prevent infinite loops
                 if (msg.hasOwnProperty('knx')) {
                   if (msg.knx.destination == grpaddr && ((msg.knx.event === 'GroupValue_Response' || msg.knx.event === 'GroupValue_Read'))) {
-                    if (node.sysLogger !== undefined && node.sysLogger !== null) node.sysLogger.error(`knxUltimate: Circular reference protection during READ-3. Node ${node.id} The read request hasn't been sent. Two nodes with same group address and reaction/output type are linked. See the FAQ in the Wiki. Msg:${JSON.stringify(msg)}`);
+                    if (node.sysLogger !== undefined && node.sysLogger !== null) node.sysLogger.error(`knxUltimate: Circular reference protection during READ-3. Node ${node.id} The read request hasn't been sent. Two nodes with same group address and reaction/Telegram type are linked. See the FAQ in the Wiki. Msg:${JSON.stringify(msg)}`);
                     node.setNodeStatus({
                       fill: 'red', shape: 'ring', text: `NOT SENT due to a circulare reference while READ-3 (${grpaddr}).`, payload: '', GA: '', dpt: '', devicename: '',
                     });
@@ -322,7 +322,7 @@ module.exports = function (RED) {
           // Protection over circular references (for example, if you link two Ultimate Nodes toghether with the same group address), to prevent infinite loops
           if (msg.hasOwnProperty('knx')) {
             if (msg.knx.destination == grpaddr && ((msg.knx.event === 'GroupValue_Write' && outputtype === 'write') || (msg.knx.event === 'GroupValue_Response' && outputtype === 'response') || (msg.knx.event === 'GroupValue_Response' && outputtype === 'read') || (msg.knx.event === 'GroupValue_Read' && outputtype === 'read'))) {
-              if (node.sysLogger !== undefined && node.sysLogger !== null) node.sysLogger.error(`knxUltimate: Circular reference protection. The node ${node.id} has been temporarely disabled. Two nodes with same group address and reaction/output type are linked. See the FAQ in the Wiki. Msg:${JSON.stringify(msg)}`);
+              if (node.sysLogger !== undefined && node.sysLogger !== null) node.sysLogger.error(`knxUltimate: Circular reference protection. The node ${node.id} has been temporarely disabled. Two nodes with same group address and reaction/Telegram type are linked. See the FAQ in the Wiki. Msg:${JSON.stringify(msg)}`);
               const t = setTimeout(() => { // 21/03/2022 fixed possible memory leak. Previously was setTimeout without "let t = ".
                 node.setNodeStatus({
                   fill: 'red', shape: 'ring', text: `DISABLED due to a circulare reference (${grpaddr}).`, payload: '', GA: '', dpt: '', devicename: '',

package/nodes/knxUltimateAlerter.js

@@ -54,233 +54,233 @@ module.exports = function (RED) {
         dpt = (typeof dpt === 'undefined' || dpt == '') ? '' : ' DPT' + dpt
         payload = typeof payload === 'object' ? JSON.stringify(payload) : payload
         node.status({ fill, shape, text: GA + payload + (node.listenallga === true ? ' ' + devicename : '') + ' (' + dDate.getDate() + ', ' + dDate.toLocaleTimeString() + ' ' + text })
-        } catch (error) {
-        }
+      } catch (error) {
       }
+    }
 
     // Used to call the status update from the config node.
     node.setLocalStatus = ({ fill, shape, text, payload, GA, dpt, devicename }) => {
-        const dDate = new Date()
-        // 30/08/2019 Display only the things selected in the config
-        GA = (typeof GA === 'undefined' || GA == '') ? '' : '(' + GA + ') '
-        devicename = devicename || ''
-        dpt = (typeof dpt === 'undefined' || dpt == '') ? '' : ' DPT' + dpt
-        try {
-          node.status({ fill, shape, text: GA + payload + (node.listenallga === true ? ' ' + devicename : '') + ' (' + dDate.getDate() + ', ' + dDate.toLocaleTimeString() + ' ' + text })
-        } catch (error) {
-        }
+      const dDate = new Date()
+      // 30/08/2019 Display only the things selected in the config
+      GA = (typeof GA === 'undefined' || GA == '') ? '' : '(' + GA + ') '
+      devicename = devicename || ''
+      dpt = (typeof dpt === 'undefined' || dpt == '') ? '' : ' DPT' + dpt
+      try {
+        node.status({ fill, shape, text: GA + payload + (node.listenallga === true ? ' ' + devicename : '') + ' (' + dDate.getDate() + ', ' + dDate.toLocaleTimeString() + ' ' + text })
+      } catch (error) {
       }
+    }
 
-      // This function is called by the knx-ultimate config node, to output a msg.payload.
-      node.handleSend = msg => {
-        try {
-          if (!msg.knx.dpt.startsWith('1.')) return
-        } catch (error) {
-          return
-        }
-        let bFound = false // 24/04/2021 true if the cycle below found a match, otherwise false
+    // This function is called by the knx-ultimate config node, to output a msg.payload.
+    node.handleSend = msg => {
+      try {
+        if (!msg.knx.dpt.startsWith('1.')) return
+      } catch (error) {
+        return
+      }
+      let bFound = false // 24/04/2021 true if the cycle below found a match, otherwise false
 
-        // Update the node.rules with the values taken from the file, if any, otherwise leave the default value
-        for (let i = 0; i < node.rules.length; i++) {
-          // rule is { topic: rowRuleTopic, devicename: rowRuleDeviceName, longdevicename: rowRuleLongDeviceName}
-          var rule = node.rules[i]
-          if (msg.topic === rule.topic) {
-            if (msg.payload == true) {
-              bFound = true
-              // Add the device to the array of alertedDevices
-              const oTrovato = node.alertedDevices.find(a => a.topic === rule.topic)
-              if (oTrovato === undefined) {
-                node.alertedDevices.unshift({ topic: rule.topic, devicename: rule.devicename, longdevicename: rule.longdevicename }) // Add to the begin of array
-                if (node.whentostart === 'ifnewalert') node.send([null, null, node.getThirdPinMSG()])
-              }
-              node.setLocalStatus({ fill: 'red', shape: 'dot', text: 'Alert', payload: '', GA: msg.topic, dpt: '', devicename: rule.devicename })
-            } else {
-              // Remove the device from the array
-              node.alertedDevices = node.alertedDevices.filter(a => a.topic !== msg.topic)
-              node.setLocalStatus({ fill: 'green', shape: 'dot', text: 'Restore', payload: '', GA: msg.topic, dpt: '', devicename: rule.devicename })
+      // Update the node.rules with the values taken from the file, if any, otherwise leave the default value
+      for (let i = 0; i < node.rules.length; i++) {
+        // rule is { topic: rowRuleTopic, devicename: rowRuleDeviceName, longdevicename: rowRuleLongDeviceName}
+        var rule = node.rules[i]
+        if (msg.topic === rule.topic) {
+          if (msg.payload == true) {
+            bFound = true
+            // Add the device to the array of alertedDevices
+            const oTrovato = node.alertedDevices.find(a => a.topic === rule.topic)
+            if (oTrovato === undefined) {
+              node.alertedDevices.unshift({ topic: rule.topic, devicename: rule.devicename, longdevicename: rule.longdevicename }) // Add to the begin of array
+              if (node.whentostart === 'ifnewalert') node.send([null, null, node.getThirdPinMSG()])
             }
+            node.setLocalStatus({ fill: 'red', shape: 'dot', text: 'Alert', payload: '', GA: msg.topic, dpt: '', devicename: rule.devicename })
+          } else {
+            // Remove the device from the array
+            node.alertedDevices = node.alertedDevices.filter(a => a.topic !== msg.topic)
+            node.setLocalStatus({ fill: 'green', shape: 'dot', text: 'Restore', payload: '', GA: msg.topic, dpt: '', devicename: rule.devicename })
           }
         }
-
-        // If there's some device to alert, stop current timer and restart
-        // This allow the last alerted device to be outputted immediately
-        if (bFound && node.whentostart === 'ifnewalert' && node.alertedDevices.length > 0) {
-          clearTimeout(node.timerSend)
-          // Send directly the second and third message PIN
-          node.send([null, node.getSecondPinMSG(), null])
-          node.curIndexAlertedDevice = 0 // Restart form the beginning
-          node.startTimer()
-        }
       }
 
-      // Get the msg to be outputted on second PIN
-      node.getSecondPinMSG = () => {
-        if (node.alertedDevices.length > 0) {
-          const msg = {}
-          let sRet = ''
-          let sRetLong = ''
-          let sTopic = ''
-          node.alertedDevices.forEach(function (item) {
-            sTopic += item.topic + ', '
-            if (item.devicename !== undefined && item.devicename !== '') sRet += item.devicename + ', '
-            if (item.longdevicename !== undefined && item.longdevicename !== '') sRetLong += item.longdevicename + ', '
-          })
-          sTopic = sTopic.slice(0, -2)
-          if (sRet.length > 2) sRet = sRet.slice(0, -2)
-          if (sRetLong.length > 2) sRetLong = sRetLong.slice(0, -2)
-          msg.topic = sTopic
-          msg.devicename = sRet
-          msg.longdevicename = sRetLong
-          msg.count = node.alertedDevices.length
-          msg.payload = true
-          return msg
-        }
+      // If there's some device to alert, stop current timer and restart
+      // This allow the last alerted device to be outputted immediately
+      if (bFound && node.whentostart === 'ifnewalert' && node.alertedDevices.length > 0) {
+        clearTimeout(node.timerSend)
+        // Send directly the second and third message PIN
+        node.send([null, node.getSecondPinMSG(), null])
+        node.curIndexAlertedDevice = 0 // Restart form the beginning
+        node.startTimer()
       }
+    }
 
-      // Get the msg to be outputted on third PIN
-      node.getThirdPinMSG = () => {
-        if (node.alertedDevices.length > 0) {
-          const msg = {}
-          let sRet = ''
-          let sRetLong = ''
-          let sTopic = ''
-          const item = node.alertedDevices[0] // Pick the last alerted device
-          sTopic = item.topic
-          if (item.devicename !== undefined && item.devicename !== '') sRet = item.devicename
-          if (item.longdevicename !== undefined && item.longdevicename !== '') sRetLong = item.longdevicename
-          msg.topic = sTopic
-          msg.devicename = sRet
-          msg.longdevicename = sRetLong
-          msg.count = node.alertedDevices.length
-          msg.payload = true
-          return msg
-        }
+    // Get the msg to be outputted on second PIN
+    node.getSecondPinMSG = () => {
+      if (node.alertedDevices.length > 0) {
+        const msg = {}
+        let sRet = ''
+        let sRetLong = ''
+        let sTopic = ''
+        node.alertedDevices.forEach(function (item) {
+          sTopic += item.topic + ', '
+          if (item.devicename !== undefined && item.devicename !== '') sRet += item.devicename + ', '
+          if (item.longdevicename !== undefined && item.longdevicename !== '') sRetLong += item.longdevicename + ', '
+        })
+        sTopic = sTopic.slice(0, -2)
+        if (sRet.length > 2) sRet = sRet.slice(0, -2)
+        if (sRetLong.length > 2) sRetLong = sRetLong.slice(0, -2)
+        msg.topic = sTopic
+        msg.devicename = sRet
+        msg.longdevicename = sRetLong
+        msg.count = node.alertedDevices.length
+        msg.payload = true
+        return msg
       }
+    }
 
-      // 24/04/2021 perform a read on all GA in the rule list. Called both from node.on("input") and knxUltimate-config
-      node.initialReadAllDevicesInRules = () => {
-        if (node.server) {
-          let grpaddr = ''
-          for (let i = 0; i < node.rules.length; i++) {
-            // rule is { topic: rowRuleTopic, devicename: rowRuleDeviceName, longdevicename: rowRuleLongDeviceName}
-            const rule = node.rules[i]
-            // READ: Send a Read request to the bus
-            grpaddr = rule.topic
-            try {
-              // Check if it's a group address
-              // const ret = Address.KNXAddress.createFromString(grpaddr, Address.KNXAddress.TYPE_GROUP)
-              node.setLocalStatus({ fill: 'grey', shape: 'dot', text: 'Read', payload: '', GA: grpaddr, dpt: '', devicename: rule.devicename })
-              node.server.writeQueueAdd({ grpaddr, payload: '', dpt: '', outputtype: 'read', nodecallerid: node.id })
-            } catch (error) {
-              node.setLocalStatus({ fill: 'grey', shape: 'dot', text: 'Not a KNX GA ' + error.message, payload: '', GA: grpaddr, dpt: '', devicename: rule.devicename })
-            }
-          }
-        } else {
-          node.setLocalStatus({ fill: 'red', shape: 'ring', text: 'No gateway selected. Unable to read from KNX bus', payload: '', GA: '', dpt: '', devicename: '' })
-        }
+    // Get the msg to be outputted on third PIN
+    node.getThirdPinMSG = () => {
+      if (node.alertedDevices.length > 0) {
+        const msg = {}
+        let sRet = ''
+        let sRetLong = ''
+        let sTopic = ''
+        const item = node.alertedDevices[0] // Pick the last alerted device
+        sTopic = item.topic
+        if (item.devicename !== undefined && item.devicename !== '') sRet = item.devicename
+        if (item.longdevicename !== undefined && item.longdevicename !== '') sRetLong = item.longdevicename
+        msg.topic = sTopic
+        msg.devicename = sRet
+        msg.longdevicename = sRetLong
+        msg.count = node.alertedDevices.length
+        msg.payload = true
+        return msg
       }
+    }
 
-      node.on('input', function (msg) {
-        if (typeof msg === 'undefined') return
-        if (msg.hasOwnProperty('start')) {
-          clearTimeout(node.timerSend)
-          node.curIndexAlertedDevice = 0 // Restart form the beginning
-          if (node.alertedDevices.length > 0) {
-            node.send([null, node.getSecondPinMSG(), node.getThirdPinMSG()])
-            node.startTimer()
-          } else {
-            // Nothing more to output
-            node.sendNoMoreDevices()
+    // 24/04/2021 perform a read on all GA in the rule list. Called both from node.on("input") and knxUltimate-config
+    node.initialReadAllDevicesInRules = () => {
+      if (node.server) {
+        let grpaddr = ''
+        for (let i = 0; i < node.rules.length; i++) {
+          // rule is { topic: rowRuleTopic, devicename: rowRuleDeviceName, longdevicename: rowRuleLongDeviceName}
+          const rule = node.rules[i]
+          // READ: Send a Read request to the bus
+          grpaddr = rule.topic
+          try {
+            // Check if it's a group address
+            // const ret = Address.KNXAddress.createFromString(grpaddr, Address.KNXAddress.TYPE_GROUP)
+            node.setLocalStatus({ fill: 'grey', shape: 'dot', text: 'Read', payload: '', GA: grpaddr, dpt: '', devicename: rule.devicename })
+            node.server.writeQueueAdd({ grpaddr, payload: '', dpt: '', outputtype: 'read', nodecallerid: node.id })
+          } catch (error) {
+            node.setLocalStatus({ fill: 'grey', shape: 'dot', text: 'Not a KNX GA ' + error.message, payload: '', GA: grpaddr, dpt: '', devicename: rule.devicename })
           }
-          return
-        }
-
-        // 24/04/2021 if payload is read or the output type is set to "read", do a read
-        if ((msg.hasOwnProperty('readstatus') && msg.readstatus === true)) {
-          node.initialReadAllDevicesInRules()
-          return
-        }
-
-        if (msg.topic === undefined) {
-          node.setLocalStatus({ fill: 'grey', shape: 'dot', text: 'ERROR: You must provide a msg.topic', payload: '', GA: '', dpt: '', devicename: '' })
-          return
         }
-        if (msg.payload === undefined) {
-          node.setLocalStatus({ fill: 'grey', shape: 'dot', text: 'ERROR: You must provide payload (true/false)', payload: '', GA: '', dpt: '', devicename: '' })
-          return
-        }
-        msg.knx = { dpt: '1.001' }
-        node.handleSend(msg)
-      })
+      } else {
+        node.setLocalStatus({ fill: 'red', shape: 'ring', text: 'No gateway selected. Unable to read from KNX bus', payload: '', GA: '', dpt: '', devicename: '' })
+      }
+    }
 
-      node.on('close', function (done) {
+    node.on('input', function (msg) {
+      if (typeof msg === 'undefined') return
+      if (msg.hasOwnProperty('start')) {
         clearTimeout(node.timerSend)
-        if (node.server) {
-          node.server.removeClient(node)
-        }
-        done()
-      })
-
-      node.handleTimer = () => {
+        node.curIndexAlertedDevice = 0 // Restart form the beginning
         if (node.alertedDevices.length > 0) {
-          const count = node.alertedDevices.length
-          if (node.curIndexAlertedDevice > count - 1) {
-            node.curIndexAlertedDevice = 0
-            if (node.whentostart === 'manualstart') {
-              node.curIndexAlertedDevice = 0 // Restart form the beginning
-              return
-            }
-          }
-          // Create output message
-          try {
-            const curDev = node.alertedDevices[node.curIndexAlertedDevice] // is { topic: rule.topic, devicename: rule.devicename }
-            const msg = {}
-            msg.topic = curDev.topic
-            msg.count = count
-            msg.devicename = curDev.devicename
-            msg.longdevicename = curDev.longdevicename
-            msg.payload = true
-            node.send([msg, null, null])
-          } catch (error) {
-          }
-          node.curIndexAlertedDevice += 1
-          // Restart timer
+          node.send([null, node.getSecondPinMSG(), node.getThirdPinMSG()])
           node.startTimer()
         } else {
           // Nothing more to output
           node.sendNoMoreDevices()
         }
+        return
       }
 
-      // Start timer
-      node.startTimer = () => {
-        clearTimeout(node.timerSend)
-        node.timerSend = setTimeout(() => {
-          node.handleTimer()
-        }, node.timerinterval * 1000)
+      // 24/04/2021 if payload is read or the Telegram type is set to "read", do a read
+      if ((msg.hasOwnProperty('readstatus') && msg.readstatus === true)) {
+        node.initialReadAllDevicesInRules()
+        return
       }
 
-      // As soon as there no more devices..
-      node.sendNoMoreDevices = () => {
-        const msg = {}
-        msg.topic = ''
-        msg.count = 0
-        msg.devicename = ''
-        msg.longdevicename = ''
-        msg.payload = false
-        node.send([msg, msg, msg])
+      if (msg.topic === undefined) {
+        node.setLocalStatus({ fill: 'grey', shape: 'dot', text: 'ERROR: You must provide a msg.topic', payload: '', GA: '', dpt: '', devicename: '' })
+        return
       }
+      if (msg.payload === undefined) {
+        node.setLocalStatus({ fill: 'grey', shape: 'dot', text: 'ERROR: You must provide payload (true/false)', payload: '', GA: '', dpt: '', devicename: '' })
+        return
+      }
+      msg.knx = { dpt: '1.001' }
+      node.handleSend(msg)
+    })
 
-      // Init
-      node.sendNoMoreDevices()
-
-      // On each deploy, unsubscribe+resubscribe
+    node.on('close', function (done) {
+      clearTimeout(node.timerSend)
       if (node.server) {
         node.server.removeClient(node)
-        if (node.topic !== '' || node.topicSave !== '') {
-          node.server.addClient(node)
+      }
+      done()
+    })
+
+    node.handleTimer = () => {
+      if (node.alertedDevices.length > 0) {
+        const count = node.alertedDevices.length
+        if (node.curIndexAlertedDevice > count - 1) {
+          node.curIndexAlertedDevice = 0
+          if (node.whentostart === 'manualstart') {
+            node.curIndexAlertedDevice = 0 // Restart form the beginning
+            return
+          }
+        }
+        // Create output message
+        try {
+          const curDev = node.alertedDevices[node.curIndexAlertedDevice] // is { topic: rule.topic, devicename: rule.devicename }
+          const msg = {}
+          msg.topic = curDev.topic
+          msg.count = count
+          msg.devicename = curDev.devicename
+          msg.longdevicename = curDev.longdevicename
+          msg.payload = true
+          node.send([msg, null, null])
+        } catch (error) {
         }
+        node.curIndexAlertedDevice += 1
+        // Restart timer
+        node.startTimer()
+      } else {
+        // Nothing more to output
+        node.sendNoMoreDevices()
+      }
+    }
+
+    // Start timer
+    node.startTimer = () => {
+      clearTimeout(node.timerSend)
+      node.timerSend = setTimeout(() => {
+        node.handleTimer()
+      }, node.timerinterval * 1000)
+    }
+
+    // As soon as there no more devices..
+    node.sendNoMoreDevices = () => {
+      const msg = {}
+      msg.topic = ''
+      msg.count = 0
+      msg.devicename = ''
+      msg.longdevicename = ''
+      msg.payload = false
+      node.send([msg, msg, msg])
+    }
+
+    // Init
+    node.sendNoMoreDevices()
+
+    // On each deploy, unsubscribe+resubscribe
+    if (node.server) {
+      node.server.removeClient(node)
+      if (node.topic !== '' || node.topicSave !== '') {
+        node.server.addClient(node)
       }
     }
-    RED.nodes.registerType('knxUltimateAlerter', knxUltimateAlerter)
   }
+  RED.nodes.registerType('knxUltimateAlerter', knxUltimateAlerter)
+}

package/nodes/knxUltimateHueBattery.js

@@ -22,6 +22,7 @@ module.exports = function (RED) {
     node.formatmultiplyvalue = 1;
     node.formatnegativevalue = 'leave';
     node.formatdecimalsvalue = 2;
+    node.hueDevice = config.hueDevice;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({
@@ -83,11 +84,7 @@ module.exports = function (RED) {
     }
     if (node.serverHue) {
       node.serverHue.removeClient(node);
-      // I must get the object, to store read the battery status
-      // I queue the state request, by passing the callback to call whenever the HUE bridge send me the light status async
-      if (node.serverHue !== null && node.serverHue.hueManager !== null) {
-        node.serverHue.addClient(node);
-      }
+      node.serverHue.addClient(node);
     }
 
     node.on('input', (msg) => {

package/nodes/knxUltimateHueButton.js

@@ -26,6 +26,7 @@ module.exports = function (RED) {
     node.formatdecimalsvalue = 2;
     node.short_releaseValue = false;
     node.isTimerDimStopRunning = false;
+    node.hueDevice = config.hueDevice;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({

package/nodes/knxUltimateHueLight.js

@@ -58,7 +58,7 @@ module.exports = function (RED) {
     node.handleSend = (msg) => {
       if (node.currentHUEDevice === undefined) {
         node.setNodeStatusHue({
-          fill: "grey",
+          fill: "red",
           shape: "ring",
           text: "Currently not ready.",
           payload: "",
@@ -621,13 +621,7 @@ module.exports = function (RED) {
     }
     if (node.serverHue) {
       node.serverHue.removeClient(node);
-      if (node.serverHue !== null && node.serverHue.hueManager !== null) {
-        try {
-          node.serverHue.addClient(node);
-        } catch (err) {
-          RED.log.error(`Errore knxUltimateHueLight node.currentHUEDevice ${err.message}`);
-        }
-      }
+      node.serverHue.addClient(node);
     }
 
     node.on("input", (msg) => { });

package/nodes/knxUltimateHueLightSensor.js

@@ -22,6 +22,7 @@ module.exports = function (RED) {
     node.formatmultiplyvalue = 1;
     node.formatnegativevalue = 'leave';
     node.formatdecimalsvalue = 2;
+    node.hueDevice = config.hueDevice;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({ fill, shape, text, payload }) => {
@@ -76,11 +77,7 @@ module.exports = function (RED) {
     }
     if (node.serverHue) {
       node.serverHue.removeClient(node);
-      // I must get the object, to store read the status
-      // I queue the state request, by passing the callback to call whenever the HUE bridge send me the light status async
-      if (node.serverHue !== null && node.serverHue.hueManager !== null) {
-        node.serverHue.addClient(node);
-      }
+      node.serverHue.addClient(node);
     }
 
     node.on('input', function (msg) {

package/nodes/knxUltimateHueMotion.js

@@ -22,6 +22,7 @@ module.exports = function (RED) {
     node.formatmultiplyvalue = 1;
     node.formatnegativevalue = "leave";
     node.formatdecimalsvalue = 2;
+    node.hueDevice = config.hueDevice;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({ fill, shape, text, payload }) => { };

package/nodes/knxUltimateHueScene.js

@@ -24,6 +24,7 @@ module.exports = function (RED) {
     node.formatmultiplyvalue = 1;
     node.formatnegativevalue = 'leave';
     node.formatdecimalsvalue = 2;
+    node.hueDevice = config.hueDevice;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({ fill, shape, text, payload }) => {

package/nodes/knxUltimateHueTapDial.js

@@ -25,6 +25,7 @@ module.exports = function (RED) {
     node.formatdecimalsvalue = 2;
     node.brightnessState = 0;
     node.isTimerDimStopRunning = false;
+    node.hueDevice = config.hueDevice;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({

package/nodes/knxUltimateHueTemperatureSensor.js

@@ -22,6 +22,7 @@ module.exports = function (RED) {
     node.formatmultiplyvalue = 1;
     node.formatnegativevalue = 'leave';
     node.formatdecimalsvalue = 2;
+    node.hueDevice = config.hueDevice;
 
     // Used to call the status update from the config node.
     node.setNodeStatus = ({
@@ -84,9 +85,7 @@ module.exports = function (RED) {
     }
     if (node.serverHue) {
       node.serverHue.removeClient(node);
-      if (node.serverHue !== null && node.serverHue.hueManager !== null) {
-        node.serverHue.addClient(node);
-      }
+      node.serverHue.addClient(node);
     }
 
     node.on('input', (msg) => {

package/nodes/knxUltimateLoadControl.js

@@ -334,7 +334,7 @@ module.exports = function (RED) {
         node.send({ topic: node.name || node.topic, operation: 'Enabled', payload: node.sheddingStage })
       }
 
-      // 24/04/2021 if payload is read or the output type is set to "read", do a read
+      // 24/04/2021 if payload is read or the Telegram type is set to "read", do a read
       if ((msg.hasOwnProperty('readstatus') && msg.readstatus === true)) {
         node.initialReadAllDevicesInRules()
       }

package/nodes/locales/en-US/knxUltimate.json

@@ -8,21 +8,21 @@
             "node-input-topic": "Group Addr.",
             "node-input-outputtopic": "Topic",
             "node-input-dpt": "Datapoint",
-            "node-input-initialread": "Read status upon connection",
+            "node-input-initialread": "Read status on start",
             "node-input-initialread0": "No",
-            "node-input-initialread1": "Read from KNX BUS (to receive the response from BUS, enable also -React to event GroupValue response-)",
+            "node-input-initialread1": "Read from KNX BUS (to receive the response from BUS, enable also -React to response telegrams-)",
             "node-input-initialread2": "Read the last value saved to file prior to disconnection",
-            "node-input-initialread3": "Read the last value saved to file prior to disconnection. If the file is missing, read from KNX BUS (to receive the response from BUS, enable also -React to event GroupValue response-)",
-            "node-input-notifyreadrequest": "React to event GroupValue read",
-            "node-input-notifyresponse": "React to event GroupValue response",
-            "node-input-notifywrite": "React to event GroupValue write",
+            "node-input-initialread3": "Read the last value saved to file prior to disconnection. If the file is missing, read from KNX BUS (to receive the response from BUS, enable also -React to response telegrams-)",
+            "node-input-notifyreadrequest": "React to read telegrams",
+            "node-input-notifyresponse": "React to response telegrams",
+            "node-input-notifywrite": "React to write telegrams",
             "node-input-notifyreadrequestalsorespondtobus": "Auto send node value as response to the BUS",
             "node-input-notifyreadrequestalsorespondtobusdefaultvalueifnotinitialized": "If value is undefined, send",
             "node-input-listenallga": "Universal mode (listen to all Group Addresses)",
             "node-input-name": "Node Name",
-            "node-input-outputtype": "Output Type",
-            "node-input-outputRBE": "Send payload to BUS only if changed (RBE filter)",
-            "node-input-inputRBE": "React only if payload from BUS is changed (RBE filter)",
+            "node-input-outputtype": "Telegram type",
+            "node-input-outputRBE": "Send payload to KNX only if changed (RBE filter)",
+            "node-input-inputRBE": "React only by changed payload (RBE filter)",
             "node-input-formatmultiplyvalue": "Multiply",
             "node-input-formatnegativevalue": "Negatives",
             "node-input-formatdecimalsvalue": "Decimals",
@@ -36,8 +36,8 @@
             "Advancedoptions": "Advanced options",
             "Formatting": "Format INPUT (datagram coming from the KNX BUS)",
             "Operations": "NUMERIC VALUES (operations are performed in order)",
-            "OUTPUT": "OUTPUT (sends datagram to the KNX BUS)",
-            "INPUT": "INPUT (listen to datagram from the KNX BUS)",
+            "OUTPUT": "CONTROL (payload to KNX bus)",
+            "INPUT": "STATUS (payload from KNX BUS)",
             "NUMERICVALUES": "NUMERIC VALUES (operations are performed in order)",
             "noETSWarning": "You haven't imported the ETS csv file. <br/> The node will work as blind transmitter/receiver. <a href=\"https://github.com/Supergiovane/node-red-contrib-knx-ultimate/wiki/2.-Node-Configuration\" target='_blank'>Click here for help about that</a>.",
             "notify-DPT3007": "You selected a relative DIM. Please <a target='_blank' href='https://github.com/Supergiovane/node-red-contrib-knx-ultimate/wiki/-Sample---Dimming'>click here to view this sample</a> and learn how to handle such payload.",

package/nodes/locales/en-US/knxUltimateSceneController.html

@@ -11,7 +11,7 @@
         <table style="font-size:12px">
             <tr>
                 <td><img src="https://raw.githubusercontent.com/Supergiovane/node-red-contrib-knx-ultimate/master/img/greendot.png"></img></td>
-                <td>React to event GroupValue write</td>
+                <td>React to write telegrams</td>
             </tr>
             <tr>
                 <td><img src="https://raw.githubusercontent.com/Supergiovane/node-red-contrib-knx-ultimate/master/img/greenring.png"></img></td>
@@ -19,7 +19,7 @@
             </tr>
             <tr>
                 <td><img src="https://raw.githubusercontent.com/Supergiovane/node-red-contrib-knx-ultimate/master/img/bluedot.png"></img></td>
-                <td>React to event GroupValue response.</td>
+                <td>React to response telegrams.</td>
             </tr>
             <tr>
                 <td><img src="https://raw.githubusercontent.com/Supergiovane/node-red-contrib-knx-ultimate/master/img/bluering.png"></img></td>
@@ -27,7 +27,7 @@
             </tr>
             <tr>
                 <td><img src="https://raw.githubusercontent.com/Supergiovane/node-red-contrib-knx-ultimate/master/img/greudot.png"></img></td>
-                <td>React to event GroupValue read.</td>
+                <td>React to read telegrams.</td>
             </tr>
             <tr>
                 <td><img src="https://raw.githubusercontent.com/Supergiovane/node-red-contrib-knx-ultimate/master/img/greyring.png"></img></td>

package/package.json

@@ -3,7 +3,7 @@
   "engines": {
     "node": ">=16.0.0"
   },
-  "version": "2.2.4",
+  "version": "2.2.5",
   "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",