From 8c4a7f4f91d19f0a152bcd248f1e7e4f5b7f3921 Mon Sep 17 00:00:00 2001
From: Kendall Garner <17521368+kgarner7@users.noreply.github.com>
Date: Tue, 24 Jun 2025 17:58:43 -0700
Subject: [PATCH] only show lastfm/listenbrainz if configured

---
 .../components/album-detail-content.tsx       | 40 ++++++++++---------
 .../album-artist-detail-content.tsx           | 38 +++++++++---------
 2 files changed, 41 insertions(+), 37 deletions(-)

diff --git a/src/renderer/features/albums/components/album-detail-content.tsx b/src/renderer/features/albums/components/album-detail-content.tsx
index 939d4bf3..4dd49e03 100644
--- a/src/renderer/features/albums/components/album-detail-content.tsx
+++ b/src/renderer/features/albums/components/album-detail-content.tsx
@@ -400,25 +400,27 @@ export const AlbumDetailContent = ({ background, tableRef }: AlbumDetailContentP
                 {externalLinks && (lastFM || musicBrainz) ? (
                     <section>
                         <Group gap="sm">
-                            <ActionIcon
-                                component="a"
-                                href={`https://www.last.fm/music/${encodeURIComponent(
-                                    detailQuery?.data?.albumArtist || '',
-                                )}/${encodeURIComponent(detailQuery.data?.name || '')}`}
-                                icon="brandLastfm"
-                                iconProps={{
-                                    fill: 'default',
-                                    size: 'xl',
-                                }}
-                                radius="md"
-                                rel="noopener noreferrer"
-                                target="_blank"
-                                tooltip={{
-                                    label: t('action.openIn.lastfm'),
-                                }}
-                                variant="subtle"
-                            />
-                            {mbzId ? (
+                            {lastFM && (
+                                <ActionIcon
+                                    component="a"
+                                    href={`https://www.last.fm/music/${encodeURIComponent(
+                                        detailQuery?.data?.albumArtist || '',
+                                    )}/${encodeURIComponent(detailQuery.data?.name || '')}`}
+                                    icon="brandLastfm"
+                                    iconProps={{
+                                        fill: 'default',
+                                        size: 'xl',
+                                    }}
+                                    radius="md"
+                                    rel="noopener noreferrer"
+                                    target="_blank"
+                                    tooltip={{
+                                        label: t('action.openIn.lastfm'),
+                                    }}
+                                    variant="subtle"
+                                />
+                            )}
+                            {mbzId && musicBrainz ? (
                                 <ActionIcon
                                     component="a"
                                     href={`https://musicbrainz.org/release/${mbzId}`}
diff --git a/src/renderer/features/artists/components/album-artist-detail-content.tsx b/src/renderer/features/artists/components/album-artist-detail-content.tsx
index 1e0c5e9f..4cdac2df 100644
--- a/src/renderer/features/artists/components/album-artist-detail-content.tsx
+++ b/src/renderer/features/artists/components/album-artist-detail-content.tsx
@@ -439,24 +439,26 @@ export const AlbumArtistDetailContent = ({ background }: AlbumArtistDetailConten
                 {externalLinks && (lastFM || musicBrainz) ? (
                     <section>
                         <Group gap="sm">
-                            <ActionIcon
-                                component="a"
-                                href={`https://www.last.fm/music/${encodeURIComponent(
-                                    detailQuery?.data?.name || '',
-                                )}`}
-                                icon="brandLastfm"
-                                iconProps={{
-                                    fill: 'default',
-                                    size: 'xl',
-                                }}
-                                rel="noopener noreferrer"
-                                target="_blank"
-                                tooltip={{
-                                    label: t('action.openIn.lastfm'),
-                                }}
-                                variant="subtle"
-                            />
-                            {mbzId ? (
+                            {lastFM && (
+                                <ActionIcon
+                                    component="a"
+                                    href={`https://www.last.fm/music/${encodeURIComponent(
+                                        detailQuery?.data?.name || '',
+                                    )}`}
+                                    icon="brandLastfm"
+                                    iconProps={{
+                                        fill: 'default',
+                                        size: 'xl',
+                                    }}
+                                    rel="noopener noreferrer"
+                                    target="_blank"
+                                    tooltip={{
+                                        label: t('action.openIn.lastfm'),
+                                    }}
+                                    variant="subtle"
+                                />
+                            )}
+                            {mbzId && musicBrainz ? (
                                 <ActionIcon
                                     component="a"
                                     href={`https://musicbrainz.org/artist/${mbzId}`}