private void OnUsagePermissionChanged(string userIdString)
        {
			if (permission.UserHasPermission(userIdString, UsagePermission))
			{
				var userId = ulong.Parse(userIdString);
				Backpack.Get(userId).Initialize();
			}
        }​

#2 - Added if (_backpacks.ContainsKey(player.userID))

private void OnEntityDeath(BaseCombatEntity victim, HitInfo info)
        {
            if (victim is BasePlayer && !(victim is NPCPlayer) && !(victim is HTNPlayer))
            {
                var player = (BasePlayer) victim;
				
				if (_backpacks.ContainsKey(player.userID))
				{
					var backpack = Backpack.Get(player.userID);

					backpack.ForceClose();

					if (_config.EraseOnDeath)
						backpack.EraseContents();
					else if (_config.DropOnDeath)
						backpack.Drop(player.transform.position);
				}
            }
        }

In the first situation it gates the OnUsagePermissionChanged hook to make sure it only gets a backpack if the player has permissions. This hook might be called for something completely unrelated.

In OnEntityDeath, Backpack.Get should only be called if a backpack already exists. This way it stops on initialising a new backpack everytime a fresh player has their first death.

First of all, thanks for contributing. Since I've got my exams this week, this might go under before I get it done.

In case I don't get back to this until next week, can you ping me again if you get a chance?

Kinesis

Amazing job @LaserHydra, really appreciate it! Good luck with the studies!

Thanks!

LaserHydra

Thanks!

Thanks to you for claiming the plugin , stops the kids "fool capping" all the time.

pookins

Thanks to you for claiming the plugin , stops the kids "fool capping" all the time.

sorry this should be in the caps no caps feedback