From c70388869b3f32650dd8bc492807ed6b5d866a4f Mon Sep 17 00:00:00 2001 From: jjlawren Date: Sun, 4 Dec 2022 20:11:30 -0600 Subject: [PATCH] Do not discard zero positions for Sonos media players (#83249) fixes undefined --- homeassistant/components/sonos/media.py | 6 +++--- homeassistant/components/sonos/media_player.py | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/homeassistant/components/sonos/media.py b/homeassistant/components/sonos/media.py index 24233b1316f..ab34457e3fc 100644 --- a/homeassistant/components/sonos/media.py +++ b/homeassistant/components/sonos/media.py @@ -43,11 +43,11 @@ DURATION_SECONDS = "duration_in_s" POSITION_SECONDS = "position_in_s" -def _timespan_secs(timespan: str | None) -> None | float: +def _timespan_secs(timespan: str | None) -> None | int: """Parse a time-span into number of seconds.""" if timespan in UNAVAILABLE_VALUES: return None - return time_period_str(timespan).total_seconds() # type: ignore[arg-type] + return int(time_period_str(timespan).total_seconds()) # type: ignore[arg-type] class SonosMedia: @@ -73,7 +73,7 @@ class SonosMedia: self.title: str | None = None self.uri: str | None = None - self.position: float | None = None + self.position: int | None = None self.position_updated_at: datetime.datetime | None = None def clear(self) -> None: diff --git a/homeassistant/components/sonos/media_player.py b/homeassistant/components/sonos/media_player.py index 4195f284ffe..bd50a090175 100644 --- a/homeassistant/components/sonos/media_player.py +++ b/homeassistant/components/sonos/media_player.py @@ -323,7 +323,7 @@ class SonosMediaPlayerEntity(SonosEntity, MediaPlayerEntity): @property def media_position(self) -> int | None: """Position of current playing media in seconds.""" - return int(self.media.position) if self.media.position else None + return self.media.position @property def media_position_updated_at(self) -> datetime.datetime | None: