From 58ae8d91f931151d3d2dbc2338d4831c52b1ec84 Mon Sep 17 00:00:00 2001 From: Sebastian Muszynski Date: Sat, 28 Apr 2018 12:35:19 +0200 Subject: [PATCH] Fix the optional friendly name of the Yeelight (Closes: #14088) (#14110) --- homeassistant/components/light/yeelight.py | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/homeassistant/components/light/yeelight.py b/homeassistant/components/light/yeelight.py index d6d860cbd9e..202c6ac594d 100644 --- a/homeassistant/components/light/yeelight.py +++ b/homeassistant/components/light/yeelight.py @@ -32,16 +32,17 @@ LEGACY_DEVICE_TYPE_MAP = { 'ceiling1': 'ceiling', } -CONF_TRANSITION = 'transition' +DEFAULT_NAME = 'Yeelight' DEFAULT_TRANSITION = 350 +CONF_TRANSITION = 'transition' CONF_SAVE_ON_CHANGE = 'save_on_change' CONF_MODE_MUSIC = 'use_music_mode' DATA_KEY = 'light.yeelight' DEVICE_SCHEMA = vol.Schema({ - vol.Optional(CONF_NAME): cv.string, + vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string, vol.Optional(CONF_TRANSITION, default=DEFAULT_TRANSITION): cv.positive_int, vol.Optional(CONF_MODE_MUSIC, default=False): cv.boolean, vol.Optional(CONF_SAVE_ON_CHANGE, default=True): cv.boolean, @@ -136,20 +137,18 @@ def setup_platform(hass, config, add_devices, discovery_info=None): # Not using hostname, as it seems to vary. name = "yeelight_%s_%s" % (device_type, discovery_info['properties']['mac']) - device = {'name': name, 'ipaddr': discovery_info['host']} + host = discovery_info['host'] + device = {'name': name, 'ipaddr': host} light = YeelightLight(device, DEVICE_SCHEMA({})) lights.append(light) - hass.data[DATA_KEY][name] = light + hass.data[DATA_KEY][host] = light else: - for ipaddr, device_config in config[CONF_DEVICES].items(): - name = device_config[CONF_NAME] - _LOGGER.debug("Adding configured %s", name) - - device = {'name': name, 'ipaddr': ipaddr} + for host, device_config in config[CONF_DEVICES].items(): + device = {'name': device_config[CONF_NAME], 'ipaddr': host} light = YeelightLight(device, device_config) lights.append(light) - hass.data[DATA_KEY][name] = light + hass.data[DATA_KEY][host] = light add_devices(lights, True)