fix: Hiding all setpoints didn't actually work

nervetattoo · Apr 8, 2021 · 3bc227c · 3bc227c
1 parent fcc30b0
commit 3bc227c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/config/setpoints.ts b/src/config/setpoints.ts
@@ -11,6 +11,10 @@ export default function parseSetpoints(
   setpoints: Setpoints | false,
   attributes: any
 ) {
+  if (setpoints === false) {
+    return {}
+  }
+
   if (setpoints) {
     const def = Object.keys(setpoints)
     return def.reduce((result, name: string) => {

diff --git a/src/main.ts b/src/main.ts
@@ -77,7 +77,10 @@ function getModeList(
   return attributes[`${type}_modes`]
     .filter((modeOption) => shouldShowModeControl(modeOption, specification))
     .map((modeOption) => {
-      const values = typeof specification[modeOption] === 'object' ? specification[modeOption] : {} as {}
+      const values =
+        typeof specification[modeOption] === 'object'
+          ? specification[modeOption]
+          : ({} as {})
       return {
         icon: MODE_ICONS[modeOption],
         value: modeOption,
@@ -166,7 +169,7 @@ export default class SimpleThermostat extends LitElement {
 
     const attributes = entity.attributes
 
-    let values = parseSetpoints(this.config?.setpoints ?? false, attributes)
+    let values = parseSetpoints(this.config?.setpoints ?? null, attributes)
 
     // If we are updating the values, and they are now equal
     // we can safely assume we've been able to update the set points