change knife give handling

README.md

@@ -22,6 +22,7 @@ Unfinished, unoptimized and not fully functional ugly demo weapon paints plugin
 - in `addons/counterstrikesharp/configs/core.json` set **FollowCS2ServerGuidelines** to **false**
 
 ### Web install:
+- requires PHP (tested on php ver `8.2.3` and nginx webserver)
 - copy website to web server
 - import `database.sql` to mysql
 - get steam api key [https://steamcommunity.com/dev/apikey](https://steamcommunity.com/dev/apikey)

WeaponPaints.cs

@@ -131,11 +131,27 @@ public class WeaponPaints : BasePlugin, IPluginConfig<WeaponPaintsConfig>
     private HookResult OnPlayerSpawn(EventPlayerSpawn @event, GameEventInfo info)
     {
         var player = @event.Userid;
-        if (!player.IsValid || !player.PlayerPawn.IsValid || player.IsBot)
+        if (!player.IsValid || !player.PlayerPawn.IsValid)
         {
             return HookResult.Continue;
         }
-        if (!PlayerHasKnife(player)) player.GiveNamedItem(g_playersKife[(int)player.EntityIndex!.Value.Value]);
+        if (player.IsBot) 
+        { 
+            player.GiveNamedItem("weapon_knife");
+            return HookResult.Continue;
+        }
+        
+        if (!PlayerHasKnife(player)) 
+        {
+            if (g_playersKife.ContainsKey((int)player.EntityIndex!.Value.Value))
+            {
+                player.GiveNamedItem(g_playersKife[(int)player.EntityIndex!.Value.Value]);
+            }
+            else
+            {
+                player.GiveNamedItem("weapon_knife");
+            }
+        } 
 
         return HookResult.Continue;
     }
@@ -187,6 +203,7 @@ public class WeaponPaints : BasePlugin, IPluginConfig<WeaponPaintsConfig>
     public void RemoveKnifeFromPlayer(CCSPlayerController player)
     {
         if (!player.PawnIsAlive) return;
+        if (!g_playersKife.ContainsKey((int)player.EntityIndex!.Value.Value)) return;
         var weapons = player.PlayerPawn.Value.WeaponServices!.MyWeapons;
         foreach (var weapon in weapons)
         {
@@ -325,10 +342,6 @@ public class WeaponPaints : BasePlugin, IPluginConfig<WeaponPaintsConfig>
             {
                 g_playersKife[playerIndex] = knife;
             }
-            else
-            {
-                g_playersKife[playerIndex] = "weapon_knife";
-            }
             //Log($"{player.PlayerName} has this knife -> {g_playersKife[playerIndex]}");
         }
         catch (Exception e)
@@ -352,7 +365,6 @@ public class WeaponPaints : BasePlugin, IPluginConfig<WeaponPaintsConfig>
             return;
         }
     }
-
     private static void Log(string message)
     {
         Console.BackgroundColor = ConsoleColor.DarkGray;