[Neuro Toxin]

Thread completely derailed.

1. Just loop clients OnPluginStart. Clients will only be in game if the plugin late loaded. OnMapStart calls every map change and the connect/disconnect events are called around it, so OnMapStart is not the right choice.

2. Hook clients in OnClientPutInServer. Before OnClientPutInServer is called IsClientInGame returns false. IsClientInGame returns true during and after OnClientPutInServer. Hooking OnClientPostAdminCheck has a delay from OnClientPutInServer, therefore you might have unexpected results between OnClientPutInServer and OnClientPostAdminCheck when not hooking at OnClientPutInServer.

So during OnPluginStart if IsClientInGame they are already hooked. If !IsClientInGame call OnClientPutInServer yourself so you call the same code.

Example.

public void OnPluginStart()
{
	for (int client = 1; client <= MaxClients; client++)
	{
		if (!IsClientInGame(client))
			continue;

		OnClientPutInServer(client);
	}
}

public void OnClientPutInServer(int client)
{
	SDKHook(client, SDKHook_OnTakeDamageAlive, OnTakeDamage);
}

public void OnClientDisconnect(int client)
{
	SDKUnhook(client, SDKHook_OnTakeDamageAlive, OnTakeDamage);
}

[Psyk0tik]

Yeah let's stop the derailment here. Great input though!

[Ilusion9]

Originally Posted by Porcellian Sorry, I don't know how to do all this. I am new to this. Can someone help me a bit with the piece of code?

This is the Scripting category, not Plugin Requests.
make a thread here https://forums.alliedmods.net/forumdisplay.php?f=60