From e6250bfe0f2afcf30f3d0bc5b98682f8d9bea86a 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

Fixes #448

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

diff --git a/src/Readarr.Api.V1/ProviderControllerBase.cs b/src/Readarr.Api.V1/ProviderControllerBase.cs
index 642375409..786b2ced8 100644
--- a/src/Readarr.Api.V1/ProviderControllerBase.cs
+++ b/src/Readarr.Api.V1/ProviderControllerBase.cs
@@ -76,8 +76,10 @@ public ActionResult<TProviderResource> CreateProvider(TProviderResource provider
         public ActionResult<TProviderResource> 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);
             }