From b8db546f640dfbc50fb69732e0ead7eecdd773e4 Mon Sep 17 00:00:00 2001
From: Thomas Amland <thomas.amland@gmail.com>
Date: Wed, 12 Aug 2020 17:52:50 +0200
Subject: [PATCH] show starred artists and albums

---
 src/library/starred/Starred.vue | 23 ++++++++++++++++++++---
 src/shared/api.ts               | 16 ++++++++++------
 2 files changed, 30 insertions(+), 9 deletions(-)

diff --git a/src/library/starred/Starred.vue b/src/library/starred/Starred.vue
index 2d9accb..a2c3996 100644
--- a/src/library/starred/Starred.vue
+++ b/src/library/starred/Starred.vue
@@ -1,22 +1,39 @@
 <template>
-  <TrackList v-if="items" :tracks="items" />
+  <div v-if="result">
+    <div v-if="result.albums.length > 0" class="mb-4">
+      <h1>Albums</h1>
+      <AlbumList :items="result.albums" />
+    </div>
+    <div v-if="result.artists.length > 0" class="mb-4">
+      <h1>Artists</h1>
+      <ArtistList :items="result.artists" />
+    </div>
+    <div v-if="result.tracks.length > 0" class="mb-4">
+      <h1>Tracks</h1>
+      <TrackList :tracks="result.tracks" />
+    </div>
+  </div>
 </template>
 <script lang="ts">
   import Vue from 'vue'
+  import AlbumList from '@/library/album/AlbumList.vue'
+  import ArtistList from '@/library/artist/ArtistList.vue'
   import TrackList from '@/library/TrackList.vue'
 
   export default Vue.extend({
     components: {
+      AlbumList,
+      ArtistList,
       TrackList,
     },
     data() {
       return {
-        items: null as any,
+        result: null as any,
       }
     },
     created() {
       this.$api.getStarred().then(result => {
-        this.items = result
+        this.result = result
       })
     }
   })
diff --git a/src/shared/api.ts b/src/shared/api.ts
index 00308bf..f82427a 100644
--- a/src/shared/api.ts
+++ b/src/shared/api.ts
@@ -107,7 +107,7 @@ export class API {
       lastFmUrl: info2.lastFmUrl,
       musicBrainzUrl: info2.musicBrainzId
         ? `https://musicbrainz.org/artist/${info2.musicBrainzId}` : null,
-      albums: info1.album.map((album: any) => this.normalizeAlbumResponse(album)),
+      albums: info1.album.map((album: any) => this.normalizeAlbum(album)),
       similarArtist: (info2.similarArtist || []).map((artist: any) => ({
         id: artist.id,
         name: artist.name,
@@ -193,7 +193,11 @@ export class API {
 
   async getStarred() {
     const response = await this.get('rest/getStarred2')
-    return this.normalizeTrackList(response.starred2?.song || [])
+    return {
+      albums: (response.starred2?.album || []).map(this.normalizeAlbum, this),
+      artists: (response.starred2?.artist || []).map(this.normalizeArtist, this),
+      tracks: this.normalizeTrackList(response.starred2?.song || [])
+    }
   }
 
   async star(type: 'track' | 'album' | 'artist', id: string) {
@@ -221,8 +225,8 @@ export class API {
     const data = await this.get('rest/search3', params)
     return {
       tracks: this.normalizeTrackList(data.searchResult3.song || []),
-      albums: (data.searchResult3.album || []).map((x: any) => this.normalizeAlbumResponse(x)),
-      artists: (data.searchResult3.artist || []).map((x: any) => this.normalizeArtistResponse(x)),
+      albums: (data.searchResult3.album || []).map((x: any) => this.normalizeAlbum(x)),
+      artists: (data.searchResult3.artist || []).map((x: any) => this.normalizeArtist(x)),
     }
   }
 
@@ -234,14 +238,14 @@ export class API {
     }))
   }
 
-  private normalizeAlbumResponse(item: any) {
+  private normalizeAlbum(item: any) {
     return {
       ...item,
       image: this.getCoverArtUrl(item)
     }
   }
 
-  private normalizeArtistResponse(item: any) {
+  private normalizeArtist(item: any) {
     return {
       ...item,
       image: this.getCoverArtUrl(item)