Changeset 795c925 in Klonkt for src/services/PlaylistService.js

Timestamp:

06/20/2026 02:20:53 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

1ee967d

Parents:

5258945

Message:

fix(playlist): album/playlist cover falls back to first track cover

PlaylistService.get returned cover= when the playlist had no own cover_url
→ empty card. Now falls back to the cover of the first track that has one.

Co-Authored-By: Claude <noreply@…>

File:

• src/services/PlaylistService.js (modified) (2 diffs)

src/services/PlaylistService.js

@@ -102 +102 @@
     `).all(id);
 
+    const mappedTracks = tracks
+      .filter(t => t.filename)  // skip orphaned references
+      .map(t => ({
+        id: t.id,
+        title: t.title || 'Untitled',
+        artist: t.artist || p.artist || '',
+        cover: t.cover_url || p.cover_url || '',
+        duration: t.duration || 0,
+        url: urlFor ? urlFor(t.filename) : null,
+      }));
+    // Geen eigen cover? Val terug op de eerste track-cover, zodat de kaart niet leeg is.
+    const fallbackCover = (mappedTracks.find(t => t.cover) || {}).cover || '';
     return {
       id: p.id,
@@ -107 +119 @@
       artist: p.artist || '',
       year: p.year || 0,
-      cover: p.cover_url || '',
+      cover: p.cover_url || fallbackCover,
       kind: (p.kind === 'playlist') ? 'playlist' : 'album',
-      tracks: tracks
-        .filter(t => t.filename)  // skip orphaned references
-        .map(t => ({
-          id: t.id,
-          title: t.title || 'Untitled',
-          artist: t.artist || p.artist || '',
-          cover: t.cover_url || p.cover_url || '',
-          duration: t.duration || 0,
-          url: urlFor ? urlFor(t.filename) : null,
-        })),
+      tracks: mappedTracks,
     };
   }