From 09e08831f9dd36cffc601ab053a6a2631f9e58c5 Mon Sep 17 00:00:00 2001
From: Broque Thomas <26755000+Nezreka@users.noreply.github.com>
Date: Wed, 15 Apr 2026 15:26:52 -0700
Subject: [PATCH] Fix discovery modal footer stuck on 'Discovering...' after
 completion
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The `complete` flag from polling responses was never forwarded into the
transformed status object passed to `updateYouTubeDiscoveryModal`, so
the `if (status.complete)` block that swaps the footer from the
'Discovering...' spinner to the Sync / Download Missing buttons never
fired. Fixed for all three affected sources: Tidal, Deezer, and Spotify
Public — both the WebSocket and HTTP polling paths for each.
---
 webui/static/script.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/webui/static/script.js b/webui/static/script.js
index 66d437be..eca69a84 100644
--- a/webui/static/script.js
+++ b/webui/static/script.js
@@ -26143,6 +26143,7 @@ function startTidalDiscoveryPolling(fakeUrlHash, playlistId) {
             // Transform to YouTube modal format
             const transformed = {
                 progress: data.progress, spotify_matches: data.spotify_matches, spotify_total: data.spotify_total,
+                complete: data.complete,
                 results: (data.results || []).map((r, i) => {
                     const isFound = r.status === 'found' || r.status === '✅ Found' || r.status_class === 'found' || r.spotify_data || r.spotify_track;
                     return {
@@ -26197,6 +26198,7 @@ function startTidalDiscoveryPolling(fakeUrlHash, playlistId) {
                 progress: status.progress,
                 spotify_matches: status.spotify_matches,
                 spotify_total: status.spotify_total,
+                complete: status.complete,
                 results: status.results.map((result, index) => {
                     const isFound = result.status === 'found' ||
                         result.status === '✅ Found' ||
@@ -27586,6 +27588,7 @@ function startDeezerDiscoveryPolling(fakeUrlHash, playlistId) {
             }
             const transformed = {
                 progress: data.progress, spotify_matches: data.spotify_matches, spotify_total: data.spotify_total,
+                complete: data.complete,
                 results: (data.results || []).map((r, i) => {
                     const isFound = r.status === 'found' || r.status === '✅ Found' || r.status_class === 'found' || r.spotify_data || r.spotify_track;
                     return {
@@ -27639,6 +27642,7 @@ function startDeezerDiscoveryPolling(fakeUrlHash, playlistId) {
                 progress: status.progress,
                 spotify_matches: status.spotify_matches,
                 spotify_total: status.spotify_total,
+                complete: status.complete,
                 results: status.results.map((result, index) => {
                     const isFound = result.status === 'found' ||
                         result.status === '✅ Found' ||
@@ -31468,6 +31472,7 @@ function startSpotifyPublicDiscoveryPolling(fakeUrlHash, urlHash) {
             }
             const transformed = {
                 progress: data.progress, spotify_matches: data.spotify_matches, spotify_total: data.spotify_total,
+                complete: data.complete,
                 results: (data.results || []).map((r, i) => {
                     const isFound = r.status === 'found' || r.status === '✅ Found' || r.status_class === 'found' || r.spotify_data || r.spotify_track;
                     return {
@@ -31521,6 +31526,7 @@ function startSpotifyPublicDiscoveryPolling(fakeUrlHash, urlHash) {
                 progress: status.progress,
                 spotify_matches: status.spotify_matches,
                 spotify_total: status.spotify_total,
+                complete: status.complete,
                 results: status.results.map((result, index) => {
                     const isFound = result.status === 'found' ||
                         result.status === '✅ Found' ||