fix: do not unregister hooks to avoid sudden server crashes

src/RollTheDice+DiceBigTaserBattery.cs

@@ -27,7 +27,6 @@ private void DiceBigTaserBatteryLoad()
 
         private void DiceBigTaserBatteryUnload()
         {
-            DeregisterEventHandler<EventWeaponFire>(EventDiceBigTaserBatteryOnWeaponFire);
             DiceBigTaserBatteryReset();
         }
 

src/RollTheDice+DiceFastBombAction.cs

@@ -49,8 +49,6 @@ private void DiceFastBombActionLoad()
 
         private void DiceFastBombActionUnload()
         {
-            DeregisterEventHandler<EventBombBegindefuse>(DiceFastBombActionEventBeginDefuse);
-            DeregisterEventHandler<EventBombBeginplant>(DiceFastBombActionEventBeginPlant);
             DiceFastBombActionReset();
         }
 
@@ -64,9 +62,9 @@ private HookResult DiceFastBombActionEventBeginDefuse(EventBombBegindefuse @even
         {
             var player = @event.Userid;
             if (player == null) return HookResult.Continue;
+            if (!_playersCanInstantDefuse.Contains(player)) return HookResult.Continue;
             var playerPawn = player.PlayerPawn.Value;
             if (playerPawn == null) return HookResult.Continue;
-            if (!_playersCanInstantDefuse.Contains(player)) return HookResult.Continue;
             var c4 = Utilities.FindAllEntitiesByDesignerName<CPlantedC4>("planted_c4").First();
             Server.NextFrame(() =>
             {
@@ -80,9 +78,9 @@ private HookResult DiceFastBombActionEventBeginPlant(EventBombBeginplant @event,
         {
             var player = @event.Userid;
             if (player == null) return HookResult.Continue;
+            if (!_playersCanInstantPlant.Contains(player)) return HookResult.Continue;
             var playerPawn = player.PlayerPawn.Value;
             if (playerPawn == null) return HookResult.Continue;
-            if (!_playersCanInstantPlant.Contains(player)) return HookResult.Continue;
             var weaponService = playerPawn.WeaponServices?.ActiveWeapon;
             if (weaponService == null) return HookResult.Continue;
             var activeWeapon = weaponService.Value;

src/RollTheDice+DiceIncreaseSpeed.cs

@@ -32,7 +32,6 @@ private void DiceIncreaseSpeedLoad()
 
         private void DiceIncreaseSpeedUnload()
         {
-            DeregisterEventHandler<EventPlayerHurt>(EventDiceIncreaseSpeedOnPlayerHurt);
             DiceIncreaseSpeedReset();
         }
 
@@ -56,13 +55,9 @@ private void DiceIncreaseSpeedReset()
         private HookResult EventDiceIncreaseSpeedOnPlayerHurt(EventPlayerHurt @event, GameEventInfo info)
         {
             var victim = @event.Userid;
-            // ignore if victim is null
             if (victim == null) return HookResult.Continue;
-            // ignore if player is not in the list
             if (!_playersWithIncreasedSpeed.Contains(victim)) return HookResult.Continue;
-            // check if player is valid
             if (victim == null || victim.PlayerPawn == null || !victim.PlayerPawn.IsValid || victim.PlayerPawn.Value == null || victim.LifeState != (byte)LifeState_t.LIFE_ALIVE) return HookResult.Continue;
-            // get player pawn
             var playerPawn = victim.PlayerPawn.Value!;
             AddTimer(0f, () =>
             {

src/RollTheDice+DicePlayerAsChicken.cs

@@ -50,7 +50,6 @@ private void DicePlayerAsChickenUnload()
         private void DicePlayerAsChickenReset()
         {
             // remove listeners
-            DeregisterEventHandler<EventPlayerDeath>(EventDicePlayerAsChickenOnPlayerDeath);
             RemoveListener<Listeners.OnTick>(EventDicePlayerAsChickenOnTick);
             RemoveListener<Listeners.CheckTransmit>(EventDicePlayerAsChickenCheckTransmit);
             // iterate through all players

src/RollTheDice+DicePlayerDisguiseAsPlant.cs

@@ -56,7 +56,6 @@ private void DicePlayerDisguiseAsPlantLoad()
 
         private void DicePlayerDisguiseAsPlantUnload()
         {
-            DeregisterEventHandler<EventPlayerDeath>(EventDicePlayerDisguiseAsPlantOnPlayerDeath);
             DicePlayerDisguiseAsPlantReset();
         }
 

src/RollTheDice+DicePlayerRespawn.cs

@@ -30,8 +30,6 @@ private void DicePlayerRespawnLoad()
 
         private void DicePlayerRespawnUnload()
         {
-            DeregisterEventHandler<EventPlayerDeath>(EventDicePlayerRespawnOnPlayerDeath);
-            DeregisterEventHandler<EventPlayerTeam>(EventDicePlayerRespawnOnPlayerTeam);
             DicePlayerRespawnReset();
         }
 
@@ -90,15 +88,15 @@ private void EventDicePlayerRespawnOnTick()
         private HookResult EventDicePlayerRespawnOnPlayerDeath(EventPlayerDeath @event, GameEventInfo info)
         {
             var player = @event.Userid;
-            var attacker = @event.Attacker;
             if (player == null) return HookResult.Continue;
+            if (!_playersWithRespawnAbility.ContainsKey(player)) return HookResult.Continue;
             var playerPawn = player.PlayerPawn.Value;
             if (playerPawn == null) return HookResult.Continue;
             var weaponService = playerPawn.WeaponServices;
             if (weaponService == null) return HookResult.Continue;
-            if (!_playersWithRespawnAbility.ContainsKey(player)) return HookResult.Continue;
             // give weapons from attacker because player is dead and has no weapons in weaponsService (they are removed)
             var tmpWeaponList = new List<string>();
+            var attacker = @event.Attacker;
             if (attacker != null
                 && attacker.PlayerPawn != null
                 && attacker.PlayerPawn.Value != null

src/RollTheDice+DicePlayerVampire.cs

@@ -25,7 +25,7 @@ private void DicePlayerVampireLoad()
 
         private void DicePlayerVampireUnload()
         {
-            DeregisterEventHandler<EventPlayerHurt>(EventDicePlayerVampireOnPlayerHurt);
+            DicePlayerVampireReset();
         }
 
         private void DicePlayerVampireReset()