From be88010c20553a63dbcbbf58987db9ac1f5dd54c Mon Sep 17 00:00:00 2001
From: Mark McDowall <mark@mcdowall.ca>
Date: Wed, 9 Sep 2020 01:09:45 -0700
Subject: [PATCH] New: Don't forcibly retest indexers/download
 clients/connections on save if previously enabled

(cherry picked from commit b2b1600ebe7f022a3248ea12b69553e2d51a3a7c)
---
 src/Lidarr.Api.V1/ProviderModuleBase.cs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/Lidarr.Api.V1/ProviderModuleBase.cs b/src/Lidarr.Api.V1/ProviderModuleBase.cs
index c56c9e4f7..1e82fd7c6 100644
--- a/src/Lidarr.Api.V1/ProviderModuleBase.cs
+++ b/src/Lidarr.Api.V1/ProviderModuleBase.cs
@@ -84,8 +84,10 @@ namespace Lidarr.Api.V1
         private void UpdateProvider(TProviderResource providerResource)
         {
             var providerDefinition = GetDefinition(providerResource, false);
+            var existingDefinition = _providerFactory.Get(providerDefinition.Id);
 
-            if (providerDefinition.Enable)
+            // Only test existing definitions if it was previously disabled
+            if (providerDefinition.Enable && !existingDefinition.Enable)
             {
                 Test(providerDefinition, false);
             }