jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-23 05:07:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

Check for command topics when determinig the capabilities of an MQTT light. (#5770)

Browse Source 
Previous code used the state topic which is obviously wrong:
- The state topic is already used to select optimistic mode.
- A light with only the state topic but no command topic would still announce the capability.
This commit is contained in:
Hermann Kraus 2017-02-07 10:13:00 +01:00 committed by Paulus Schoutsen
parent c7e282257a
commit 51810620fb
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
homeassistant/components/light/mqtt.py
View File

@ -121,12 +121,12 @@ class MqttLight(Light):
self._state = False
self._supported_features = 0
self._supported_features |= (
topic[CONF_RGB_STATE_TOPIC] is not None and SUPPORT_RGB_COLOR)
topic[CONF_RGB_COMMAND_TOPIC] is not None and SUPPORT_RGB_COLOR)
self._supported_features |= (
topic[CONF_BRIGHTNESS_STATE_TOPIC] is not None and
topic[CONF_BRIGHTNESS_COMMAND_TOPIC] is not None and
SUPPORT_BRIGHTNESS)
self._supported_features |= (
topic[CONF_COLOR_TEMP_STATE_TOPIC] is not None and
topic[CONF_COLOR_TEMP_COMMAND_TOPIC] is not None and
SUPPORT_COLOR_TEMP)
for key, tpl in list(templates.items()):