From a015df7b0156329e245b6bfeab224f70d3e2fb3a Mon Sep 17 00:00:00 2001 From: magnusknutas Date: Thu, 8 Oct 2015 13:41:58 +0200 Subject: [PATCH 1/3] Test for media_content_id KeyError --- homeassistant/components/media_player/kodi.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/homeassistant/components/media_player/kodi.py b/homeassistant/components/media_player/kodi.py index 7bfd385f65b..b25055ec6ec 100644 --- a/homeassistant/components/media_player/kodi.py +++ b/homeassistant/components/media_player/kodi.py @@ -166,8 +166,9 @@ class KodiDevice(MediaPlayerDevice): @property def media_content_id(self): """ Content ID of current playing media. """ + _LOGGER.debug(self._item) if self._item is not None: - return self._item['uniqueid'] + return self._item.get('uniqueid', None) @property def media_content_type(self): From 61c955779b9021d1c0e9de6f57585b41d5a5ff07 Mon Sep 17 00:00:00 2001 From: magnusknutas Date: Thu, 8 Oct 2015 13:55:01 +0200 Subject: [PATCH 2/3] Logging with info --- homeassistant/components/media_player/kodi.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/homeassistant/components/media_player/kodi.py b/homeassistant/components/media_player/kodi.py index b25055ec6ec..6d5d90549a3 100644 --- a/homeassistant/components/media_player/kodi.py +++ b/homeassistant/components/media_player/kodi.py @@ -166,7 +166,7 @@ class KodiDevice(MediaPlayerDevice): @property def media_content_id(self): """ Content ID of current playing media. """ - _LOGGER.debug(self._item) + _LOGGER.info(self._item) if self._item is not None: return self._item.get('uniqueid', None) From 75c3e42064109012fec3e92c36c1f1db5ebb94bb Mon Sep 17 00:00:00 2001 From: magnusknutas Date: Thu, 8 Oct 2015 14:00:23 +0200 Subject: [PATCH 3/3] Removes log for cleanup --- homeassistant/components/media_player/kodi.py | 1 - 1 file changed, 1 deletion(-) diff --git a/homeassistant/components/media_player/kodi.py b/homeassistant/components/media_player/kodi.py index 6d5d90549a3..2fe42e2e707 100644 --- a/homeassistant/components/media_player/kodi.py +++ b/homeassistant/components/media_player/kodi.py @@ -166,7 +166,6 @@ class KodiDevice(MediaPlayerDevice): @property def media_content_id(self): """ Content ID of current playing media. """ - _LOGGER.info(self._item) if self._item is not None: return self._item.get('uniqueid', None)