potentially fix mira vent bug

All-Of-Us-Mods · Sep 23, 2024 · aa2350b · aa2350b
1 parent 52daf69
commit aa2350b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/MiraAPI/Modifiers/BaseModifier.cs b/MiraAPI/Modifiers/BaseModifier.cs
@@ -75,6 +75,6 @@ public virtual void OnDeath(DeathReason reason)
     /// <summary>
     /// Determines whether the player can vent.
     /// </summary>
-    /// <returns>True if the player can vent, false otherwise.</returns>
-    public virtual bool CanVent() => Player?.Data.Role.CanVent == true;
+    /// <returns>True if the player can vent, false otherwise. Null for no effect.</returns>
+    public virtual bool? CanVent() => null;
 }
diff --git a/MiraAPI/Patches/VentPatches.cs b/MiraAPI/Patches/VentPatches.cs
@@ -1,4 +1,5 @@
-using HarmonyLib;
+using System.Linq;
+using HarmonyLib;
 using MiraAPI.Roles;
 using MiraAPI.Utilities;
 using UnityEngine;
@@ -31,10 +32,10 @@ public static void VentCanUsePostfix(Vent __instance, ref float __result, [Harmo
 
         switch (canVent)
         {
-            case true when modifiers.Exists(x => !x.CanVent()):
+            case true when modifiers.Exists(x => x.CanVent().HasValue && x.CanVent()==false):
                 couldUse = canUse = false;
                 return;
-            case false when modifiers.Exists(x => x.CanVent()):
+            case false when modifiers.Exists(x => x.CanVent().HasValue && x.CanVent()==true):
                 couldUse = true;
                 break;
         }