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

fix empty TOPIC_BASE issue (#21740)

Browse Source 
* fix empty TOPIC_BASE issue

if the value of the TOPIC_BASE is empty then we need to remove "~" from the topic value if it exists. 
by doing `if base:` on line 239 the condition will be false if the value is empty so the '~' will not be stripped from the topic value.
I simply removed the `if base:` line and added `if TOPIC_BASE in payload:`

* Update homeassistant/components/mqtt/discovery.py

Co-Authored-By: engrbm87 <engrbm87@gmail.com>
This commit is contained in:
engrbm87 2019-03-09 02:48:54 +02:00 committed by Paulus Schoutsen
parent d8ac761bb6
commit 76d11e4b74
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
homeassistant/components/mqtt/discovery.py
View File

@ -235,8 +235,8 @@ async def async_start(hass: HomeAssistantType, discovery_topic, hass_config,
key = DEVICE_ABBREVIATIONS.get(key, key) key = DEVICE_ABBREVIATIONS.get(key, key)
device[key] = device.pop(abbreviated_key) device[key] = device.pop(abbreviated_key)
base = payload.pop(TOPIC_BASE, None) if TOPIC_BASE in payload:
if base: base = payload.pop(TOPIC_BASE)
for key, value in payload.items(): for key, value in payload.items():
if isinstance(value, str) and value: if isinstance(value, str) and value:
if value[0] == TOPIC_BASE and key.endswith('_topic'): if value[0] == TOPIC_BASE and key.endswith('_topic'):