From 5f412c2e6a0ba36ceb19c1c504b451a19fb111df Mon Sep 17 00:00:00 2001
From: Flaminel <flm.marius@gmail.com>
Date: Sun, 15 Jun 2025 11:27:16 +0300
Subject: [PATCH] try fix config controller

---
 .../Controllers/ConfigurationController.cs    | 40 ++++++++++++++-----
 .../queue-cleaner-config.store.ts             |  2 +-
 2 files changed, 31 insertions(+), 11 deletions(-)

diff --git a/code/Executable/Controllers/ConfigurationController.cs b/code/Executable/Controllers/ConfigurationController.cs
index be1d8a8d..45881c55 100644
--- a/code/Executable/Controllers/ConfigurationController.cs
+++ b/code/Executable/Controllers/ConfigurationController.cs
@@ -305,8 +305,12 @@ public class ConfigurationController : ControllerBase
             var oldConfig = await _dataContext.QueueCleanerConfigs
                 .FirstAsync();
 
-            // Apply updates from DTO
-            newConfig.Adapt(oldConfig);
+            // Apply updates from DTO, excluding the ID property to avoid EF key modification error
+            var config = new TypeAdapterConfig();
+            config.NewConfig<QueueCleanerConfig, QueueCleanerConfig>()
+                .Ignore(dest => dest.Id);
+            
+            newConfig.Adapt(oldConfig, config);
 
             // Persist the configuration
             await _dataContext.SaveChangesAsync();
@@ -341,8 +345,12 @@ public class ConfigurationController : ControllerBase
                 .Include(x => x.Categories)
                 .FirstAsync();
 
-            // Apply updates from DTO
-            newConfig.Adapt(oldConfig);
+            // Apply updates from DTO, excluding the ID property to avoid EF key modification error
+            var config = new TypeAdapterConfig();
+            config.NewConfig<DownloadCleanerConfig, DownloadCleanerConfig>()
+                .Ignore(dest => dest.Id);
+            
+            newConfig.Adapt(oldConfig, config);
 
             // Persist the configuration
             await _dataContext.SaveChangesAsync();
@@ -415,8 +423,12 @@ public class ConfigurationController : ControllerBase
             var oldConfig = await _dataContext.SonarrConfigs
                 .FirstAsync();
 
-            // Apply updates from DTO
-            newConfig.Adapt(oldConfig);
+            // Apply updates from DTO, excluding the ID property to avoid EF key modification error
+            var config = new TypeAdapterConfig();
+            config.NewConfig<SonarrConfig, SonarrConfig>()
+                .Ignore(dest => dest.Id);
+            
+            newConfig.Adapt(oldConfig, config);
 
             // Persist the configuration
             await _dataContext.SaveChangesAsync();
@@ -447,8 +459,12 @@ public class ConfigurationController : ControllerBase
             var oldConfig = await _dataContext.RadarrConfigs
                 .FirstAsync();
 
-            // Apply updates from DTO
-            newConfig.Adapt(oldConfig);
+            // Apply updates from DTO, excluding the ID property to avoid EF key modification error
+            var config = new TypeAdapterConfig();
+            config.NewConfig<RadarrConfig, RadarrConfig>()
+                .Ignore(dest => dest.Id);
+            
+            newConfig.Adapt(oldConfig, config);
 
             // Persist the configuration
             await _dataContext.SaveChangesAsync();
@@ -479,8 +495,12 @@ public class ConfigurationController : ControllerBase
             var oldConfig = await _dataContext.LidarrConfigs
                 .FirstAsync();
 
-            // Apply updates from DTO
-            newConfig.Adapt(oldConfig);
+            // Apply updates from DTO, excluding the ID property to avoid EF key modification error
+            var config = new TypeAdapterConfig();
+            config.NewConfig<LidarrConfig, LidarrConfig>()
+                .Ignore(dest => dest.Id);
+            
+            newConfig.Adapt(oldConfig, config);
 
             // Persist the configuration
             await _dataContext.SaveChangesAsync();
diff --git a/code/UI/src/app/settings/queue-cleaner/queue-cleaner-config.store.ts b/code/UI/src/app/settings/queue-cleaner/queue-cleaner-config.store.ts
index 5f25d80e..6db2b0ba 100644
--- a/code/UI/src/app/settings/queue-cleaner/queue-cleaner-config.store.ts
+++ b/code/UI/src/app/settings/queue-cleaner/queue-cleaner-config.store.ts
@@ -54,8 +54,8 @@ export class QueueCleanerConfigStore extends signalStore(
         switchMap(config => configService.updateQueueCleanerConfig(config).pipe(
           tap({
             next: () => {
+              // Don't set config - let the form stay as-is with string enum values
               patchState(store, { 
-                config, 
                 saving: false 
               });
             },