From 1b262f7cea911a5446174e394fa72dd4e003c297 Mon Sep 17 00:00:00 2001
From: c727 <crusader727@gmx.de>
Date: Thu, 12 Jul 2018 11:58:11 +0200
Subject: [PATCH] Picture glance: introduce DOMAINS_TOGGLE (#1440)

* Picture glance: introduce DOMAINS_TOGGLE

* Use set instead of array
---
 .../lovelace/cards/hui-picture-glance-card.js | 21 ++++++++++---------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/src/panels/lovelace/cards/hui-picture-glance-card.js b/src/panels/lovelace/cards/hui-picture-glance-card.js
index e0fbde2891..9d3257a199 100644
--- a/src/panels/lovelace/cards/hui-picture-glance-card.js
+++ b/src/panels/lovelace/cards/hui-picture-glance-card.js
@@ -6,7 +6,6 @@ import '../../../components/ha-card.js';
 import '../components/hui-image.js';
 
 import { STATES_OFF } from '../../../common/const.js';
-import canToggleState from '../../../common/entity/can_toggle_state.js';
 import computeStateDisplay from '../../../common/entity/compute_state_display.js';
 import computeStateName from '../../../common/entity/compute_state_name.js';
 import stateIcon from '../../../common/entity/state_icon.js';
@@ -17,7 +16,11 @@ import LocalizeMixin from '../../../mixins/localize-mixin.js';
 import NavigateMixin from '../../../mixins/navigate-mixin.js';
 import computeDomain from '../../../common/entity/compute_domain';
 
-const DOMAINS_FORCE_MORE_INFO = ['media_player'];
+const DOMAINS_TOGGLE = new Set([
+  'input_boolean',
+  'light',
+  'switch'
+]);
 
 /*
  * @appliesMixin EventsMixin
@@ -89,7 +92,7 @@ class HuiPictureGlanceCard extends NavigateMixin(LocalizeMixin(EventsMixin(Polym
             </template>
           </div>
           <div>
-            <template is="dom-repeat" items="[[_entitiesService]]">
+            <template is="dom-repeat" items="[[_entitiesToggle]]">
               <template is="dom-if" if="[[_showEntity(item, hass.states)]]">
                 <paper-icon-button
                   on-click="_callService"
@@ -111,12 +114,12 @@ class HuiPictureGlanceCard extends NavigateMixin(LocalizeMixin(EventsMixin(Polym
       _config: Object,
       _entitiesDialog: {
         type: Array,
-        computed: '_computeEntitiesDialog(hass, _config, _entitiesService)',
+        computed: '_computeEntitiesDialog(hass, _config, _entitiesToggle)',
       },
-      _entitiesService: {
+      _entitiesToggle: {
         type: Array,
         value: [],
-        computed: '_computeEntitiesService(hass, _config)',
+        computed: '_computeEntitiesToggle(hass, _config)',
       },
     };
   }
@@ -144,15 +147,13 @@ class HuiPictureGlanceCard extends NavigateMixin(LocalizeMixin(EventsMixin(Polym
                                   (entity in hass.states));
   }
 
-  _computeEntitiesService(hass, config) {
+  _computeEntitiesToggle(hass, config) {
     if (config.force_dialog) {
       return [];
     }
 
     return config.entities.filter(entity =>
-      (entity in hass.states) &&
-      !DOMAINS_FORCE_MORE_INFO.includes(computeDomain(entity)) &&
-      canToggleState(hass, hass.states[entity]));
+      (entity in hass.states) && DOMAINS_TOGGLE.has(computeDomain(entity)));
   }
 
   _showEntity(entityId, states) {