From 99a57f5a4e245fd737141348657605ada3c4927b Mon Sep 17 00:00:00 2001 From: Chris Talkington Date: Thu, 17 Sep 2020 09:01:28 -0500 Subject: [PATCH] Check mpd time type before splitting it (#40139) --- homeassistant/components/mpd/media_player.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/homeassistant/components/mpd/media_player.py b/homeassistant/components/mpd/media_player.py index 8a46cef6eb3..845b0ae506b 100644 --- a/homeassistant/components/mpd/media_player.py +++ b/homeassistant/components/mpd/media_player.py @@ -138,12 +138,12 @@ class MpdDevice(MediaPlayerEntity): if position is None: position = self._status.get("time") - if position is not None and ":" in position: + if isinstance(position, str) and ":" in position: position = position.split(":")[0] if position is not None and self._media_position != position: self._media_position_updated_at = dt_util.utcnow() - self._media_position = int(position) + self._media_position = int(float(position)) self._update_playlists() @@ -159,8 +159,9 @@ class MpdDevice(MediaPlayerEntity): self._connect() self._fetch_status() - except (mpd.ConnectionError, OSError, BrokenPipeError, ValueError): + except (mpd.ConnectionError, OSError, BrokenPipeError, ValueError) as error: # Cleanly disconnect in case connection is not in valid state + _LOGGER.debug("Error updating status: %s", error) self._disconnect() @property