From 046f0ed5fd631cbac0d26e4d3869ad2c6254c0f9 Mon Sep 17 00:00:00 2001 From: Tim Nitzsche Date: Thu, 16 Dec 2021 23:00:26 +0100 Subject: [PATCH] Only check presence of values, not their content --- homeassistant/components/deconz/light.py | 26 ++++++++++++------------ 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/homeassistant/components/deconz/light.py b/homeassistant/components/deconz/light.py index e287d574633..2fa04a93b4e 100644 --- a/homeassistant/components/deconz/light.py +++ b/homeassistant/components/deconz/light.py @@ -199,32 +199,32 @@ class DeconzBaseLight(DeconzDevice, LightEntity): """Turn on light.""" data: dict[str, bool | float | int | str | tuple[float, float]] = {"on": True} - if attr_brightness := kwargs.get(ATTR_BRIGHTNESS): - data["brightness"] = attr_brightness + if ATTR_BRIGHTNESS in kwargs: + data["brightness"] = kwargs[ATTR_BRIGHTNESS] - if attr_color_temp := kwargs.get(ATTR_COLOR_TEMP): - data["color_temperature"] = attr_color_temp + if ATTR_COLOR_TEMP in kwargs: + data["color_temperature"] = kwargs[ATTR_COLOR_TEMP] - if attr_hs_color := kwargs.get(ATTR_HS_COLOR): + if ATTR_HS_COLOR in kwargs: if COLOR_MODE_XY in self._attr_supported_color_modes: - data["xy"] = color_hs_to_xy(*attr_hs_color) + data["xy"] = color_hs_to_xy(*kwargs[ATTR_HS_COLOR]) else: - data["hue"] = int(attr_hs_color[0] / 360 * 65535) - data["saturation"] = int(attr_hs_color[1] / 100 * 255) + data["hue"] = int(kwargs[ATTR_HS_COLOR][0] / 360 * 65535) + data["saturation"] = int(kwargs[ATTR_HS_COLOR][1] / 100 * 255) if ATTR_XY_COLOR in kwargs: data["xy"] = kwargs[ATTR_XY_COLOR] - if attr_transition := kwargs.get(ATTR_TRANSITION): - data["transition_time"] = int(attr_transition * 10) + if ATTR_TRANSITION in kwargs: + data["transition_time"] = int(kwargs[ATTR_TRANSITION] * 10) elif "IKEA" in self._device.manufacturer: data["transition_time"] = 0 - if (alert := FLASH_TO_DECONZ.get(kwargs.get(ATTR_FLASH, ""))) is not None: + if (alert := FLASH_TO_DECONZ.get(kwargs.get(ATTR_FLASH))) is not None: data["alert"] = alert del data["on"] - if (effect := EFFECT_TO_DECONZ.get(kwargs.get(ATTR_EFFECT, ""))) is not None: + if (effect := EFFECT_TO_DECONZ.get(kwargs.get(ATTR_EFFECT))) is not None: data["effect"] = effect await self._device.set_state(**data) @@ -240,7 +240,7 @@ class DeconzBaseLight(DeconzDevice, LightEntity): data["brightness"] = 0 data["transition_time"] = int(kwargs[ATTR_TRANSITION] * 10) - if (alert := FLASH_TO_DECONZ.get(kwargs.get(ATTR_FLASH, ""))) is not None: + if (alert := FLASH_TO_DECONZ.get(kwargs.get(ATTR_FLASH))) is not None: data["alert"] = alert del data["on"]