From b4bc7333d972a623195fc3df2acc14c790e487ec Mon Sep 17 00:00:00 2001
From: Jonas Kvinge <jonas@jkvinge.net>
Date: Tue, 12 May 2020 15:53:15 +0200
Subject: [PATCH] Use album id as cover filename for Subsonic

Fixes #433
---
 src/covermanager/albumcoverloader.cpp | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/covermanager/albumcoverloader.cpp b/src/covermanager/albumcoverloader.cpp
index fa5d10479..e527ca009 100644
--- a/src/covermanager/albumcoverloader.cpp
+++ b/src/covermanager/albumcoverloader.cpp
@@ -163,12 +163,15 @@ QString AlbumCoverLoader::CoverFilenameFromSource(const Song::Source source, con
 
   switch (source) {
     case Song::Source_Tidal:
-      filename = album_id + "-" + cover_url.fileName();
-      break;
+      if (!album_id.isEmpty()) {
+        filename = album_id + "-" + cover_url.fileName();
+        break;
+      }
+      // fallthrough
     case Song::Source_Subsonic:
     case Song::Source_Qobuz:
-      filename = AlbumCoverFilename(artist, album);
-      if (filename.length() > 8 && (filename.length() - 5) >= (artist.length() + album.length() - 2)) {
+      if (!album_id.isEmpty()) {
+        filename = album_id + ".jpg";
         break;
       }
       // fallthrough