From 4074f697d73f5cac6699836550b144fd0c4e2803 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Fri, 30 Jul 2021 00:40:12 -0400
Subject: rename patch classes for consistency

---
 .../Patches/ObjectErrorPatch.cs                    | 71 ----------------------
 1 file changed, 71 deletions(-)
 delete mode 100644 src/SMAPI.Mods.ErrorHandler/Patches/ObjectErrorPatch.cs

(limited to 'src/SMAPI.Mods.ErrorHandler/Patches/ObjectErrorPatch.cs')

diff --git a/src/SMAPI.Mods.ErrorHandler/Patches/ObjectErrorPatch.cs b/src/SMAPI.Mods.ErrorHandler/Patches/ObjectErrorPatch.cs
deleted file mode 100644
index 7a4b3cfa..00000000
--- a/src/SMAPI.Mods.ErrorHandler/Patches/ObjectErrorPatch.cs
+++ /dev/null
@@ -1,71 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
-using HarmonyLib;
-using StardewModdingAPI.Framework.Patching;
-using StardewValley;
-using SObject = StardewValley.Object;
-
-namespace StardewModdingAPI.Mods.ErrorHandler.Patches
-{
-    /// <summary>A Harmony patch for <see cref="SObject.getDescription"/> which intercepts crashes due to the item no longer existing.</summary>
-    /// <remarks>Patch methods must be static for Harmony to work correctly. See the Harmony documentation before renaming patch arguments.</remarks>
-    [SuppressMessage("ReSharper", "InconsistentNaming", Justification = "Argument names are defined by Harmony and methods are named for clarity.")]
-    [SuppressMessage("ReSharper", "IdentifierTypo", Justification = "Argument names are defined by Harmony and methods are named for clarity.")]
-    internal class ObjectErrorPatch : IHarmonyPatch
-    {
-        /*********
-        ** Public methods
-        *********/
-        /// <inheritdoc />
-        public void Apply(Harmony harmony)
-        {
-            // object.getDescription
-            harmony.Patch(
-                original: AccessTools.Method(typeof(SObject), nameof(SObject.getDescription)),
-                prefix: new HarmonyMethod(this.GetType(), nameof(ObjectErrorPatch.Before_Object_GetDescription))
-            );
-
-            // object.getDisplayName
-            harmony.Patch(
-                original: AccessTools.Method(typeof(SObject), "loadDisplayName"),
-                finalizer: new HarmonyMethod(this.GetType(), nameof(ObjectErrorPatch.Finalize_Object_loadDisplayName))
-            );
-        }
-
-
-        /*********
-        ** Private methods
-        *********/
-        /// <summary>The method to call instead of <see cref="StardewValley.Object.getDescription"/>.</summary>
-        /// <param name="__instance">The instance being patched.</param>
-        /// <param name="__result">The patched method's return value.</param>
-        /// <returns>Returns whether to execute the original method.</returns>
-        private static bool Before_Object_GetDescription(SObject __instance, ref string __result)
-        {
-            // invalid bigcraftables crash instead of showing '???' like invalid non-bigcraftables
-            if (!__instance.IsRecipe && __instance.bigCraftable.Value && !Game1.bigCraftablesInformation.ContainsKey(__instance.ParentSheetIndex))
-            {
-                __result = "???";
-                return false;
-            }
-
-            return true;
-        }
-
-        /// <summary>The method to call after <see cref="StardewValley.Object.loadDisplayName"/>.</summary>
-        /// <param name="__result">The patched method's return value.</param>
-        /// <param name="__exception">The exception thrown by the wrapped method, if any.</param>
-        /// <returns>Returns the exception to throw, if any.</returns>
-        private static Exception Finalize_Object_loadDisplayName(ref string __result, Exception __exception)
-        {
-            if (__exception is KeyNotFoundException)
-            {
-                __result = "???";
-                return null;
-            }
-
-            return __exception;
-        }
-    }
-}
-- 
cgit