Don't process disabled meta data providers.

pudding-fox · Jan 28, 2024 · 3b03448 · 3b03448
1 parent 9b64cc3
commit 3b03448
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/FoxTunes.MetaData/MetaDataDecorator.cs b/FoxTunes.MetaData/MetaDataDecorator.cs
@@ -43,6 +43,10 @@ public void Decorate(string fileName, IList<MetaDataItem> metaDataItems, ISet<st
             var providers = this.MetaDataProviderManager.GetProviders();
             foreach (var provider in providers)
             {
+                if (!provider.Enabled)
+                {
+                    continue;
+                }
                 try
                 {
                     var factory = this.MetaDataProviderManager.GetProvider(provider);
@@ -68,6 +72,10 @@ public void Decorate(IFileAbstraction fileAbstraction, IList<MetaDataItem> metaD
             var providers = this.MetaDataProviderManager.GetProviders();
             foreach (var provider in providers)
             {
+                if (!provider.Enabled)
+                {
+                    continue;
+                }
                 try
                 {
                     var factory = this.MetaDataProviderManager.GetProvider(provider);

diff --git a/FoxTunes.MetaData/MetaDataDecoratorFactory.cs b/FoxTunes.MetaData/MetaDataDecoratorFactory.cs
@@ -24,6 +24,10 @@ public IEnumerable<KeyValuePair<string, MetaDataItemType>> Supported
                 var providers = this.MetaDataProviderManager.GetProviders();
                 foreach (var provider in providers)
                 {
+                    if (!provider.Enabled)
+                    {
+                        continue;
+                    }
                     yield return new KeyValuePair<string, MetaDataItemType>(provider.Name, MetaDataItemType.CustomTag);
                 }
             }
@@ -34,7 +38,7 @@ public bool CanCreate
             get
             {
                 var providers = this.MetaDataProviderManager.GetProviders();
-                return providers.Any();
+                return providers.Any(provider => provider.Enabled);
             }
         }