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

Avoid polling state machine for available state in HomeKit (#101799)

Browse Source
This commit is contained in:
J. Nick Koston 2023-10-13 14:23:15 -10:00 committed by GitHub
parent 371d988643
commit 8fd5d89d43
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
homeassistant/components/homekit/accessories.py
View File

@ -373,6 +373,7 @@ class HomeAccessory(Accessory): # type: ignore[misc]
"""Add battery service if available"""
state = self.hass.states.get(self.entity_id)
self._update_available_from_state(state)
assert state is not None
entity_attributes = state.attributes
battery_found = entity_attributes.get(ATTR_BATTERY_LEVEL)
@ -415,16 +416,20 @@ class HomeAccessory(Accessory): # type: ignore[misc]
CHAR_STATUS_LOW_BATTERY, value=0
)
def _update_available_from_state(self, new_state: State | None) -> None:
"""Update the available property based on the state."""
self._available = new_state is not None and new_state.state != STATE_UNAVAILABLE
@property
def available(self) -> bool:
"""Return if accessory is available."""
state = self.hass.states.get(self.entity_id)
return state is not None and state.state != STATE_UNAVAILABLE
return self._available
async def run(self) -> None:
"""Handle accessory driver started event."""
if state := self.hass.states.get(self.entity_id):
self.async_update_state_callback(state)
self._update_available_from_state(state)
self._subscriptions.append(
async_track_state_change_event(
self.hass, [self.entity_id], self.async_update_event_state_callback
@ -474,6 +479,7 @@ class HomeAccessory(Accessory): # type: ignore[misc]
"""Handle state change event listener callback."""
new_state = event.data["new_state"]
old_state = event.data["old_state"]
self._update_available_from_state(new_state)
if (
new_state
and old_state