From a5935bce414fce4e9f69fcc4358a82b60b3ada2e Mon Sep 17 00:00:00 2001
From: James Barnsley <james@barnsley.nz>
Date: Wed, 25 Jan 2023 20:09:49 +1300
Subject: [PATCH] Always set `loading: false` - We don't always get a result,
 but regardless we aren't loading any more - Only format playlist result if it
 is an object - Fixes #882

---
 src/js/services/mopidy/middleware.js | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/js/services/mopidy/middleware.js b/src/js/services/mopidy/middleware.js
index 91fd4f00..ac186365 100755
--- a/src/js/services/mopidy/middleware.js
+++ b/src/js/services/mopidy/middleware.js
@@ -1899,11 +1899,15 @@ const MopidyMiddleware = (function () {
               playlistsToLoad.map((uri) => {
                 request(store, 'playlists.lookup', { uri })
                   .then((playlist) => {
-                    playlists[uri] = formatPlaylist({
-                      name: playlist.name,
-                      uri: playlist.uri,
-                      tracks: formatTracks(playlist.tracks),
-                    });
+                    playlists[uri] = {
+                      uri,
+                      loading: false,
+                      ...playlist ? formatPlaylist({
+                        name: playlist.name,
+                        uri: playlist.uri,
+                        tracks: formatTracks(playlist.tracks),
+                      }) : {},
+                    };
                     loaded += 1;
                     if (loaded === toLoad) {
                       resolve(playlists);