From fb49c588e57056123f8f829a8f8d18fd139deb74 Mon Sep 17 00:00:00 2001 From: John Mihalic Date: Thu, 26 Jan 2017 18:30:42 -0500 Subject: [PATCH] Handle Squeezebox issues (#5566) * Handle Squeezebox issues * Fix double logging --- homeassistant/components/media_player/squeezebox.py | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/homeassistant/components/media_player/squeezebox.py b/homeassistant/components/media_player/squeezebox.py index c338c6dffd8..852ce522559 100644 --- a/homeassistant/components/media_player/squeezebox.py +++ b/homeassistant/components/media_player/squeezebox.py @@ -182,7 +182,7 @@ class SqueezeBoxDevice(MediaPlayerDevice): @property def state(self): """Return the state of the device.""" - if 'power' in self._status and self._status['power'] == '0': + if 'power' in self._status and self._status['power'] == 0: return STATE_OFF if 'mode' in self._status: if self._status['mode'] == 'pause': @@ -213,8 +213,16 @@ class SqueezeBoxDevice(MediaPlayerDevice): "status", "-", "1", "tags:{tags}" .format(tags=tags)) + if response is False: + return + + self._status = response.copy() + + try: + self._status.update(response["playlist_loop"][0]) + except KeyError: + pass try: - self._status = response.copy() self._status.update(response["remoteMeta"]) except KeyError: pass