From 8ef5ee7d8f80970e078eb4cb027c0c626ed9df25 Mon Sep 17 00:00:00 2001 From: Josh Patra Date: Sat, 31 Jan 2026 19:44:47 -0500 Subject: [PATCH] fix background service to use configured playlist names --- .../Spotify/SpotifyMissingTracksFetcher.cs | 28 ++++++------------- 1 file changed, 8 insertions(+), 20 deletions(-) diff --git a/allstarr/Services/Spotify/SpotifyMissingTracksFetcher.cs b/allstarr/Services/Spotify/SpotifyMissingTracksFetcher.cs index e4744b3..0143fee 100644 --- a/allstarr/Services/Spotify/SpotifyMissingTracksFetcher.cs +++ b/allstarr/Services/Spotify/SpotifyMissingTracksFetcher.cs @@ -111,27 +111,15 @@ public class SpotifyMissingTracksFetcher : BackgroundService { _playlistIdToName.Clear(); - using var scope = _serviceProvider.CreateScope(); - var proxyService = scope.ServiceProvider.GetRequiredService(); - - foreach (var playlistId in _spotifySettings.Value.PlaylistIds) + // Use configured playlist names instead of fetching from API + for (int i = 0; i < _spotifySettings.Value.PlaylistIds.Count; i++) { - try - { - var playlistInfo = await proxyService.GetJsonAsync($"Items/{playlistId}", null, null); - if (playlistInfo != null && playlistInfo.RootElement.TryGetProperty("Name", out var nameElement)) - { - var name = nameElement.GetString() ?? ""; - if (!string.IsNullOrEmpty(name)) - { - _playlistIdToName[playlistId] = name; - } - } - } - catch (Exception ex) - { - _logger.LogWarning(ex, "Failed to get name for playlist {PlaylistId}", playlistId); - } + var playlistId = _spotifySettings.Value.PlaylistIds[i]; + var playlistName = i < _spotifySettings.Value.PlaylistNames.Count + ? _spotifySettings.Value.PlaylistNames[i] + : playlistId; // Fallback to ID if name not configured + + _playlistIdToName[playlistId] = playlistName; } }