fix: remove temp code locks when plugin is unloaded

Author: RebeccaStevens

src/AutoCode.cs

@@ -41,18 +41,8 @@ protected override void LoadDefaultConfig() {
 
     void OnServerSave()
     {
+      RemoveAllTempCodeLocks();
       data.Save();
-
-      // Remove all temp code locks - we don't want to save them.
-      foreach (TempCodeLockInfo codeLockInfo in tempCodeLocks.Values)
-      {
-        if (!codeLockInfo.CodeLock.IsDestroyed)
-        {
-          codeLockInfo.CodeLock.Kill();
-        }
-      }
-      tempCodeLocks.Clear();
-      UnsubscribeFromUnneedHooks();
     }
 
     void OnServerShutdown()
@@ -62,6 +52,7 @@ void OnServerShutdown()
 
     void Unload()
     {
+      RemoveAllTempCodeLocks();
       data.Save();
     }
 
@@ -487,6 +478,23 @@ private void DestoryTempCodeLock(BasePlayer player)
       UnsubscribeFromUnneedHooks();
     }
 
+    /// <summary>
+    /// Remove all the temporary code locks.
+    /// </summary>
+    private void RemoveAllTempCodeLocks()
+    {
+      // Remove all temp code locks - we don't want to save them.
+      foreach (TempCodeLockInfo codeLockInfo in tempCodeLocks.Values)
+      {
+        if (!codeLockInfo.CodeLock.IsDestroyed)
+        {
+          codeLockInfo.CodeLock.Kill();
+        }
+      }
+      tempCodeLocks.Clear();
+      UnsubscribeFromUnneedHooks();
+    }
+
     /// <summary>
     /// Unsubscribe from things that there is not point currently being subscribed to.
     /// </summary>