From b3bfa16b93c7697a0bc3ccad673451b81091a205 Mon Sep 17 00:00:00 2001
From: Josh Patra <joshpatra12@gmail.com>
Date: Tue, 3 Feb 2026 17:35:54 -0500
Subject: [PATCH] Fix RemovePlaylist to use new 4-field config format

- Include JellyfinId when serializing playlists after removal
- Ensures config format stays consistent: [Name,SpotifyId,JellyfinId,position]
- Fixes issue where removed playlists would stay in the list
---
 allstarr/Controllers/AdminController.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/allstarr/Controllers/AdminController.cs b/allstarr/Controllers/AdminController.cs
index 998be69..ff86d6d 100644
--- a/allstarr/Controllers/AdminController.cs
+++ b/allstarr/Controllers/AdminController.cs
@@ -461,9 +461,9 @@ public class AdminController : ControllerBase
         
         currentPlaylists.Remove(playlist);
         
-        // Convert to JSON format for env var
+        // Convert to JSON format for env var: [["Name","SpotifyId","JellyfinId","first|last"],...]
         var playlistsJson = JsonSerializer.Serialize(
-            currentPlaylists.Select(p => new[] { p.Name, p.Id, p.LocalTracksPosition.ToString().ToLower() }).ToArray()
+            currentPlaylists.Select(p => new[] { p.Name, p.Id, p.JellyfinId, p.LocalTracksPosition.ToString().ToLower() }).ToArray()
         );
         
         // Update .env file