From 08d8a487014f25ee44aaff1d7114a0acb88d758e Mon Sep 17 00:00:00 2001
From: XtraCube <72575280+XtraCube@users.noreply.github.com>
Date: Wed, 12 Mar 2025 16:01:34 -0400
Subject: [PATCH] make Player not nullable in basemodifier (because it shouldnt
 ever be)

---
 MiraAPI.Example/Modifiers/Freezer/FreezeModifier.cs | 8 ++++----
 MiraAPI/Modifiers/BaseModifier.cs                   | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/MiraAPI.Example/Modifiers/Freezer/FreezeModifier.cs b/MiraAPI.Example/Modifiers/Freezer/FreezeModifier.cs
index 12a9587..8eee41e 100644
--- a/MiraAPI.Example/Modifiers/Freezer/FreezeModifier.cs
+++ b/MiraAPI.Example/Modifiers/Freezer/FreezeModifier.cs
@@ -15,7 +15,7 @@ public class FreezeModifier : TimedModifier
 
     public override void OnActivate()
     {
-        if (Player?.AmOwner == true)
+        if (Player.AmOwner)
         {
             Player.moveable = false;
         }
@@ -25,15 +25,15 @@ public class FreezeModifier : TimedModifier
     {
         base.FixedUpdate();
 
-        if (Player?.AmOwner == true || PlayerControl.LocalPlayer.Data.Role is FreezerRole)
+        if (Player.AmOwner || PlayerControl.LocalPlayer.Data.Role is FreezerRole)
         {
-            Player?.cosmetics.SetOutline(true, new Nullable<Color>(Palette.LightBlue));
+            Player.cosmetics.SetOutline(true, new Nullable<Color>(Palette.LightBlue));
         }
     }
 
     public override void OnTimerComplete()
     {
-        if (Player?.AmOwner == true)
+        if (Player.AmOwner)
         {
             Player.moveable = true;
         }
diff --git a/MiraAPI/Modifiers/BaseModifier.cs b/MiraAPI/Modifiers/BaseModifier.cs
index 0dd2149..7b3dc3f 100644
--- a/MiraAPI/Modifiers/BaseModifier.cs
+++ b/MiraAPI/Modifiers/BaseModifier.cs
@@ -12,7 +12,7 @@ public abstract class BaseModifier : IOptionable
     /// <summary>
     /// Gets the player that the modifier is attached to.
     /// </summary>
-    public PlayerControl? Player { get; internal set; }
+    public PlayerControl Player { get; internal set; } = null!;
 
     /// <summary>
     /// Gets the modifier component that the modifier is attached to.