evlli/hass-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Keep track of already added players (#7149)

Browse source
This commit is contained in:
Erik Eriksson 2017-04-19 00:20:52 +02:00 • committed by Pascal Vizeli
parent d007269ecc
commit 1e758ed030
1 changed files with 5 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

17
homeassistant/components/media_player/squeezebox.py
View file

@ -29,8 +29,6 @@ _LOGGER = logging.getLogger(__name__)
DEFAULT_PORT = 9000 DEFAULT_PORT = 9000
TIMEOUT = 10 TIMEOUT = 10
KNOWN_DEVICES = []
SUPPORT_SQUEEZEBOX = SUPPORT_PAUSE | SUPPORT_VOLUME_SET | \ SUPPORT_SQUEEZEBOX = SUPPORT_PAUSE | SUPPORT_VOLUME_SET | \
SUPPORT_VOLUME_MUTE | SUPPORT_PREVIOUS_TRACK | SUPPORT_NEXT_TRACK | \ SUPPORT_VOLUME_MUTE | SUPPORT_PREVIOUS_TRACK | SUPPORT_NEXT_TRACK | \
SUPPORT_SEEK | SUPPORT_TURN_ON | SUPPORT_TURN_OFF | SUPPORT_PLAY_MEDIA | \ SUPPORT_SEEK | SUPPORT_TURN_ON | SUPPORT_TURN_OFF | SUPPORT_PLAY_MEDIA | \
@ -71,18 +69,8 @@ def async_setup_platform(hass, config, async_add_devices, discovery_info=None):
host, port, error) host, port, error)
return False return False
# Combine it with port to allow multiple servers at the same host
key = "{}:{}".format(ipaddr, port)
# Only add a media server once
if key in KNOWN_DEVICES:
return False
KNOWN_DEVICES.append(key)
_LOGGER.debug("Creating LMS object for %s", ipaddr) _LOGGER.debug("Creating LMS object for %s", ipaddr)
lms = LogitechMediaServer(hass, host, port, username, password) lms = LogitechMediaServer(hass, host, port, username, password)
if lms is False:
return False
players = yield from lms.create_players() players = yield from lms.create_players()
async_add_devices(players) async_add_devices(players)
@ -173,6 +161,11 @@ class SqueezeBoxDevice(MediaPlayerDevice):
"""Return the name of the device.""" """Return the name of the device."""
return self._name return self._name
@property
def unique_id(self):
"""Return an unique ID."""
return self._id
@property @property
def state(self): def state(self):
"""Return the state of the device.""" """Return the state of the device."""