From 209650dca199d1ebc48aea825003b91cfa4cbf5d Mon Sep 17 00:00:00 2001 From: luar123 <49960470+luar123@users.noreply.github.com> Date: Mon, 15 May 2023 21:23:24 +0200 Subject: [PATCH] Remove snapcast friendly_name attribute (#93116) Remove friendly_name attribute and fix names --- homeassistant/components/snapcast/media_player.py | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/homeassistant/components/snapcast/media_player.py b/homeassistant/components/snapcast/media_player.py index 4fd7c587d40..bb54bfabf9f 100644 --- a/homeassistant/components/snapcast/media_player.py +++ b/homeassistant/components/snapcast/media_player.py @@ -175,7 +175,7 @@ class SnapcastGroupDevice(MediaPlayerEntity): @property def name(self): """Return the name of the device.""" - return f"{GROUP_PREFIX}{self._group.identifier}" + return f"{self._group.friendly_name} {GROUP_SUFFIX}" @property def source(self): @@ -197,12 +197,6 @@ class SnapcastGroupDevice(MediaPlayerEntity): """List of available input sources.""" return list(self._group.streams_by_name().keys()) - @property - def extra_state_attributes(self): - """Return the state attributes.""" - name = f"{self._group.friendly_name} {GROUP_SUFFIX}" - return {"friendly_name": name} - async def async_select_source(self, source: str) -> None: """Set input source.""" streams = self._group.streams_by_name() @@ -270,7 +264,7 @@ class SnapcastClientDevice(MediaPlayerEntity): @property def name(self): """Return the name of the device.""" - return f"{CLIENT_PREFIX}{self._client.identifier}" + return f"{self._client.friendly_name} {CLIENT_SUFFIX}" @property def source(self): @@ -307,8 +301,6 @@ class SnapcastClientDevice(MediaPlayerEntity): state_attrs = {} if self.latency is not None: state_attrs["latency"] = self.latency - name = f"{self._client.friendly_name} {CLIENT_SUFFIX}" - state_attrs["friendly_name"] = name return state_attrs @property