From aab948ae089c098e0502c9e93138794b3e28507a Mon Sep 17 00:00:00 2001
From: droideparanoico <david.alvarez.81@gmail.com>
Date: Mon, 6 Jul 2026 16:03:25 +0200
Subject: [PATCH] Fix monitoring and removal: restore artist list OOB, improve
 feed reload script

---
 .../java/com/discdrop/web/ArtistController.java  | 16 ++++++++--------
 .../templates/fragments/artist-list.html         | 10 +++++-----
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/src/main/java/com/discdrop/web/ArtistController.java b/src/main/java/com/discdrop/web/ArtistController.java
index 8b7af63..203d38a 100644
--- a/src/main/java/com/discdrop/web/ArtistController.java
+++ b/src/main/java/com/discdrop/web/ArtistController.java
@@ -91,13 +91,15 @@ public class ArtistController {
             results = artistService.search(query);
         }
         var searchHtml = searchResultsTemplate.data("artists", results).render();
-        return Response.ok(searchHtml + feedReloadScript()).build();
+        var allArtists = artistService.findAll();
+        var listHtml = artistListTemplate.data("artists", allArtists).render();
+        return Response.ok(searchHtml + listHtml + feedReloadScript()).build();
     }
 
     @PATCH
     @Path("/{mbid}/monitor")
     @Produces(MediaType.TEXT_HTML)
-    public TemplateInstance updateMonitoring(
+    public Response updateMonitoring(
             @PathParam("mbid") String mbid,
             @QueryParam("type") String type) {
         var existing = artistService.findByMbid(mbid)
@@ -110,11 +112,9 @@ public class ArtistController {
             type.equals("other") ? !existing.otherMonitored() : existing.otherMonitored()
         );
         artistService.updateMonitoring(mbid, flags);
-        var result = feedService.getFeedEntries(1);
-        return feedEntriesTemplate
-            .data("entries", result.entries())
-            .data("hasMore", result.hasMore())
-            .data("nextPage", 2);
+        var allArtists = artistService.findAll();
+        var listHtml = artistListTemplate.data("artists", allArtists).render();
+        return Response.ok(listHtml + feedReloadScript()).build();
     }
 
     @GET
@@ -147,6 +147,6 @@ public class ArtistController {
     }
 
     private String feedReloadScript() {
-        return "<script>htmx.ajax('GET','/api/artists/feed-table',{target:'#feed-entries',swap:'innerHTML'})<\\/script>";
+        return "<script>setTimeout(function(){var e=document.getElementById('feed-entries');if(e)htmx.ajax('GET','/api/artists/feed-table',{target:'#feed-entries',swap:'innerHTML'})},200)</" + "script>";
     }
 }
diff --git a/src/main/resources/templates/fragments/artist-list.html b/src/main/resources/templates/fragments/artist-list.html
index 0a5217c..efaf9d3 100644
--- a/src/main/resources/templates/fragments/artist-list.html
+++ b/src/main/resources/templates/fragments/artist-list.html
@@ -34,7 +34,7 @@
               <input type="checkbox" class="checkbox checkbox-xs checkbox-primary"
                      {#if artist.albumMonitored}checked{/if}
                      hx-patch="/api/artists/{artist.mbid}/monitor?type=album"
-                     hx-target="#feed-entries"
+                     hx-target="#artist-list"
                      hx-swap="innerHTML">
               <span class="label-text text-xs" style="color:var(--p)">Album</span>
             </label>
@@ -42,7 +42,7 @@
               <input type="checkbox" class="checkbox checkbox-xs checkbox-secondary"
                      {#if artist.singleMonitored}checked{/if}
                      hx-patch="/api/artists/{artist.mbid}/monitor?type=single"
-                     hx-target="#feed-entries"
+                     hx-target="#artist-list"
                      hx-swap="innerHTML">
               <span class="label-text text-xs" style="color:var(--s)">Single</span>
             </label>
@@ -50,7 +50,7 @@
               <input type="checkbox" class="checkbox checkbox-xs checkbox-accent"
                      {#if artist.epMonitored}checked{/if}
                      hx-patch="/api/artists/{artist.mbid}/monitor?type=ep"
-                     hx-target="#feed-entries"
+                     hx-target="#artist-list"
                      hx-swap="innerHTML">
               <span class="label-text text-xs" style="color:var(--a)">EP</span>
             </label>
@@ -58,7 +58,7 @@
               <input type="checkbox" class="checkbox checkbox-xs"
                      {#if artist.broadcastMonitored}checked{/if}
                      hx-patch="/api/artists/{artist.mbid}/monitor?type=broadcast"
-                     hx-target="#feed-entries"
+                     hx-target="#artist-list"
                      hx-swap="innerHTML">
               <span class="label-text text-xs text-base-content/60">Broadcast</span>
             </label>
@@ -66,7 +66,7 @@
               <input type="checkbox" class="checkbox checkbox-xs"
                      {#if artist.otherMonitored}checked{/if}
                      hx-patch="/api/artists/{artist.mbid}/monitor?type=other"
-                     hx-target="#feed-entries"
+                     hx-target="#artist-list"
                      hx-swap="innerHTML">
               <span class="label-text text-xs text-base-content/60">Other</span>
             </label>