homebridge-melcloud-control 4.3.9-beta.2 → 4.3.9-beta.4

package/homebridge-ui/public/index.html

@@ -255,9 +255,9 @@
           if (device.Type === 3) devicesInMelCloudByType.erv.push(device);
         });
 
-        account.ataDevices ??= [];
-        account.atwDevices ??= [];
-        account.ervDevices ??= [];
+        account.ataDevices = account.ataDevices.filter(d => String(d.id) !== "0");
+        account.atwDevices = account.atwDevices.filter(d => String(d.id) !== "0");
+        account.ervDevices = account.ervDevices.filter(d => String(d.id) !== "0");
 
         const removedAta = removeStaleDevices(account.ataDevices, devicesInMelCloudByType.ata);
         const removedAtw = removeStaleDevices(account.atwDevices, devicesInMelCloudByType.atw);
@@ -294,7 +294,7 @@
 
                 // === Process presets ===
                 const presetsInMelCloud = device.Presets || [];
-                const presetsInConfig = deviceInConfig.presets || [];
+                const presetsInConfig = (deviceInConfig.presets ?? []).filter(p => String(p.id) !== "0");
                 const presetIds = new Set(presetsInConfig.map(p => String(p.id)));
 
                 let addedNewPreset = false;
@@ -317,7 +317,7 @@
                 // === Remove placeholder presets/schedules (id === '0') if new ones were added ===
                 if (addedNewPreset) {
                   const beforeCount = presetsInConfig.length;
-                  deviceInConfig.presets = presetsInConfig.filter(p => String(p.id) !== '0');
+                  deviceInConfig.presets = presetsInConfig;
                   const removedCount = beforeCount - deviceInConfig.presets.length;
 
                   if (removedCount > 0 && removedCount < beforeCount) {
@@ -331,7 +331,7 @@
 
                 // === Process schedules ===
                 const schedulesInMelCloud = device.Schedule || [];
-                const schedulesInConfig = deviceInConfig.schedules || [];
+                const schedulesInConfig = (deviceInConfig.schedules ?? []).filter(s => String(s.id) !== "0");
                 const scheduleIds = new Set(schedulesInConfig.map(s => String(s.id)));
 
                 let addedNewSchedule = false;
@@ -354,7 +354,7 @@
                 // === Remove placeholder schedules (id === '0') if new ones were added ===
                 if (addedNewSchedule) {
                   const beforeCount = schedulesInConfig.length;
-                  deviceInConfig.schedules = schedulesInConfig.filter(s => String(s.id) !== '0');
+                  deviceInConfig.schedules = schedulesInConfig;
                   const removedCount = beforeCount - deviceInConfig.schedules.length;
 
                   if (removedCount > 0 && removedCount < beforeCount) {
@@ -363,7 +363,7 @@
                 }
 
                 // === Process scenes ===
-                const scenesInConfig = deviceInConfig.scenes || [];
+                const scenesInConfig = (deviceInConfig.scenes ?? []).filter(s => String(s.id) !== "0");
                 const sceneIds = new Set(scenesInConfig.map(s => String(s.id)));
 
                 let addedNewScenes = false;
@@ -386,7 +386,7 @@
                 // === Remove placeholder scenes (id === '0') if new ones were added ===
                 if (addedNewScenes) {
                   const beforeCount = scenesInConfig.length;
-                  deviceInConfig.scenes = scenesInConfig.filter(s => String(s.id) !== '0');
+                  deviceInConfig.scenes = scenesInConfig;
                   const removedCount = beforeCount - deviceInConfig.scenes.length;
 
                   if (removedCount > 0 && removedCount < beforeCount) {
@@ -397,8 +397,7 @@
             });
 
             // Return filtered devicesInConfig to make sure upstream code uses it
-            const filteredDevices = devicesInConfig.filter(d => String(d.id) !== '0');
-            return filteredDevices;
+            return devicesInConfig;
           } catch (error) {
             updateInfo('info', `Error while processing device: ${JSON.stringify(error)}`, 'red');
           }
@@ -434,11 +433,7 @@
         document.getElementById('logIn').className = "btn btn-secondary";
       } finally {
         document.getElementById('logIn').className = "btn btn-secondary";
-        requestAnimationFrame(() => {
-          requestAnimationFrame(() => {
-            homebridge.hideSpinner();
-          });
-        });
+        homebridge.hideSpinner();
       }
     });
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "MELCloud Control",
   "name": "homebridge-melcloud-control",
-  "version": "4.3.9-beta.2",
+  "version": "4.3.9-beta.4",
   "description": "Homebridge plugin to control Mitsubishi Air Conditioner, Heat Pump and Energy Recovery Ventilation.",
   "license": "MIT",
   "author": "grzegorz914",