From 8c15b0b708d90ae4b4c866c885cfa8cd4bc0b32e Mon Sep 17 00:00:00 2001
From: Thomas Amland <thomas.amland@gmail.com>
Date: Thu, 24 Oct 2024 12:23:43 +0200
Subject: [PATCH] add description, lastfm and musicbrainz url to album page

---
 src/library/album/AlbumDetails.vue | 55 ++++++++++++++++++++++--------
 src/shared/api.ts                  | 15 ++++++--
 2 files changed, 54 insertions(+), 16 deletions(-)

diff --git a/src/library/album/AlbumDetails.vue b/src/library/album/AlbumDetails.vue
index db5e27d..591bea6 100644
--- a/src/library/album/AlbumDetails.vue
+++ b/src/library/album/AlbumDetails.vue
@@ -7,21 +7,42 @@
           <Icon :icon="isFavourite ? 'heart-fill' : 'heart'" />
         </b-button>
       </h1>
-      <p>
-        by
-        <template v-for="(artist, index) in album.artists">
-          <span v-if="index > 0" :key="artist.id" class="text-muted">, </span>
-          <router-link :key="`${artist.id}-link`" :to="{name: 'artist', params: { id: artist.id }}">{{ artist.name }}</router-link>
-        </template>
-        <span v-if="album.year"> • {{ album.year }}</span>
-        <span v-if="album.genres.length"> •
-          <template v-for="({ name: genre }, index) in album.genres">
-            <span v-if="index > 0" :key="genre" class="text-muted">, </span>
-            <router-link :key="`${genre}-link`" :to="{name: 'genre', params: { id: genre }}">{{ genre }}</router-link>
-          </template>
+      <div class="d-flex flex-wrap align-items-center">
+        <div>
+          by
+          <span v-for="(artist, index) in album.artists" :key="artist.id">
+            <span v-if="index > 0">, </span>
+            <router-link :to="{name: 'artist', params: { id: artist.id }}">
+              {{ artist.name }}
+            </router-link>
+          </span>
+        </div>
+        <span v-if="album.year" class="mx-1"> • {{ album.year }}</span>
+        <span class="mr-3" />
+        <div class="d-flex flex-nowrap">
+          <ExternalLink v-if="album.lastFmUrl" :href="album.lastFmUrl" class="btn btn-link p-0 mr-2" title="Last.fm">
+            <IconLastFm />
+          </ExternalLink>
+          <ExternalLink v-if="album.musicBrainzUrl" :href="album.musicBrainzUrl" class="btn btn-link mr-2 p-0" title="MusicBrainz">
+            <IconMusicBrainz />
+          </ExternalLink>
+        </div>
+      </div>
+
+      <div v-if="album.genres.length">
+        <span v-for="({ name: genre }, index) in album.genres" :key="genre">
+          <span v-if="index > 0">•</span>
+          <router-link :to="{name: 'genre', params: { id: genre }}">
+            {{ genre }}
+          </router-link>
         </span>
-      </p>
-      <div class="text-nowrap">
+      </div>
+
+      <OverflowFade v-if="album.description" class="mt-3">
+        {{ album.description }}
+      </OverflowFade>
+
+      <div class="text-nowrap mt-3">
         <b-button variant="secondary" class="mr-2" @click="playNow">
           <Icon icon="play" /> Play
         </b-button>
@@ -50,9 +71,15 @@
   import TrackList from '@/library/track/TrackList.vue'
   import { Album } from '@/shared/api'
   import { useFavouriteStore } from '@/library/favourite/store'
+  import IconLastFm from '@/shared/components/IconLastFm.vue'
+  import IconMusicBrainz from '@/shared/components/IconMusicBrainz.vue'
+  import OverflowFade from '@/shared/components/OverflowFade.vue'
 
   export default defineComponent({
     components: {
+      OverflowFade,
+      IconMusicBrainz,
+      IconLastFm,
       TrackList,
     },
     props: {
diff --git a/src/shared/api.ts b/src/shared/api.ts
index 59a2b63..e854a6a 100644
--- a/src/shared/api.ts
+++ b/src/shared/api.ts
@@ -33,11 +33,14 @@ export interface Genre {
 export interface Album {
   id: string
   name: string
+  description?: string
   artists: {name: string, id: string}[]
   year: number
   favourite: boolean
   genres: Genre[]
   image?: string
+  lastFmUrl?: string
+  musicBrainzUrl?: string
   tracks?: Track[]
 }
 
@@ -244,8 +247,11 @@ export class API {
 
   async getAlbumDetails(id: string): Promise<Album> {
     const params = { id }
-    const data = await this.fetch('rest/getAlbum', params)
-    return this.normalizeAlbum(data.album)
+    const [info, info2] = await Promise.all([
+      this.fetch('rest/getAlbum', params),
+      this.fetch('rest/getAlbumInfo2', params),
+    ])
+    return this.normalizeAlbum({ ...info.album, ...info2.albumInfo })
   }
 
   async getPlaylists() {
@@ -549,6 +555,7 @@ export class API {
     return {
       id: item.id,
       name: item.name,
+      description: (item.notes || '').replace(/<a[^>]*>.*?<\/a>/gm, ''),
       artists: item.artists?.length
         ? item.artists
         : [{ id: item.artistId, name: item.artist }],
@@ -556,6 +563,10 @@ export class API {
       year: item.year || 0,
       favourite: !!item.starred,
       genres: this.normalizeGenres(item),
+      lastFmUrl: item.lastFmUrl,
+      musicBrainzUrl: item.musicBrainzId
+        ? `https://musicbrainz.org/album/${item.musicBrainzId}`
+        : undefined,
       tracks: (item.song || []).map(this.normalizeTrack, this)
     }
   }