From 882914d2718239bfe5ddfefbb7582c4bdcb80201 Mon Sep 17 00:00:00 2001
From: droideparanoico <david.alvarez.81@gmail.com>
Date: Fri, 3 Jul 2026 12:21:29 +0200
Subject: [PATCH] Fix settings save: use @FormParam instead of @QueryParam for
 PATCH body

---
 src/main/java/com/discdrop/web/SettingsController.java | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/main/java/com/discdrop/web/SettingsController.java b/src/main/java/com/discdrop/web/SettingsController.java
index 093e69d..63d61de 100644
--- a/src/main/java/com/discdrop/web/SettingsController.java
+++ b/src/main/java/com/discdrop/web/SettingsController.java
@@ -8,6 +8,7 @@ import jakarta.inject.Inject;
 import jakarta.ws.rs.*;
 import jakarta.ws.rs.core.MediaType;
 import jakarta.ws.rs.core.Response;
+import java.util.List;
 
 @Path("/api/settings")
 public class SettingsController {
@@ -30,12 +31,13 @@ public class SettingsController {
     }
 
     @PATCH
+    @Consumes(MediaType.APPLICATION_FORM_URLENCODED)
     @Produces(MediaType.TEXT_HTML)
     public Response saveSettings(
-            @QueryParam("cache_ttl_hours") @DefaultValue("8") String cacheTtl,
-            @QueryParam("default_types") @DefaultValue("Album") String defaultTypes) {
+            @FormParam("cache_ttl_hours") @DefaultValue("8") String cacheTtl,
+            @FormParam("default_types") List<String> defaultTypes) {
         db.setSetting("cache_ttl_hours", cacheTtl);
-        db.setSetting("default_types", defaultTypes);
+        db.setSetting("default_types", defaultTypes != null ? String.join(",", defaultTypes) : "Album");
         return Response.ok("<div class=\"alert alert-success\">Settings saved</div>")
             .build();
     }