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

Warnings for not divisible settings

Browse Source
This commit is contained in:
magnusknutas 2016-01-26 17:30:55 +01:00
parent a918be517d
commit e8e2814313
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
homeassistant/components/automation/time.py
View File

@ -37,6 +37,15 @@ def trigger(hass, config, action):
hours = config.get(CONF_HOURS) hours = config.get(CONF_HOURS)
minutes = config.get(CONF_MINUTES) minutes = config.get(CONF_MINUTES)
seconds = config.get(CONF_SECONDS) seconds = config.get(CONF_SECONDS)
if minutes.startswith('/') and not minutes.lstrip('/') % 60 == 0:
_LOGGER.warning('Periodic minutes should be divisible with 60'
'there will be an offset every hour')
if seconds.startswith('/') and not seconds.lstrip('/') % 60 == 0:
_LOGGER.warning('Periodic seconds should be divisible with 60'
'there will be an offset every minute')
if hours.startswith('/') and not hours.lstrip('/') % 24 == 0:
_LOGGER.warning('Periodic hours should be divisible with 24'
'there will be an offset every midnight')
else: else:
_LOGGER.error('One of %s, %s, %s OR %s needs to be specified', _LOGGER.error('One of %s, %s, %s OR %s needs to be specified',
CONF_HOURS, CONF_MINUTES, CONF_SECONDS, CONF_AFTER) CONF_HOURS, CONF_MINUTES, CONF_SECONDS, CONF_AFTER)