fix error of gui editor

xBourner · Nov 29, 2024 · efd2cbc · efd2cbc
1 parent b34b8dd
commit efd2cbc
Show file tree

Hide file tree

Showing 3 changed files with 46 additions and 38 deletions.
diff --git a/dist/status-card.js b/dist/status-card.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "status-card",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "description": "A card that will show certain entities when active grouped by domain/device class",
   "main": "status-card.js",
   "author": "xBourner",

diff --git a/src/editor.js b/src/editor.js
@@ -78,10 +78,8 @@ class StatusCardEditor extends LitElement {
     }
 
     toggleSetting(type, checked) {
-      if (['showPerson', 'bulkMode', 'showPersonName', 'showBadgeName'].includes(type)) {
-          this.config[type] = checked; 
-          this.configChanged(this.config);
-      }
+      const updatedConfig = { ...this.config, [type]: checked };
+      this.configChanged(updatedConfig);
     }
 
     showMoreHiddenItems() {
@@ -91,21 +89,24 @@ class StatusCardEditor extends LitElement {
 
 
     updateConfig(property, domain, deviceClassName, value) {
-      this.config[property] = this.config[property] || {};
-      this.config[property][domain] = this.config[property][domain] || {};
+      const updatedConfig = { ...this.config }; 
+      updatedConfig[property] = { ...this.config[property] }; 
+      updatedConfig[property][domain] = { ...this.config[property]?.[domain] }; 
 
-      deviceClassName ? 
-        (this.config[property][domain][deviceClassName] = value) : 
-        (this.config[property][domain] = value);
-
-      this.configChanged(this.config);
-      this.requestUpdate();
-  }
+      if (deviceClassName) {
+        updatedConfig[property][domain][deviceClassName] = value; 
+      } else {
+        updatedConfig[property][domain] = value; 
+      }
+
+      this.configChanged(updatedConfig); 
+      this.requestUpdate(); 
+    }
 
 
-  updateSortOrder(domain, deviceClassName, newSortOrder) {
-    this.updateConfig('newSortOrder', domain, deviceClassName, newSortOrder);
-  }
+    updateSortOrder(domain, deviceClassName, newSortOrder) {
+      this.updateConfig('newSortOrder', domain, deviceClassName, newSortOrder);
+    }
 
     updateIcons(domain, deviceClassName, icon) {
       this.updateConfig('icons', domain, deviceClassName, icon);
@@ -116,7 +117,10 @@ class StatusCardEditor extends LitElement {
     }
 
     updateColors(domain, deviceClassName, color) {
-      this.updateConfig('colors', domain, deviceClassName, color);
+      const updatedConfig = { ...this.config };
+      updatedConfig.colors = { ...this.config.colors };
+      updatedConfig.colors[domain] = color;
+      this.configChanged(updatedConfig);
     }
 
     updateVisibility(domain, deviceClassName, visible) {
@@ -125,36 +129,39 @@ class StatusCardEditor extends LitElement {
 
 
     manageExtraEntity(action, index = null, field = null, value = null) {
-      this.config.extra_entities = this.config.extra_entities || [];
-
+      const updatedConfig = { ...this.config };
+      updatedConfig.extra_entities = updatedConfig.extra_entities ? updatedConfig.extra_entities.map(entity => ({ ...entity })) : [];
+
       if (action === 'add') {
-          this.config.extra_entities.push({ entity: '', status: '', icon: '', color: '' });
+        updatedConfig.extra_entities.push({ entity: '', status: '', icon: '', color: '' });
       } else if (action === 'update' && index !== null && field !== null) {
-          this.config.extra_entities[index][field] = value;
+        const updatedEntity = { ...updatedConfig.extra_entities[index] };
+        updatedEntity[field] = value;
+        updatedConfig.extra_entities[index] = updatedEntity;
       } else if (action === 'remove' && index !== null) {
-          this.config.extra_entities.splice(index, 1);
+        updatedConfig.extra_entities.splice(index, 1);
       }
-
-      this.configChanged(this.config);
+      this.configChanged(updatedConfig); 
+      setTimeout(() => {this.requestUpdate();}, 100); 
     }
-
+    
 
     manageHiddenItem(type, item, action) {
       const validTypes = ["hidden_entities", "hidden_labels"];
       if (!validTypes.includes(type)) {
         throw new Error(`Invalid type: ${type}. Expected one of ${validTypes.join(", ")}`);
       }
-      
-      if (!this.config[type]) {
-        this.config[type] = [];
-      }
-
-      const list = this.config[type];
+
+      const updatedConfig = { ...this.config };
+      updatedConfig[type] = updatedConfig[type] ? [...updatedConfig[type]] : [];
+
+      const list = updatedConfig[type];
+
       const cleanedItem = typeof item === "string" ? item.trim() : item;
       if (!cleanedItem || (typeof cleanedItem === "string" && cleanedItem === "")) {
-        return; 
+        return;
       }
-
+    
       if (action === "add" && !list.includes(cleanedItem)) {
         list.push(cleanedItem);
       } else if (action === "remove") {
@@ -164,10 +171,11 @@ class StatusCardEditor extends LitElement {
         }
       }
 
-      this.config = { ...this.config, [type]: list };
-      this.configChanged(this.config);
+      this.configChanged(updatedConfig);
+      this.requestUpdate();
     }
 
+
 
     updateAreaFloorSelection(type, value) {
       this.config = {