Fixed additional pylint and flake issues

This commit is contained in:
miniconfig 2015-09-20 16:13:26 -04:00
parent 48306ddbf6
commit 5027acfda1

View file

@ -52,6 +52,8 @@ SUPPORT_PLEX = SUPPORT_PAUSE | SUPPORT_PREVIOUS_TRACK | SUPPORT_NEXT_TRACK
# pylint: disable=abstract-method # pylint: disable=abstract-method
# pylint: disable=unused-argument # pylint: disable=unused-argument
def setup_platform(hass, config, add_devices, discovery_info=None): def setup_platform(hass, config, add_devices, discovery_info=None):
""" Sets up the plex platform. """ """ Sets up the plex platform. """
name = config.get('name', '') name = config.get('name', '')
@ -61,15 +63,15 @@ def setup_platform(hass, config, add_devices, discovery_info=None):
plexserver = plexuser.getResource(name).connect() plexserver = plexuser.getResource(name).connect()
dev = plexserver.clients() dev = plexserver.clients()
for device in dev: for device in dev:
if not "PlayStation" in device.name: if "PlayStation" not in device.name:
add_devices([PlexClient(device.name, plexserver)]) add_devices([PlexClient(device.name, plexserver)])
class PlexClient(MediaPlayerDevice): class PlexClient(MediaPlayerDevice):
""" Represents a Plex device. """ """ Represents a Plex device. """
# pylint: disable=too-many-public-methods # pylint: disable=too-many-public-methods
def __init__(self, name, plexserver): def __init__(self, name, plexserver):
self.client = plexserver.client(name) self.client = plexserver.client(name)
self._name = name self._name = name
@ -114,7 +116,8 @@ class PlexClient(MediaPlayerDevice):
if self._media is None: if self._media is None:
return None return None
else: else:
media_type = self.server.library.getByKey(self.media_content_id).type media_type = self.server.library.getByKey(
self.media_content_id).type
if media_type == 'episode': if media_type == 'episode':
return MEDIA_TYPE_TVSHOW return MEDIA_TYPE_TVSHOW
elif media_type == 'movie': elif media_type == 'movie':
@ -143,11 +146,13 @@ class PlexClient(MediaPlayerDevice):
# find a string we can use as a title # find a string we can use as a title
if self._media is not None: if self._media is not None:
return self.server.library.getByKey(self.media_content_id).title return self.server.library.getByKey(self.media_content_id).title
@property @property
def media_season(self): def media_season(self):
""" Season of curent playing media. (TV Show only) """ """ Season of curent playing media. (TV Show only) """
if self._media is not None: if self._media is not None:
show_season = self.server.library.getByKey(self.media_content_id).season().index show_season = self.server.library.getByKey(
self.media_content_id).season().index
return show_season return show_season
else: else:
return None return None
@ -156,7 +161,8 @@ class PlexClient(MediaPlayerDevice):
def media_series_title(self): def media_series_title(self):
""" Series title of current playing media. (TV Show only)""" """ Series title of current playing media. (TV Show only)"""
if self._media is not None: if self._media is not None:
series_title = self.server.library.getByKey(self.media_content_id).show().title series_title = self.server.library.getByKey(
self.media_content_id).show().title
return series_title return series_title
else: else:
return None return None
@ -165,7 +171,8 @@ class PlexClient(MediaPlayerDevice):
def media_episode(self): def media_episode(self):
""" Episode of current playing media. (TV Show only) """ """ Episode of current playing media. (TV Show only) """
if self._media is not None: if self._media is not None:
show_episode = self.server.library.getByKey(self.media_content_id).index show_episode = self.server.library.getByKey(
self.media_content_id).index
return show_episode return show_episode
else: else:
return None return None