homebridge-melcloud-control 3.9.0-beta.11 → 3.9.0-beta.12

package/homebridge-ui/public/index.html

@@ -15,6 +15,11 @@
       <img src="homebridge-melcloud-control.png" alt="Image" height="100" />
     </div>
 
+    <!-- Config button -->
+    <div class="text-center mt-3">
+      <button type="button" class="btn btn-primary" id="configButton" disabled>⚙️ Config</button>
+    </div>
+
     <!-- Tabs -->
     <ul class="nav nav-tabs mt-3" id="accountTabs" role="tablist"></ul>
 
@@ -43,6 +48,17 @@
       const tabs = document.getElementById("accountTabs");
       const content = document.getElementById("accountTabsContent");
 
+      let configButtonState = false;
+      const configButton = document.getElementById('configButton');
+
+      function updateConfigButtonState(account) {
+        const hasMultipleDevices =
+          (account.ataDevices?.length || 0) > 1 ||
+          (account.atwDevices?.length || 0) > 1 ||
+          (account.ervDevices?.length || 0) > 1;
+        configButton.disabled = !hasMultipleDevices;
+      }
+
       function createAccountForm(account, i) {
         // Tab header
         const li = document.createElement("li");
@@ -103,7 +119,6 @@
             </div>
             <button type="button" class="btn btn-secondary" id="logIn-${i}">Connect</button>
             <button type="button" class="btn btn-danger ms-2" id="remove-${i}">❌ Remove Account</button>
-            <button type="button" class="btn btn-primary" id="configButton">⚙️ Config</button>
           </form>`;
         content.appendChild(pane);
 
@@ -124,45 +139,11 @@
           accounts.splice(i, 1);
           document.getElementById(`tab-li-${i}`).remove();
           document.getElementById(`pane-${i}`).remove();
-
-          // Rebuild IDs
-          const panes = document.querySelectorAll(".tab-pane");
-          panes.forEach((p, index) => {
-            p.id = `pane-${index}`;
-            p.querySelector("form").id = `configForm-${index}`;
-            p.querySelector(`[id^="name-"]`).id = `name-${index}`;
-            p.querySelector(`[id^="user-"]`).id = `user-${index}`;
-            p.querySelector(`[id^="passwd-"]`).id = `passwd-${index}`;
-            p.querySelector(`[id^="language-"]`).id = `language-${index}`;
-            p.querySelector(`[id^="logIn-"]`).id = `logIn-${index}`;
-            p.querySelector(`[id^="remove-"]`).id = `remove-${index}`;
-          });
-          const tabButtons = document.querySelectorAll(".nav-link");
-          tabButtons.forEach((btn, idx) => {
-            btn.id = `tab-${idx}`;
-            btn.dataset.bsTarget = `#pane-${idx}`;
-          });
-
           await homebridge.updatePluginConfig(pluginConfig);
           await homebridge.savePluginConfig(pluginConfig);
-
           if (accounts.length > 0) new bootstrap.Tab(document.getElementById("tab-0")).show();
         });
 
-        document.getElementById('configButton').addEventListener('click', async () => {
-          if (this.configButtonState) {
-            homebridge.hideSchemaForm();
-            document.getElementById('configButton').className = "btn btn-secondary";
-          } else {
-            homebridge.showSchemaForm();
-            document.getElementById('configButton').className = "btn btn-primary";
-          }
-          this.configButtonState = !this.configButtonState;
-        });
-
-        // Initialize state
-        this.configButtonState = false;
-
         // Login button
         document.getElementById(`logIn-${i}`).addEventListener("click", async () => {
           homebridge.showSpinner();
@@ -201,31 +182,23 @@
             const atwDevices = account.atwDevices ?? (account.atwDevices = []);
             const ervDevices = account.ervDevices ?? (account.ervDevices = []);
 
-            const removedAta = removeStaleDevices(ataDevices, devicesByType.ata);
-            const removedAtw = removeStaleDevices(atwDevices, devicesByType.atw);
-            const removedErv = removeStaleDevices(ervDevices, devicesByType.erv);
+            removeStaleDevices(ataDevices, devicesByType.ata);
+            removeStaleDevices(atwDevices, devicesByType.atw);
+            removeStaleDevices(ervDevices, devicesByType.erv);
 
             // Add new devices & presets
             function handleDevices(configArray, devicesArray, typeName, newDevicesArr, newPresetsArr) {
               devicesArray.forEach(d => {
-                const existing = configArray.find(x => x.id === d.DeviceID);
+                let existing = configArray.find(x => x.id === d.DeviceID);
                 if (!existing) {
-                  const devObj = {
-                    id: d.DeviceID,
-                    type: d.Type,
-                    typeString: typeName,
-                    name: d.DeviceName,
-                    displayMode: 1,
-                    presets: [],
-                    buttonsSensors: []
-                  };
-                  configArray.push(devObj);
-                  newDevicesArr.push(devObj);
+                  existing = { id: d.DeviceID, type: d.Type, typeString: typeName, name: d.DeviceName, displayMode: 1, presets: [], buttonsSensors: [] };
+                  configArray.push(existing);
+                  newDevicesArr.push(existing);
                 }
                 (d.Presets || []).forEach(p => {
                   const preset = { ...p, id: p.ID, name: p.NumberDescription, displayType: 0, namePrefix: false };
-                  if (!existing?.presets.some(pr => pr.id === p.ID)) {
-                    existing?.presets.push(preset);
+                  if (!existing.presets.some(pr => pr.id === p.ID)) {
+                    existing.presets.push(preset);
                     newPresetsArr.push(preset);
                   }
                 });
@@ -236,19 +209,17 @@
             handleDevices(atwDevices, devicesByType.atw, "Heat Pump", newDevices.atw, newDevices.atwPresets);
             handleDevices(ervDevices, devicesByType.erv, "ERV", newDevices.erv, newDevices.ervPresets);
 
-            // Info display
+            // Update info
             function updateInfo(id, text, color) {
               const el = document.getElementById(id);
               if (el) { el.innerHTML = text; el.style.color = color; }
             }
 
             const textInfo = `Found devices: ATA(${newDevices.ata.length}), ATW(${newDevices.atw.length}), ERV(${newDevices.erv.length})`;
-            const textPresets = `Found presets: ATA(${newDevices.ataPresets.length}), ATW(${newDevices.atwPresets.length}), ERV(${newDevices.ervPresets.length})`;
-            const textRemoved = `Removed devices: ATA(${removedAta.length}), ATW(${removedAtw.length}), ERV(${removedErv.length})`;
+            updateInfo('info', textInfo, 'green');
 
-            updateInfo('info', textInfo, newDevices.ata.length + newDevices.atw.length + newDevices.erv.length > 0 ? 'green' : 'white');
-            updateInfo('info1', textPresets, newDevices.ataPresets.length + newDevices.atwPresets.length + newDevices.ervPresets.length > 0 ? 'green' : 'white');
-            updateInfo('info2', textRemoved, removedAta.length + removedAtw.length + removedErv.length > 0 ? 'orange' : 'white');
+            // Update Config button state
+            updateConfigButtonState(account);
 
             await homebridge.updatePluginConfig(pluginConfig);
             await homebridge.savePluginConfig(pluginConfig);
@@ -270,7 +241,7 @@
       tabs.appendChild(addLi);
 
       document.getElementById("addTab").addEventListener("click", async () => {
-        const newAccount = { name: "", user: "", passwd: "", language: 0 };
+        const newAccount = { name: "", user: "", passwd: "", language: 0, ataDevices: [{}], atwDevices: [{}], ervDevices: [{}] };
         accounts.push(newAccount);
         const i = accounts.length - 1;
         createAccountForm(newAccount, i);
@@ -279,6 +250,18 @@
         new bootstrap.Tab(document.getElementById(`tab-${i}`)).show();
       });
 
+      // Config button toggle
+      configButton.addEventListener('click', () => {
+        if (configButtonState) {
+          homebridge.hideSchemaForm();
+          configButton.className = "btn btn-secondary";
+        } else {
+          homebridge.showSchemaForm();
+          configButton.className = "btn btn-primary";
+        }
+        configButtonState = !configButtonState;
+      });
+
     })();
   </script>
 </body>

package/package.json

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