[xXNewGuyXx]

Solutions for this thread:
1.)You shouldn't use a single global integer to represent all clients, instead pass the client as data: Pass in userid : GetClientUserId(client)/GetClientOfUserId(userid)

2.) It is important to understand that Weapon name strings for events differ from weapon name strings for entities (applicable on player_death Hook).

[Kolapsicle]

You shouldn't use a single global integer to represent all clients, instead pass the client as data.

PHP Code:

CreateTimer(1.0, Timer_MyFunction, client);

public Action Timer_MyFunction(Handle timer, int client)
{
    // ...
} 


Edit: Chaosxk is correct, you should pass a userid in the code above as client indexes are reused frequently and arbitrarily.

[Chaosxk]

^ Pass in userid : GetClientUserId(client)/GetClientOfUserId(userid)

[xXNewGuyXx]

Hello,

Thank you so much for your reply. I have changed the SpawnWait timer to:

PHP Code:

public Action SpawnWait(Handle timer, int userid)
{
    new client = GetClientOfUserId(userid);
    GiveWeapons(client);
    CPrintToChatAll("{grey}SpawnWait Handle timer: FIRED");
} 


But now it doesn't seem to be working even solo and I'm seeing errors in server console:

PHP Code:

[SM] Native "IsClientInGame" reported: Client index 0 is invalid
[SM] Displaying call stack trace for plugin "Plugin.smx":
[SM]    [0]    Line 230, Plugin.sp::GiveWeapons()
[SM]    [1]    Line 189, Plugin.sp::SpawnWait() 


Am I missing something?

[Kolapsicle]

Are you creating your timer like this?

PHP Code:

CreateTimer(0.1, SpawnWait, GetClientUserId(client)); 


Make sure to also exclude invalid client ranges such as < 1 and > MaxClients

PHP Code:

    if (0 < client <= MaxClients && IsClientInGame(client))
    {
        // ...
    } 


[xXNewGuyXx]

When I create the timer like that, client becomes undefined because the rest of the function is using client_id from

PHP Code:

public Event_Spawn(Handle:event, const String:name[], bool:dontBroadcast)
{
    new client_id = GetClientOfUserId(GetEventInt(event, "userid"));
} 


[Addicted.]

PHP Code:

CreateTimer(0.1, SpawnWait, GetEventInt(event, "userid"));

public Action SpawnWait(Handle timer, int userid) {

    int client = GetClientOfUserId(userid);

} 


[xXNewGuyXx]

Thanks oaaron, this is much more stable.

However, after about 10 map changes, an rtv caused one player to spawn with weapons the stripper was supposed to get rid of, even though the players output was:

PHP Code:

Event_Spawn: FIRED
(StripAllWeapons): FIRED
SpawnWait Handle timer: FIRED
(GiveWeapons): FIRED 


Looking at the server console, I am now seeing the following errors:
[SM] Native "AcceptEntityInput" reported: Entity 85 (85) is not a CBaseEntity
[SM] [0] Line 195, Plugin.sp::KillStripper()

[Kolapsicle]

Is there any reason why you can't strip weapons with a loop in the player_spawn event callback?

PHP Code:

void StripClientWeapons(int client)
{
    for (int i = 0; i < 5; i++) {
        int weapon = GetPlayerWeaponSlot(client, i);
        if (weapon != -1) {
            AcceptEntityInput(weapon, "Kill");
        }
    }
} 


[xXNewGuyXx]

Hey there Kolapsicle,

Your suggestion compiles fine, but it doesn't seem to be working in game. There are no errors in the console that I can see either.