From 5be36912b2f8eb15f4d72a5ff5c63d083569bd9d Mon Sep 17 00:00:00 2001 From: epenet <6771947+epenet@users.noreply.github.com> Date: Mon, 28 Nov 2022 09:19:41 +0100 Subject: [PATCH] Use _attr_state in openhome media player (#82833) --- .../components/openhome/media_player.py | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/homeassistant/components/openhome/media_player.py b/homeassistant/components/openhome/media_player.py index f352d7101ac..fba397c1326 100644 --- a/homeassistant/components/openhome/media_player.py +++ b/homeassistant/components/openhome/media_player.py @@ -124,7 +124,7 @@ class OpenhomeDevice(MediaPlayerEntity): self._source_index = {} self._source = {} self._name = None - self._state = MediaPlayerState.PLAYING + self._attr_state = MediaPlayerState.PLAYING self._available = True @property @@ -178,16 +178,16 @@ class OpenhomeDevice(MediaPlayerEntity): ) if self._in_standby: - self._state = MediaPlayerState.OFF + self._attr_state = MediaPlayerState.OFF elif self._transport_state == "Paused": - self._state = MediaPlayerState.PAUSED + self._attr_state = MediaPlayerState.PAUSED elif self._transport_state in ("Playing", "Buffering"): - self._state = MediaPlayerState.PLAYING + self._attr_state = MediaPlayerState.PLAYING elif self._transport_state == "Stopped": - self._state = MediaPlayerState.IDLE + self._attr_state = MediaPlayerState.IDLE else: # Device is playing an external source with no transport controls - self._state = MediaPlayerState.PLAYING + self._attr_state = MediaPlayerState.PLAYING self._available = True except (asyncio.TimeoutError, aiohttp.ClientError, UpnpError): @@ -279,11 +279,6 @@ class OpenhomeDevice(MediaPlayerEntity): """Return a unique ID.""" return self._device.uuid() - @property - def state(self): - """Return the state of the device.""" - return self._state - @property def source_list(self): """List of available input sources."""