From 10531e537fda7c4901304b295f4ef60ac1f83eea Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Sun, 10 May 2020 11:50:35 -0400
Subject: rewrite AccessTools methods which changed in Harmony 2.0 (#711)

---
 .../ModLoading/Rewriters/MethodParentRewriter.cs   |  5 ++--
 .../Framework/RewriteFacades/AccessToolsMethods.cs | 32 ++++++++++++++++++++++
 .../RewriteFacades/HarmonyInstanceMethods.cs       | 11 ++------
 3 files changed, 38 insertions(+), 10 deletions(-)
 create mode 100644 src/SMAPI/Framework/RewriteFacades/AccessToolsMethods.cs

(limited to 'src/SMAPI/Framework')

diff --git a/src/SMAPI/Framework/ModLoading/Rewriters/MethodParentRewriter.cs b/src/SMAPI/Framework/ModLoading/Rewriters/MethodParentRewriter.cs
index c6388295..d0fe8b13 100644
--- a/src/SMAPI/Framework/ModLoading/Rewriters/MethodParentRewriter.cs
+++ b/src/SMAPI/Framework/ModLoading/Rewriters/MethodParentRewriter.cs
@@ -42,8 +42,9 @@ namespace StardewModdingAPI.Framework.ModLoading.Rewriters
         /// <param name="fromType">The type whose methods to remap.</param>
         /// <param name="toType">The type with methods to map to.</param>
         /// <param name="onlyIfPlatformChanged">Whether to only rewrite references if loading the assembly on a different platform than it was compiled on.</param>
-        public MethodParentRewriter(Type fromType, Type toType, bool onlyIfPlatformChanged = false)
-            : this(fromType.FullName, toType, onlyIfPlatformChanged) { }
+        /// <param name="nounPhrase">A brief noun phrase indicating what the instruction finder matches (or <c>null</c> to generate one).</param>
+        public MethodParentRewriter(Type fromType, Type toType, bool onlyIfPlatformChanged = false, string nounPhrase = null)
+            : this(fromType.FullName, toType, onlyIfPlatformChanged, nounPhrase) { }
 
 
         /// <summary>Perform the predefined logic for an instruction if applicable.</summary>
diff --git a/src/SMAPI/Framework/RewriteFacades/AccessToolsMethods.cs b/src/SMAPI/Framework/RewriteFacades/AccessToolsMethods.cs
new file mode 100644
index 00000000..cb40bbcc
--- /dev/null
+++ b/src/SMAPI/Framework/RewriteFacades/AccessToolsMethods.cs
@@ -0,0 +1,32 @@
+using System;
+using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
+using System.Reflection;
+using HarmonyLib;
+
+namespace StardewModdingAPI.Framework.RewriteFacades
+{
+    /// <summary>Maps Harmony 1.x <see cref="AccessTools"/> methods to Harmony 2.x to avoid breaking older mods.</summary>
+    /// <remarks>This is public to support SMAPI rewriting and should not be referenced directly by mods.</remarks>
+    [SuppressMessage("ReSharper", "CS1591", Justification = "Documentation not needed for facade classes.")]
+    public class AccessToolsMethods
+    {
+        /*********
+        ** Public methods
+        *********/
+        public static ConstructorInfo DeclaredConstructor(Type type, Type[] parameters = null)
+        {
+            return AccessTools.DeclaredConstructor(type, parameters, searchForStatic: true);
+        }
+
+        public static ConstructorInfo Constructor(Type type, Type[] parameters = null)
+        {
+            return AccessTools.Constructor(type, parameters, searchForStatic: true);
+        }
+
+        public static List<ConstructorInfo> GetDeclaredConstructors(Type type)
+        {
+            return AccessTools.GetDeclaredConstructors(type, searchForStatic: true);
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/RewriteFacades/HarmonyInstanceMethods.cs b/src/SMAPI/Framework/RewriteFacades/HarmonyInstanceMethods.cs
index bca76981..aad62c9b 100644
--- a/src/SMAPI/Framework/RewriteFacades/HarmonyInstanceMethods.cs
+++ b/src/SMAPI/Framework/RewriteFacades/HarmonyInstanceMethods.cs
@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Linq;
 using System.Reflection;
 using System.Reflection.Emit;
@@ -7,8 +8,9 @@ using HarmonyLib;
 
 namespace StardewModdingAPI.Framework.RewriteFacades
 {
-    /// <summary>Maps Harmony 1.x methods to Harmony 2.x to avoid breaking older mods.</summary>
+    /// <summary>Maps Harmony 1.x <code>HarmonyInstance</code> methods to Harmony 2.x's <see cref="Harmony"/> to avoid breaking older mods.</summary>
     /// <remarks>This is public to support SMAPI rewriting and should not be referenced directly by mods.</remarks>
+    [SuppressMessage("ReSharper", "CS1591", Justification = "Documentation not needed for facade classes.")]
     public class HarmonyInstanceMethods : Harmony
     {
         /*********
@@ -19,18 +21,11 @@ namespace StardewModdingAPI.Framework.RewriteFacades
         public HarmonyInstanceMethods(string id)
             : base(id) { }
 
-        /// <summary>Creates a new Harmony instance.</summary>
-        /// <param name="id">A unique identifier for the instance.</param>
         public static Harmony Create(string id)
         {
             return new Harmony(id);
         }
 
-        /// <summary>Apply one or more patches to a method.</summary>
-        /// <param name="original">The original method.</param>
-        /// <param name="prefix">The prefix to apply.</param>
-        /// <param name="postfix">The postfix to apply.</param>
-        /// <param name="transpiler">The transpiler to apply.</param>
         public DynamicMethod Patch(MethodBase original, HarmonyMethod prefix = null, HarmonyMethod postfix = null, HarmonyMethod transpiler = null)
         {
             try
-- 
cgit