Always validate settings when testing thingies

Fixed: Validation skipped when saving connections
8 years ago · 6b1a4c4198
parent 1072c5247c
commit 6b1a4c4198
1 changed files with 3 additions and 1 deletions
--- a/src/NzbDrone.Api/ProviderModuleBase.cs
+++ b/src/NzbDrone.Api/ProviderModuleBase.cs
@ -160,8 +160,10 @@ namespace NzbDrone.Api

        private Response Test(TProviderResource providerResource)
        {
-            var providerDefinition = GetDefinition(providerResource, true);
+            // Don't validate when getting the definition so we can validate afterwards (avoids validation being skipped because the provider is disabled)
+            var providerDefinition = GetDefinition(providerResource, true, false);

+            Validate(providerDefinition, true);
            Test(providerDefinition, true);

            return "{}";