From dc45ed38e70c76ba7c21ce88c89e9e6e6c4c4d7a Mon Sep 17 00:00:00 2001 From: cribbstechnologies Date: Mon, 12 Jun 2017 01:58:20 -0400 Subject: [PATCH] fixing potential null issue with optional param being parsed as a script (#7928) * fixing potential null issue with optional param being parsed as a script * Create template.py --- homeassistant/components/light/template.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/homeassistant/components/light/template.py b/homeassistant/components/light/template.py index 6854fac550e..07703d6c067 100644 --- a/homeassistant/components/light/template.py +++ b/homeassistant/components/light/template.py @@ -59,7 +59,7 @@ def async_setup_platform(hass, config, async_add_devices, discovery_info=None): state_template = device_config[CONF_VALUE_TEMPLATE] on_action = device_config[CONF_ON_ACTION] off_action = device_config[CONF_OFF_ACTION] - level_action = device_config[CONF_LEVEL_ACTION] + level_action = device_config.get(CONF_LEVEL_ACTION) level_template = device_config[CONF_LEVEL_TEMPLATE] template_entity_ids = set() @@ -108,7 +108,9 @@ class LightTemplate(Light): self._template = state_template self._on_script = Script(hass, on_action) self._off_script = Script(hass, off_action) - self._level_script = Script(hass, level_action) + self._level_script = None + if level_action is not None: + self._level_script = Script(hass, level_action) self._level_template = level_template self._state = False