@homebridge-plugins/homebridge-tado 8.4.0 → 8.5.1-beta.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## v8.5.1 - 2025-10-28
+- Fix: Incorrect update of config zone properties leading to unwanted heating changes (#178)
+
+## v8.5.0 - 2025-10-27
+- Change minimum polling interval to 30s due to improvements made in v8.2.0
+
 ## v8.4.0 - 2025-10-26
 - Improve state handling to ensure Apple Home states always reflect current tado states
 

package/config.schema.json

@@ -43,7 +43,7 @@
             },
             "polling": {
               "title": "Polling",
-              "description": "The polling interval in seconds.",
+              "description": "The polling interval in seconds (recommended value: 300).",
               "type": "integer",
               "default": 300,
               "minimum": 30

package/homebridge-ui/public/js/schema.js

@@ -36,7 +36,7 @@ const schema = {
         },
         'polling': {
           'title': 'Polling',
-          'description': 'The polling interval in seconds.',
+          'description': 'The polling interval in seconds (recommended value: 300).',
           'type': 'integer',
           'default': 300,
           'minimum': 30

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homebridge-plugins/homebridge-tado",
-  "version": "8.4.0",
+  "version": "8.5.1-beta.0",
   "description": "Homebridge plugin for controlling tado° devices.",
   "main": "index.js",
   "scripts": {
@@ -51,4 +51,4 @@
     "globals": "^16.4.0",
     "prettier": "^3.6.2"
   }
-}
+}

package/src/helper/handler.js

@@ -773,7 +773,7 @@ export default (api, accessories, config, tado, telegram) => {
     tasksInitialized = true;
 
     void getStates();
-    setInterval(() => getStates(), Math.max(config.polling, 300) * 1000);
+    setInterval(() => getStates(), Math.max(config.polling, 30) * 1000);
 
     void logCounter();
     setInterval(() => logCounter(), 60 * 60 * 1000);
@@ -875,36 +875,34 @@ export default (api, accessories, config, tado, telegram) => {
         }
       }
 
-      if (idToUpdate === undefined) {
-        const allZones = (await tado.getZones(config.homeId)) || [];
+      const allZones = (await tado.getZones(config.homeId)) || [];
 
-        for (const [index, zone] of config.zones.entries()) {
-          allZones.forEach((zoneWithID) => {
-            if (zoneWithID.name === zone.name) {
-              const heatAccessory = accessories.filter(
-                (acc) => acc && acc.displayName === config.homeName + ' ' + zone.name + ' Heater'
-              );
+      for (const [index, zone] of config.zones.entries()) {
+        allZones.forEach((zoneWithID) => {
+          if (zoneWithID.name === zone.name) {
+            const heatAccessory = accessories.filter(
+              (acc) => acc && acc.displayName === config.homeName + ' ' + zone.name + ' Heater'
+            );
 
-              if (heatAccessory.length) heatAccessory[0].context.config.zoneId = zoneWithID.id;
-
-              config.zones[index].id = zoneWithID.id;
-              config.zones[index].battery = !config.zones[index].noBattery
-                ? zoneWithID.devices.filter(
-                  (device) =>
-                    device &&
-                    (zone.type === 'HEATING' || zone.type === 'AIR_CONDITIONING') &&
-                    typeof device.batteryState === 'string' &&
-                    !device.batteryState.includes('NORMAL')
-                ).length
-                  ? zoneWithID.devices.filter((device) => device && !device.batteryState.includes('NORMAL'))[0]
-                    .batteryState
-                  : zoneWithID.devices.filter((device) => device && device.duties.includes('ZONE_LEADER'))[0].batteryState
-                : false;
-              config.zones[index].openWindowEnabled =
-                zoneWithID.openWindowDetection && zoneWithID.openWindowDetection.enabled ? true : false;
-            }
-          });
-        }
+            if (heatAccessory.length) heatAccessory[0].context.config.zoneId = zoneWithID.id;
+
+            config.zones[index].id = zoneWithID.id;
+            config.zones[index].battery = !config.zones[index].noBattery
+              ? zoneWithID.devices.filter(
+                (device) =>
+                  device &&
+                  (zone.type === 'HEATING' || zone.type === 'AIR_CONDITIONING') &&
+                  typeof device.batteryState === 'string' &&
+                  !device.batteryState.includes('NORMAL')
+              ).length
+                ? zoneWithID.devices.filter((device) => device && !device.batteryState.includes('NORMAL'))[0]
+                  .batteryState
+                : zoneWithID.devices.filter((device) => device && device.duties.includes('ZONE_LEADER'))[0].batteryState
+              : false;
+            config.zones[index].openWindowEnabled =
+              zoneWithID.openWindowDetection && zoneWithID.openWindowDetection.enabled ? true : false;
+          }
+        });
       }
 
       let zonesToUpdate = [];

package/src/tado/tado-config.js

@@ -519,7 +519,7 @@ export default {
               home.extras && home.extras.childLockSwitches
                 ? home.extras.childLockSwitches.filter((childLockSwitch) => childLockSwitch && childLockSwitch.active)
                 : [],
-            polling: Number.isInteger(home.polling) ? (home.polling < 300 ? 300 : home.polling) : 300,
+            polling: Number.isInteger(home.polling) ? (home.polling < 30 ? 30 : home.polling) : 300,
           };
 
           if (home.zones && home.zones.length) {