Use self._attr_state in demo media player (#78520)

This commit is contained in:
epenet 2022-09-15 14:44:20 +02:00 committed by GitHub
parent 69ca055fd8
commit a443cefa7b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -114,7 +114,7 @@ class AbstractDemoPlayer(MediaPlayerEntity):
) -> None: ) -> None:
"""Initialize the demo device.""" """Initialize the demo device."""
self._attr_name = name self._attr_name = name
self._player_state = MediaPlayerState.PLAYING self._attr_state = MediaPlayerState.PLAYING
self._volume_level = 1.0 self._volume_level = 1.0
self._volume_muted = False self._volume_muted = False
self._shuffle = False self._shuffle = False
@ -122,11 +122,6 @@ class AbstractDemoPlayer(MediaPlayerEntity):
self._sound_mode = DEFAULT_SOUND_MODE self._sound_mode = DEFAULT_SOUND_MODE
self._attr_device_class = device_class self._attr_device_class = device_class
@property
def state(self) -> str:
"""Return the state of the player."""
return self._player_state
@property @property
def volume_level(self) -> float: def volume_level(self) -> float:
"""Return the volume level of the media player (0..1).""" """Return the volume level of the media player (0..1)."""
@ -154,12 +149,12 @@ class AbstractDemoPlayer(MediaPlayerEntity):
def turn_on(self) -> None: def turn_on(self) -> None:
"""Turn the media player on.""" """Turn the media player on."""
self._player_state = MediaPlayerState.PLAYING self._attr_state = MediaPlayerState.PLAYING
self.schedule_update_ha_state() self.schedule_update_ha_state()
def turn_off(self) -> None: def turn_off(self) -> None:
"""Turn the media player off.""" """Turn the media player off."""
self._player_state = MediaPlayerState.OFF self._attr_state = MediaPlayerState.OFF
self.schedule_update_ha_state() self.schedule_update_ha_state()
def mute_volume(self, mute: bool) -> None: def mute_volume(self, mute: bool) -> None:
@ -184,17 +179,17 @@ class AbstractDemoPlayer(MediaPlayerEntity):
def media_play(self) -> None: def media_play(self) -> None:
"""Send play command.""" """Send play command."""
self._player_state = MediaPlayerState.PLAYING self._attr_state = MediaPlayerState.PLAYING
self.schedule_update_ha_state() self.schedule_update_ha_state()
def media_pause(self) -> None: def media_pause(self) -> None:
"""Send pause command.""" """Send pause command."""
self._player_state = MediaPlayerState.PAUSED self._attr_state = MediaPlayerState.PAUSED
self.schedule_update_ha_state() self.schedule_update_ha_state()
def media_stop(self) -> None: def media_stop(self) -> None:
"""Send stop command.""" """Send stop command."""
self._player_state = MediaPlayerState.OFF self._attr_state = MediaPlayerState.OFF
self.schedule_update_ha_state() self.schedule_update_ha_state()
def set_shuffle(self, shuffle: bool) -> None: def set_shuffle(self, shuffle: bool) -> None:
@ -264,7 +259,7 @@ class DemoYoutubePlayer(AbstractDemoPlayer):
position = self._progress position = self._progress
if self._player_state == MediaPlayerState.PLAYING: if self.state == MediaPlayerState.PLAYING:
position += int( position += int(
(dt_util.utcnow() - self._progress_updated_at).total_seconds() (dt_util.utcnow() - self._progress_updated_at).total_seconds()
) )
@ -277,7 +272,7 @@ class DemoYoutubePlayer(AbstractDemoPlayer):
Returns value from homeassistant.util.dt.utcnow(). Returns value from homeassistant.util.dt.utcnow().
""" """
if self._player_state == MediaPlayerState.PLAYING: if self.state == MediaPlayerState.PLAYING:
return self._progress_updated_at return self._progress_updated_at
return None return None
@ -396,7 +391,7 @@ class DemoMusicPlayer(AbstractDemoPlayer):
"""Clear players playlist.""" """Clear players playlist."""
self.tracks = [] self.tracks = []
self._cur_track = 0 self._cur_track = 0
self._player_state = MediaPlayerState.OFF self._attr_state = MediaPlayerState.OFF
self.schedule_update_ha_state() self.schedule_update_ha_state()
def set_repeat(self, repeat: RepeatMode) -> None: def set_repeat(self, repeat: RepeatMode) -> None: