From 78bc9f9b4bd636f39d8d828dbea30b2381acdf3c Mon Sep 17 00:00:00 2001 From: Mark McDowall Date: Sun, 19 Nov 2023 11:13:11 -0800 Subject: [PATCH] Fixed: Saving indexer, download client, etc settings (cherry picked from commit 804a5921b3b620e2407d5d6a7fd69fb1fd9b0cbf) --- src/Radarr.Api.V3/ProviderControllerBase.cs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/Radarr.Api.V3/ProviderControllerBase.cs b/src/Radarr.Api.V3/ProviderControllerBase.cs index 6df4f30d4..78861c0a0 100644 --- a/src/Radarr.Api.V3/ProviderControllerBase.cs +++ b/src/Radarr.Api.V3/ProviderControllerBase.cs @@ -90,7 +90,9 @@ public ActionResult UpdateProvider([FromBody] TProviderResour var providerDefinition = GetDefinition(providerResource, existingDefinition, true, !forceSave, false); // Comparing via JSON string to eliminate the need for every provider implementation to implement equality checks. - var hasDefinitionChanged = STJson.ToJson(existingDefinition) != STJson.ToJson(providerDefinition); + // Compare settings separately because they are not serialized with the definition. + var hasDefinitionChanged = STJson.ToJson(existingDefinition) != STJson.ToJson(providerDefinition) || + STJson.ToJson(existingDefinition.Settings) != STJson.ToJson(providerDefinition.Settings); // Only test existing definitions if it is enabled and forceSave isn't set or the definition has changed. if (providerDefinition.Enable && (!forceSave || hasDefinitionChanged))