From 76bb700e1e3bab582ff79c1fec03ba657a179a8d Mon Sep 17 00:00:00 2001
From: Arne Kiesewetter <banane9@live.de>
Date: Fri, 22 Nov 2024 22:17:34 +0100
Subject: [PATCH] Fix failing shutdown with already shutdown monkeys

---
 MonkeyLoader/Meta/IShutdown.cs      | 5 ++++-
 MonkeyLoader/Patching/MonkeyBase.cs | 5 +++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/MonkeyLoader/Meta/IShutdown.cs b/MonkeyLoader/Meta/IShutdown.cs
index 849f13d..bfddde8 100644
--- a/MonkeyLoader/Meta/IShutdown.cs
+++ b/MonkeyLoader/Meta/IShutdown.cs
@@ -23,6 +23,9 @@ public static class ShutdownEnumerableExtensions
         /// Calls the <see cref="IShutdown.Shutdown"/> method on all elements of the collection,
         /// aggregating their success state as an 'all'.
         /// </summary>
+        /// <remarks>
+        /// Already <see cref="IShutdown.ShutdownRan">shut down</see> elements are ignored.
+        /// </remarks>
         /// <param name="shutdowns">The <see cref="IShutdown"/> instances to process.</param>
         /// <param name="applicationExiting">Whether the shutdown was caused by the application exiting.</param>
         /// <returns><c>true</c> if all instances successfully shut down, <c>false</c> otherwise.</returns>
@@ -30,7 +33,7 @@ public static bool ShutdownAll(this IEnumerable<IShutdown> shutdowns, bool appli
         {
             var success = true;
 
-            foreach (var shutdown in shutdowns)
+            foreach (var shutdown in shutdowns.Where(shutdown => !shutdown.ShutdownRan))
                 success &= shutdown.Shutdown(applicationExiting);
 
             return success;
diff --git a/MonkeyLoader/Patching/MonkeyBase.cs b/MonkeyLoader/Patching/MonkeyBase.cs
index 8078666..5783049 100644
--- a/MonkeyLoader/Patching/MonkeyBase.cs
+++ b/MonkeyLoader/Patching/MonkeyBase.cs
@@ -209,9 +209,10 @@ public bool Shutdown(bool applicationExiting)
 
         /// <inheritdoc/>
         /// <remarks>
-        /// <i>Format:</i> <c>{<see cref="Mod">Mod</see>.<see cref="Mod.Title">Title</see>}/{<see cref="Name">Name</see>}</c>
+        /// <i>Format:</i> <c>{<see cref="Mod">Mod</see>.<see cref="Mod.Title">Title</see>}/{<see cref="Name">Name</see>}
+        /// ({(<see cref="Ran">Ran</see> ? (<see cref="ShutdownRan">ShutdownRan</see> ? "Ended" : (<see cref="Enabled">Enabled</see> ? "Active" : "Inactive")) : "Pending")})</c>
         /// </remarks>
-        public override string ToString() => $"{Mod.Title}/{Name}";
+        public override string ToString() => $"{Mod.Title}/{Name} ({(Ran ? (ShutdownRan ? "Ended" : (Enabled ? "Active" : "Inactive")) : "Pending")})";
 
         internal static TMonkey GetInstance<TMonkey>(Type type, Mod mod) where TMonkey : IMonkey
         {