From 78144bc6dedea0bf53ff8f77f9f5f74ffab858e9 Mon Sep 17 00:00:00 2001 From: Teemu R Date: Fri, 16 Mar 2018 12:14:21 +0100 Subject: [PATCH] Use the first, not the last volume controller when multiple are available on songpal (#13222) * use the first, not the last volume controller * Do not mutate the list but simply pick the first by index --- homeassistant/components/media_player/songpal.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/homeassistant/components/media_player/songpal.py b/homeassistant/components/media_player/songpal.py index b1dc7df3319..e43f5951db7 100644 --- a/homeassistant/components/media_player/songpal.py +++ b/homeassistant/components/media_player/songpal.py @@ -154,7 +154,7 @@ class SongpalDevice(MediaPlayerDevice): _LOGGER.warning("Got %s volume controls, using the first one", volumes) - volume = volumes.pop() + volume = volumes[0] _LOGGER.debug("Current volume: %s", volume) self._volume_max = volume.maxVolume