self-hosted/Sonarr
1
0
Fork 0
mirror of https://github.com/Sonarr/Sonarr.git synced 2025-01-10 23:29:53 +02:00
Code Issues Releases Activity

Fixed: Saving indexer, download client, etc settings

Browse Source
This commit is contained in:
Mark McDowall 2023-11-19 11:13:11 -08:00
parent 87ecbf39c1
commit 804a5921b3
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Sonarr.Api.V3/ProviderControllerBase.cs
View File

@ -91,7 +91,9 @@ public ActionResult<TProviderResource> UpdateProvider([FromBody] TProviderResour
var providerDefinition = GetDefinition(providerResource, existingDefinition, true, !forceSave, false); var providerDefinition = GetDefinition(providerResource, existingDefinition, true, !forceSave, false);
// Comparing via JSON string to eliminate the need for every provider implementation to implement equality checks. // 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. // Only test existing definitions if it is enabled and forceSave isn't set or the definition has changed.
if (providerDefinition.Enable && (!forceSave || hasDefinitionChanged)) if (providerDefinition.Enable && (!forceSave || hasDefinitionChanged))