From eb83621fce2cb88e6773a26a928a536f5490a237 Mon Sep 17 00:00:00 2001 From: Per Sandstrom Date: Wed, 5 Aug 2015 20:02:39 +0200 Subject: [PATCH] fixing pylint issues --- .../components/media_player/squeezebox.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/homeassistant/components/media_player/squeezebox.py b/homeassistant/components/media_player/squeezebox.py index fa05e304c29..16f1b3ef523 100644 --- a/homeassistant/components/media_player/squeezebox.py +++ b/homeassistant/components/media_player/squeezebox.py @@ -58,8 +58,8 @@ def setup_platform(hass, config, add_devices, discovery_info=None): """ Sets up the squeezebox platform. """ if not config.get(CONF_HOST): _LOGGER.error( - "Missing required configuration items in {}: {}".format( - DOMAIN, CONF_HOST)) + "Missing required configuration items in %s: %s", + DOMAIN, CONF_HOST) return False lms = LogitechMediaServer( @@ -77,6 +77,8 @@ def setup_platform(hass, config, add_devices, discovery_info=None): class LogitechMediaServer(object): + """ Represents a Logitech media server. """ + def __init__(self, host, port, username, password): self.host = host self.port = port @@ -92,17 +94,17 @@ class LogitechMediaServer(object): http_port = self.query('pref', 'httpport', '?') if not http_port: _LOGGER.error( - "Unable to read data from server {}:{}".format( + "Unable to read data from server %s:%s", self.host, - self.port)) + self.port) return return http_port except ConnectionError as ex: _LOGGER.error( - "Failed to connect to server {}:{} - {}".format( + "Failed to connect to server %s:%s - %s", self.host, self.port, - ex)) + ex) return def create_players(self):