From bc7ec0e87628971a0dbbe467d299132fd89f2fba Mon Sep 17 00:00:00 2001
From: Bill Thornton <thornbill@users.noreply.github.com>
Date: Wed, 30 Oct 2024 16:01:58 -0400
Subject: [PATCH 1/2] Remove self cast target

---
 src/components/playback/playbackmanager.js | 26 +++++-----------------
 src/strings/en-us.json                     |  1 -
 2 files changed, 6 insertions(+), 21 deletions(-)

diff --git a/src/components/playback/playbackmanager.js b/src/components/playback/playbackmanager.js
index 86acdff7a6..e68e8fa809 100644
--- a/src/components/playback/playbackmanager.js
+++ b/src/components/playback/playbackmanager.js
@@ -853,29 +853,15 @@ export class PlaybackManager {
             const promises = players.filter(displayPlayerIndividually).map(getPlayerTargets);
 
             return Promise.all(promises).then(function (responses) {
-                return ServerConnections.currentApiClient().getCurrentUser().then(function (user) {
-                    const targets = [];
+                const targets = [];
 
-                    targets.push({
-                        name: globalize.translate('HeaderMyDevice'),
-                        id: 'localplayer',
-                        playerName: 'localplayer',
-                        playableMediaTypes: ['Audio', 'Video', 'Photo', 'Book'],
-                        isLocalPlayer: true,
-                        supportedCommands: self.getSupportedCommands({
-                            isLocalPlayer: true
-                        }),
-                        user: user
-                    });
-
-                    for (const subTargets of responses) {
-                        for (const subTarget of subTargets) {
-                            targets.push(subTarget);
-                        }
+                for (const subTargets of responses) {
+                    for (const subTarget of subTargets) {
+                        targets.push(subTarget);
                     }
+                }
 
-                    return targets.sort(sortPlayerTargets);
-                });
+                return targets.sort(sortPlayerTargets);
             });
         };
 
diff --git a/src/strings/en-us.json b/src/strings/en-us.json
index 096b71c8fc..bf15e1c0d8 100644
--- a/src/strings/en-us.json
+++ b/src/strings/en-us.json
@@ -451,7 +451,6 @@
     "HeaderMetadataSettings": "Metadata Settings",
     "HeaderMoreLikeThis": "More Like This",
     "HeaderMusicQuality": "Music Quality",
-    "HeaderMyDevice": "My Device",
     "HeaderMyMedia": "My Media",
     "HeaderMyMediaSmall": "My Media (small)",
     "HeaderNavigation": "Navigation",

From ca7311cb08cbb1e1f077898e6bdeceb4d463ad8c Mon Sep 17 00:00:00 2001
From: Bill Thornton <thornbill@users.noreply.github.com>
Date: Thu, 21 Nov 2024 16:39:36 -0500
Subject: [PATCH 2/2] Simplify play target sorting

---
 .../components/AppToolbar/menus/RemotePlayMenu.tsx  | 13 ++++++++-----
 src/components/playback/playbackmanager.js          | 13 ++-----------
 2 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/src/apps/experimental/components/AppToolbar/menus/RemotePlayMenu.tsx b/src/apps/experimental/components/AppToolbar/menus/RemotePlayMenu.tsx
index bb53bf6ef5..33eefa72b2 100644
--- a/src/apps/experimental/components/AppToolbar/menus/RemotePlayMenu.tsx
+++ b/src/apps/experimental/components/AppToolbar/menus/RemotePlayMenu.tsx
@@ -65,17 +65,20 @@ const RemotePlayMenu: FC<RemotePlayMenuProps> = ({
             open={open}
             onClose={onMenuClose}
         >
-            {!isChromecastPluginLoaded && ([
-                <MenuItem key='cast-unsupported-item' disabled>
+            {!isChromecastPluginLoaded && (
+                <MenuItem disabled>
                     <ListItemIcon>
                         <Warning />
                     </ListItemIcon>
                     <ListItemText>
                         {globalize.translate('GoogleCastUnsupported')}
                     </ListItemText>
-                </MenuItem>,
-                <Divider key='cast-unsupported-divider' />
-            ])}
+                </MenuItem>
+            )}
+
+            {!isChromecastPluginLoaded && playbackTargets.length > 0 && (
+                <Divider />
+            )}
 
             {playbackTargets.map(target => (
                 <MenuItem
diff --git a/src/components/playback/playbackmanager.js b/src/components/playback/playbackmanager.js
index e68e8fa809..c5f5182cc8 100644
--- a/src/components/playback/playbackmanager.js
+++ b/src/components/playback/playbackmanager.js
@@ -852,17 +852,8 @@ export class PlaybackManager {
         self.getTargets = function () {
             const promises = players.filter(displayPlayerIndividually).map(getPlayerTargets);
 
-            return Promise.all(promises).then(function (responses) {
-                const targets = [];
-
-                for (const subTargets of responses) {
-                    for (const subTarget of subTargets) {
-                        targets.push(subTarget);
-                    }
-                }
-
-                return targets.sort(sortPlayerTargets);
-            });
+            return Promise.all(promises)
+                .then(responses => responses.flat().sort(sortPlayerTargets));
         };
 
         self.playerHasSecondarySubtitleSupport = function (player = self._currentPlayer) {