fix: better controller checking

Helper.cs

@@ -38,6 +38,10 @@ public static float DistanceTo(Vector a, Vector b)
         {
             return (float)Math.Sqrt(Math.Pow(a.X - b.X, 2) + Math.Pow(a.Y - b.Y, 2) + Math.Pow(a.Z - b.Z, 2));
         }
+        public static bool IsPlayerValid(CCSPlayerController? p)
+        {
+            return p != null && p.IsValid && (p.SteamID.ToString().Length == 17 || (p.SteamID == 0 && p.IsBot)) && p.Connected == PlayerConnectedState.PlayerConnected && !p.IsHLTV;
+        }
     }
 
 }

Models/Arena.cs

@@ -232,13 +232,14 @@ public ArenaResult GetArenaResult()
         return new ArenaResult(ArenaResultType.Empty, null, null);
     }
 
-    private bool isPValid(ArenaPlayer? player)
-    {
-        return player != null && player.PlayerController.IsValid && player.PlayerController.Connected == PlayerConnectedState.PlayerConnected && player.PlayerController.Pawn.IsValid;
-    }
 
     private bool anyPValid()
     {
         return isPValid(_player1) || isPValid(_player2);
     }
+
+    private bool isPValid(ArenaPlayer? p)
+    {
+        return Helper.IsPlayerValid(p?.PlayerController);
+    }
 }

Models/ArenaPlayer.cs

@@ -1,4 +1,5 @@
 using CounterStrikeSharp.API.Core;
+using CounterStrikeSharp.API.Modules.Entities;
 using CounterStrikeSharp.API.Modules.Entities.Constants;
 using CounterStrikeSharp.API.Modules.Utils;
 
@@ -21,7 +22,7 @@ public void ResetPlayerWeapons(RoundType roundType)
     {
         string knifeName = EnumUtils.GetEnumMemberAttributeValue(CsItem.Knife) ?? "";
 
-        if (PlayerController?.Pawn?.Value?.IsValid == null || PlayerController.Pawn!.Value!.IsValid == false) return;
+        if (!Helper.IsPlayerValid(PlayerController)) return;
 
         // Remove default (spawn) weapons
         foreach (CHandle<CBasePlayerWeapon> weapon in PlayerController!.Pawn!.Value!.WeaponServices!.MyWeapons)