From 16fe79db754d4b7f69165eb4f4884fe8e670af3e Mon Sep 17 00:00:00 2001
From: micha91 <michael.harbarth@gmx.de>
Date: Tue, 27 Jun 2023 20:34:04 +0200
Subject: [PATCH] Improve the source labels for MusicCast players (#74954)

---
 .../yamaha_musiccast/media_player.py          | 29 +++++++++++++++++--
 1 file changed, 26 insertions(+), 3 deletions(-)

diff --git a/homeassistant/components/yamaha_musiccast/media_player.py b/homeassistant/components/yamaha_musiccast/media_player.py
index 05518a6c3c9..cf6feb44fbd 100644
--- a/homeassistant/components/yamaha_musiccast/media_player.py
+++ b/homeassistant/components/yamaha_musiccast/media_player.py
@@ -155,6 +155,27 @@ class MusicCastMediaPlayer(MusicCastDeviceEntity, MediaPlayerEntity):
             return MediaPlayerState.PLAYING
         return MediaPlayerState.OFF
 
+    @property
+    def source_mapping(self):
+        """Return a mapping of the actual source names to their labels configured in the MusicCast App."""
+        ret = {}
+        for inp in self.coordinator.data.zones[self._zone_id].input_list:
+            label = self.coordinator.data.input_names.get(inp, "")
+            if inp != label and (
+                label in self.coordinator.data.zones[self._zone_id].input_list
+                or list(self.coordinator.data.input_names.values()).count(label) > 1
+            ):
+                label += f" ({inp})"
+            if label == "":
+                label = inp
+            ret[inp] = label
+        return ret
+
+    @property
+    def reverse_source_mapping(self):
+        """Return a mapping from the source label to the source name."""
+        return {v: k for k, v in self.source_mapping.items()}
+
     @property
     def volume_level(self):
         """Return the volume level of the media player (0..1)."""
@@ -491,17 +512,19 @@ class MusicCastMediaPlayer(MusicCastDeviceEntity, MediaPlayerEntity):
 
     async def async_select_source(self, source: str) -> None:
         """Select input source."""
-        await self.coordinator.musiccast.select_source(self._zone_id, source)
+        await self.coordinator.musiccast.select_source(
+            self._zone_id, self.reverse_source_mapping.get(source, source)
+        )
 
     @property
     def source(self):
         """Name of the current input source."""
-        return self.coordinator.data.zones[self._zone_id].input
+        return self.source_mapping.get(self.coordinator.data.zones[self._zone_id].input)
 
     @property
     def source_list(self):
         """List of available input sources."""
-        return self.coordinator.data.zones[self._zone_id].input_list
+        return list(self.source_mapping.values())
 
     @property
     def media_duration(self):