From b4774cd77fcaf71417bf41833d385aa465ddd176 Mon Sep 17 00:00:00 2001
From: jcorporation <mail@jcgames.de>
Date: Sun, 15 Sep 2024 11:49:18 +0200
Subject: [PATCH] Upd: Show placeholder image by playlist type

---
 docs/references/webserver-uris.md |  2 +-
 htdocs/js/viewBrowsePlaylist.js   |  3 ++-
 htdocs/js/views.js                |  2 +-
 src/web_server/playlistart.c      | 12 +++++++++++-
 4 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/docs/references/webserver-uris.md b/docs/references/webserver-uris.md
index b4edc7472..b9ffbe533 100644
--- a/docs/references/webserver-uris.md
+++ b/docs/references/webserver-uris.md
@@ -17,7 +17,7 @@ Reference of all webserver uris.
 | `/browse/` | Prints the list of [published directories]({{ site.baseurl }}/references/published-directories) |
 | `/ca.crt` | Returns the myMPD CA certificate |
 | `/folderart?path=<path>` | Returns the folderart thumbnail. |
-| `/playlistart?playlist=<playlist name>` | Returns the playlistart thumbnail. |
+| `/playlistart?type=<plist,smartpls>&playlist=<playlist name>` | Returns the playlistart thumbnail or a redirect to the placeholder image if not found. |
 | `/proxy?uri=<uri>` | Fetches the response from the uri (GET), allowed hosts: `jcorporation.github.io`, `musicbrainz.org`, `listenbrainz.org` |
 | `/script/<partition>/<script>` | Executes a script (Script should return a valid http response) |
 | `/script-api/<partition>` | Jsonrpc api endpoint for mympd-script |
diff --git a/htdocs/js/viewBrowsePlaylist.js b/htdocs/js/viewBrowsePlaylist.js
index e8c0f70cd..4320f74c7 100644
--- a/htdocs/js/viewBrowsePlaylist.js
+++ b/htdocs/js/viewBrowsePlaylist.js
@@ -190,7 +190,8 @@ function parsePlaylistDetail(obj) {
 
     if (obj.result.pics === true) {
         const img = elCreateEmpty('div', {});
-        img.style.backgroundImage = getCssImageUri('/playlistart?playlist=' + myEncodeURIComponent(obj.result.plist));
+        img.style.backgroundImage = getCssImageUri('/playlistart?type=' +
+            (obj.result.smartpls === false ? 'plist' : 'smartpls') + '&playlist=' + myEncodeURIComponent(obj.result.plist));
         img.addEventListener('click', function(event) {
             zoomPicture(event.target);
         }, false);
diff --git a/htdocs/js/views.js b/htdocs/js/views.js
index f137f5e66..6c3cfc348 100644
--- a/htdocs/js/views.js
+++ b/htdocs/js/views.js
@@ -702,7 +702,7 @@ function setEntryData(entry, data) {
             break;
         case 'plist':
         case 'smartpls':
-            data.Thumbnail = getCssImageUri('/playlistart?playlist=' + myEncodeURIComponent(data.uri));
+            data.Thumbnail = getCssImageUri('/playlistart?type=' + data.Type + '&playlist=' + myEncodeURIComponent(data.uri));
             break;
         case 'webradiodb':
             data.Thumbnail = getCssImageUri(data.Image);
diff --git a/src/web_server/playlistart.c b/src/web_server/playlistart.c
index f4343d1f4..fe9ea366c 100644
--- a/src/web_server/playlistart.c
+++ b/src/web_server/playlistart.c
@@ -30,14 +30,18 @@ bool request_handler_playlistart(struct mg_connection *nc, struct mg_http_messag
 {
     struct t_config *config = mg_user_data->config;
     sds name = get_uri_param(&hm->query, "playlist=");
+    sds type = get_uri_param(&hm->query, "type=");
 
     if (name == NULL ||
+        type == NULL ||
         sdslen(name) == 0 ||
+        sdslen(type) == 0 ||
         vcb_isfilepath(name) == false)
     {
         MYMPD_LOG_ERROR(NULL, "Failed to decode query");
         webserver_redirect_placeholder_image(nc, PLACEHOLDER_PLAYLIST);
         FREE_SDS(name);
+        FREE_SDS(type);
         return false;
     }
     strip_file_extension(name);
@@ -53,9 +57,15 @@ bool request_handler_playlistart(struct mg_connection *nc, struct mg_http_messag
     }
     else {
         MYMPD_LOG_DEBUG(NULL, "No image for playlist found");
-        webserver_redirect_placeholder_image(nc, PLACEHOLDER_PLAYLIST);
+        if (strcmp(type, "smartpls") == 0) {
+            webserver_redirect_placeholder_image(nc, PLACEHOLDER_SMARTPLS);
+        }
+        else {
+            webserver_redirect_placeholder_image(nc, PLACEHOLDER_PLAYLIST);
+        }
     }
     FREE_SDS(mediafile);
     FREE_SDS(name);
+    FREE_SDS(type);
     return true;
 }