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

Fix validation when automation is saved from frontend (#27195)

Browse Source
This commit is contained in:
Erik Montnemery 2019-10-04 19:17:57 +02:00 committed by Paulus Schoutsen
parent b8f41dbb75
commit e27051aa61
1 changed files with 31 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

55
homeassistant/components/automation/config.py
View File

@ -18,33 +18,40 @@ from . import CONF_ACTION, CONF_CONDITION, CONF_TRIGGER, DOMAIN, PLATFORM_SCHEMA
async def async_validate_config_item(hass, config, full_config=None): async def async_validate_config_item(hass, config, full_config=None):
"""Validate config item.""" """Validate config item."""
try: config = PLATFORM_SCHEMA(config)
config = PLATFORM_SCHEMA(config)
triggers = [] triggers = []
for trigger in config[CONF_TRIGGER]: for trigger in config[CONF_TRIGGER]:
trigger_platform = importlib.import_module( trigger_platform = importlib.import_module(
"..{}".format(trigger[CONF_PLATFORM]), __name__ "..{}".format(trigger[CONF_PLATFORM]), __name__
)
if hasattr(trigger_platform, "async_validate_trigger_config"):
trigger = await trigger_platform.async_validate_trigger_config(
hass, trigger
) )
if hasattr(trigger_platform, "async_validate_trigger_config"): triggers.append(trigger)
trigger = await trigger_platform.async_validate_trigger_config( config[CONF_TRIGGER] = triggers
hass, trigger
)
triggers.append(trigger)
config[CONF_TRIGGER] = triggers
if CONF_CONDITION in config: if CONF_CONDITION in config:
conditions = [] conditions = []
for cond in config[CONF_CONDITION]: for cond in config[CONF_CONDITION]:
cond = await condition.async_validate_condition_config(hass, cond) cond = await condition.async_validate_condition_config(hass, cond)
conditions.append(cond) conditions.append(cond)
config[CONF_CONDITION] = conditions config[CONF_CONDITION] = conditions
actions = [] actions = []
for action in config[CONF_ACTION]: for action in config[CONF_ACTION]:
action = await script.async_validate_action_config(hass, action) action = await script.async_validate_action_config(hass, action)
actions.append(action) actions.append(action)
config[CONF_ACTION] = actions config[CONF_ACTION] = actions
return config
async def _try_async_validate_config_item(hass, config, full_config=None):
"""Validate config item."""
try:
config = await async_validate_config_item(hass, config, full_config)
except (vol.Invalid, HomeAssistantError, IntegrationNotFound) as ex: except (vol.Invalid, HomeAssistantError, IntegrationNotFound) as ex:
async_log_exception(ex, DOMAIN, full_config or config, hass) async_log_exception(ex, DOMAIN, full_config or config, hass)
return None return None
@ -57,7 +64,7 @@ async def async_validate_config(hass, config):
automations = [] automations = []
validated_automations = await asyncio.gather( validated_automations = await asyncio.gather(
*( *(
async_validate_config_item(hass, p_config, config) _try_async_validate_config_item(hass, p_config, config)
for _, p_config in config_per_platform(config, DOMAIN) for _, p_config in config_per_platform(config, DOMAIN)
) )
) )