From 624b7447448a63af9b1c80c6d88e9ef2e443c637 Mon Sep 17 00:00:00 2001 From: Quentame Date: Sat, 23 Nov 2019 23:09:09 +0100 Subject: [PATCH] Move philips_js imports at top-level --- .../components/philips_js/media_player.py | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/homeassistant/components/philips_js/media_player.py b/homeassistant/components/philips_js/media_player.py index 579dc253603263..fe6d7edf80401b 100644 --- a/homeassistant/components/philips_js/media_player.py +++ b/homeassistant/components/philips_js/media_player.py @@ -2,11 +2,14 @@ from datetime import timedelta import logging +from haphilipsjs import PhilipsTV import voluptuous as vol -from homeassistant.components.media_player import MediaPlayerDevice, PLATFORM_SCHEMA +from homeassistant.components.media_player import PLATFORM_SCHEMA, MediaPlayerDevice from homeassistant.components.media_player.const import ( + MEDIA_TYPE_CHANNEL, SUPPORT_NEXT_TRACK, + SUPPORT_PLAY_MEDIA, SUPPORT_PREVIOUS_TRACK, SUPPORT_SELECT_SOURCE, SUPPORT_TURN_OFF, @@ -14,8 +17,6 @@ SUPPORT_VOLUME_MUTE, SUPPORT_VOLUME_SET, SUPPORT_VOLUME_STEP, - MEDIA_TYPE_CHANNEL, - SUPPORT_PLAY_MEDIA, ) from homeassistant.const import ( CONF_API_VERSION, @@ -70,20 +71,18 @@ def _inverted(data): def setup_platform(hass, config, add_entities, discovery_info=None): """Set up the Philips TV platform.""" - import haphilipsjs - name = config.get(CONF_NAME) host = config.get(CONF_HOST) api_version = config.get(CONF_API_VERSION) turn_on_action = config.get(CONF_ON_ACTION) - tvapi = haphilipsjs.PhilipsTV(host, api_version) + tvapi = PhilipsTV(host, api_version) on_script = Script(hass, turn_on_action) if turn_on_action else None - add_entities([PhilipsTV(tvapi, name, on_script)]) + add_entities([PhilipsTVMediaPlayer(tvapi, name, on_script)]) -class PhilipsTV(MediaPlayerDevice): +class PhilipsTVMediaPlayer(MediaPlayerDevice): """Representation of a Philips TV exposing the JointSpace API.""" def __init__(self, tv, name, on_script):