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

package/CHANGELOG.md

@@ -1,5 +1,8 @@
 # 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
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homebridge-plugins/homebridge-tado",
-  "version": "8.5.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

@@ -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 = [];