Home-Assistant
/
core
mirror of https://github.com/home-assistant/core.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use blocking light.async_turn_*

pull/12229/head
Otto Winter 2018-02-28 22:24:16 +01:00
parent ff0386d773
commit 9e83198552
No known key found for this signature in database
GPG Key ID: DB66C0BE6013F97E
2 changed files with 26 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
homeassistant/components/light/__init__.py
View File

@ -162,8 +162,12 @@ def turn_on(hass, entity_id=None, transition=None, brightness=None,
def async_turn_on(hass, entity_id=None, transition=None, brightness=None,
brightness_pct=None, rgb_color=None, xy_color=None,
color_temp=None, kelvin=None, white_value=None,
profile=None, flash=None, effect=None, color_name=None):
"""Turn all or specified light on."""
profile=None, flash=None, effect=None, color_name=None,
blocking=False):
"""Turn all or specified light on.
This method must be run in the event loop and returns a coroutine.
"""
data = {
key: value for key, value in [
(ATTR_ENTITY_ID, entity_id),
@ -182,7 +186,8 @@ def async_turn_on(hass, entity_id=None, transition=None, brightness=None,
] if value is not None
}
hass.async_add_job(hass.services.async_call(DOMAIN, SERVICE_TURN_ON, data))
return hass.async_add_job(hass.services.async_call(
DOMAIN, SERVICE_TURN_ON, data, blocking=blocking))
@bind_hass
@ -193,8 +198,11 @@ def turn_off(hass, entity_id=None, transition=None):
@callback
@bind_hass
def async_turn_off(hass, entity_id=None, transition=None):
"""Turn all or specified light off."""
def async_turn_off(hass, entity_id=None, transition=None, blocking=False):
"""Turn all or specified light off.
This method must be run in the event loop and returns a coroutine.
"""
data = {
key: value for key, value in [
(ATTR_ENTITY_ID, entity_id),
@ -202,14 +210,17 @@ def async_turn_off(hass, entity_id=None, transition=None):
] if value is not None
}
hass.async_add_job(hass.services.async_call(
DOMAIN, SERVICE_TURN_OFF, data))
return hass.async_add_job(hass.services.async_call(
DOMAIN, SERVICE_TURN_OFF, data, blocking=blocking))
@callback
@bind_hass
def async_toggle(hass, entity_id=None, transition=None):
"""Toggle all or specified light."""
def async_toggle(hass, entity_id=None, transition=None, blocking=False):
"""Toggle all or specified light.
This method must be run in the event loop and returns a coroutine.
"""
data = {
key: value for key, value in [
(ATTR_ENTITY_ID, entity_id),
@ -217,8 +228,8 @@ def async_toggle(hass, entity_id=None, transition=None):
] if value is not None
}
hass.async_add_job(hass.services.async_call(
DOMAIN, SERVICE_TOGGLE, data))
return hass.async_add_job(hass.services.async_call(
DOMAIN, SERVICE_TOGGLE, data, blocking=blocking))
@bind_hass

12
homeassistant/components/light/group.py
View File

@ -158,17 +158,13 @@ class GroupLight(light.Light):
async def async_turn_on(self, **kwargs):
"""Forward the turn_on command to all lights in the group."""
for entity_id in self._entity_ids:
payload = dict(kwargs)
payload[ATTR_ENTITY_ID] = entity_id
light.async_turn_on(self.hass, **payload)
kwargs[ATTR_ENTITY_ID] = self._entity_ids
await light.async_turn_on(self.hass, blocking=True, **kwargs)
async def async_turn_off(self, **kwargs):
"""Forward the turn_off command to all lights in the group."""
for entity_id in self._entity_ids:
payload = dict(kwargs)
payload[ATTR_ENTITY_ID] = entity_id
light.async_turn_off(self.hass, **payload)
kwargs[ATTR_ENTITY_ID] = self._entity_ids
await light.async_turn_off(self.hass, blocking=True, **kwargs)
async def async_update(self):
"""Query all members and determine the group state."""