From d0028f92fa3c6e856baa60f347c3e57a762ee75c Mon Sep 17 00:00:00 2001
From: danielpalme <daniel.palme@gmx.de>
Date: Sun, 1 May 2022 19:47:57 +0200
Subject: [PATCH] #514: Improved plugin handling for netstandard2.0

---
 .../nuget/ReportGenerator.Core.nuspec         |  1 -
 src/Readme.txt                                |  4 +++
 .../Plugin/ReflectionPluginLoader.cs          | 36 +++++++++++--------
 .../Properties/Resources.Designer.cs          | 18 +++++-----
 .../Properties/Resources.resx                 |  6 ++--
 5 files changed, 38 insertions(+), 27 deletions(-)

diff --git a/src/Deployment/nuget/ReportGenerator.Core.nuspec b/src/Deployment/nuget/ReportGenerator.Core.nuspec
index 845b0961..e43294e4 100644
--- a/src/Deployment/nuget/ReportGenerator.Core.nuspec
+++ b/src/Deployment/nuget/ReportGenerator.Core.nuspec
@@ -30,7 +30,6 @@ https://github.com/danielpalme/ReportGenerator/wiki/Custom-history-storage</desc
         <dependency id="Microsoft.Extensions.Configuration.Json" version="5.0.0" />
         <dependency id="SixLabors.ImageSharp.Drawing" version="1.0.0-beta14" />
         <dependency id="DotNetConfig" version="1.0.6" />
-        <dependency id="McMaster.NETCore.Plugins" version="1.4.0" />
       </group>
 
       <group targetFramework="netcoreapp">
diff --git a/src/Readme.txt b/src/Readme.txt
index d1aae0bc..275bbcfa 100644
--- a/src/Readme.txt
+++ b/src/Readme.txt
@@ -67,6 +67,10 @@ For further details take a look at LICENSE.txt.
 
 CHANGELOG
 
+5.1.6.0
+
+    * Fix: #514: Improved plugin handling for netstandard2.0
+
 5.1.5.0
 
     * Fix: #508: Improved plugin handling for netstandard2.0
diff --git a/src/ReportGenerator.Core/Plugin/ReflectionPluginLoader.cs b/src/ReportGenerator.Core/Plugin/ReflectionPluginLoader.cs
index dfd7eac0..06eab39e 100644
--- a/src/ReportGenerator.Core/Plugin/ReflectionPluginLoader.cs
+++ b/src/ReportGenerator.Core/Plugin/ReflectionPluginLoader.cs
@@ -19,6 +19,11 @@ internal class ReflectionPluginLoader : IPluginLoader
         /// </summary>
         private static readonly ILogger Logger = LoggerFactory.GetLogger(typeof(ReflectionPluginLoader));
 
+        /// <summary>
+        /// Indicates whether an exception occured to prevent logging same exception several times.
+        /// </summary>
+        private static bool pluginsNotSupportedMessageShown;
+
         /// <summary>
         /// The plugins.
         /// </summary>
@@ -63,8 +68,19 @@ public IReadOnlyCollection<T> LoadInstancesOfType<T>()
                 }
             }
 
-            if (this.plugins.Count == 0 || this.assemblyLoader == null)
+            if (this.plugins.Count == 0)
+            {
+                return result;
+            }
+
+            if (this.assemblyLoader == null)
             {
+                if (!pluginsNotSupportedMessageShown)
+                {
+                    Logger.Error(Resources.PluginsNotSupported);
+                    pluginsNotSupportedMessageShown = true;
+                }
+
                 return result;
             }
 
@@ -119,19 +135,11 @@ private IAssemblyLoader CreateAssemblyLoader()
                     return null;
                 }
 
-                try
-                {
-                    var dotnetCorePluginLoaderAssembly = Assembly.LoadFrom(path);
-                    var assemblyLoaderType = dotnetCorePluginLoaderAssembly.GetExportedTypes()
-                            .Where(t => t.FullName == "ReportGenerator.DotnetCorePluginLoader.DotNetCoreAssemblyLoader" && t.IsClass && !t.IsAbstract)
-                            .Single();
-                    return new ReflectionWrapperAssemblyLoader(Activator.CreateInstance(assemblyLoaderType));
-                }
-                catch (Exception ex)
-                {
-                    Logger.Error(string.Format(Resources.FailedToLoadPluginLoader, ex.Message));
-                    return null;
-                }
+                var dotnetCorePluginLoaderAssembly = Assembly.LoadFrom(path);
+                var assemblyLoaderType = dotnetCorePluginLoaderAssembly.GetExportedTypes()
+                        .Where(t => t.FullName == "ReportGenerator.DotnetCorePluginLoader.DotNetCoreAssemblyLoader" && t.IsClass && !t.IsAbstract)
+                        .Single();
+                return new ReflectionWrapperAssemblyLoader(Activator.CreateInstance(assemblyLoaderType));
             }
 
             return new DefaultAssemblyLoader();
diff --git a/src/ReportGenerator.Core/Properties/Resources.Designer.cs b/src/ReportGenerator.Core/Properties/Resources.Designer.cs
index 068d625b..1fde1164 100644
--- a/src/ReportGenerator.Core/Properties/Resources.Designer.cs
+++ b/src/ReportGenerator.Core/Properties/Resources.Designer.cs
@@ -330,15 +330,6 @@ internal class Resources {
             }
         }
         
-        /// <summary>
-        ///   Looks up a localized string similar to Failed to load plugin loader (Error: &apos;{0}&apos;)..
-        /// </summary>
-        internal static string FailedToLoadPluginLoader {
-            get {
-                return ResourceManager.GetString("FailedToLoadPluginLoader", resourceCulture);
-            }
-        }
-        
         /// <summary>
         ///   Looks up a localized string similar to Failed to load plugins from &apos;{0}&apos;. Make sure plugin has a strong name..
         /// </summary>
@@ -547,6 +538,15 @@ internal class Resources {
             }
         }
         
+        /// <summary>
+        ///   Looks up a localized string similar to Plugins are not supported in a .NET Standard environment. .
+        /// </summary>
+        internal static string PluginsNotSupported {
+            get {
+                return ResourceManager.GetString("PluginsNotSupported", resourceCulture);
+            }
+        }
+        
         /// <summary>
         ///   Looks up a localized string similar to Preprocessing report.
         /// </summary>
diff --git a/src/ReportGenerator.Core/Properties/Resources.resx b/src/ReportGenerator.Core/Properties/Resources.resx
index 08a79f88..3d7f5582 100644
--- a/src/ReportGenerator.Core/Properties/Resources.resx
+++ b/src/ReportGenerator.Core/Properties/Resources.resx
@@ -207,9 +207,6 @@
   <data name="FailedToInstantiatePlugin" xml:space="preserve">
     <value>Failed to instantiate plugin class '{0}'.</value>
   </data>
-  <data name="FailedToLoadPluginLoader" xml:space="preserve">
-    <value>Failed to load plugin loader (Error: '{0}').</value>
-  </data>
   <data name="FailedToLoadPlugins" xml:space="preserve">
     <value>Failed to load plugins from '{0}'. Make sure plugin has a strong name.</value>
   </data>
@@ -280,6 +277,9 @@
   <data name="ParsingCompleted" xml:space="preserve">
     <value>Parsing of {0} files completed</value>
   </data>
+  <data name="PluginsNotSupported" xml:space="preserve">
+    <value>Plugins are not supported in a .NET Standard environment. </value>
+  </data>
   <data name="PreprocessingReport" xml:space="preserve">
     <value>Preprocessing report</value>
   </data>