[You9]

Hey,

basically i trying to do stock what removes client all weapons expect 1 slot.
Using m_hMyWeapons but it returns WeaponIDs, but now how i get slot from WeaponID?

[Mitchell]

why not just get the weapon in the player's slot and remove it that what?

[You9]

Because it's not removing all granades somehow.

[Neuro Toxin]

Do while
GetPlayerWeaponSlot > -1

[You9]

Marked with comment line giving error logs: Plugin encountered error 30: Script execution timed out
RemoveAllWeapon(i, CS_SLOT_C4);

PHP Code:

stock RemoveAllWeapons(int client, int SkipSlot = -1) 
{ 
    int wep, removedWep = 0;
    for (int i = 0; i < 6; i++){
        if(i != SkipSlot){
            while ((wep = GetPlayerWeaponSlot(client, i)) != -1){ 
                if(DeleteClientWeapon(client, wep)) //ERROR LINE
                    removedWep++;
            }
        }
    }
    return removedWep;
} 


PHP Code:

stock DeleteClientWeapon(int client, int weapon)
{
    if(!IsValidWeapon(weapon) || !IsPlayerAlive(client) || GetEntSendPropOffs(weapon, "m_bStartedArming") != -1)
        return false;
    
    int iOwnerEntity = GetEntPropEnt(weapon, Prop_Send, "m_hOwnerEntity");
    if (iOwnerEntity != client)
        SetEntPropEnt(weapon, Prop_Send, "m_hOwnerEntity", client);
    
    SDKHooks_DropWeapon(client, weapon, NULL_VECTOR, NULL_VECTOR); //CS_DropWeapon(client, weapon, false, true);
    if (iOwnerEntity != -1)
        SetEntPropEnt(weapon, Prop_Send, "m_hOwnerEntity", iOwnerEntity);
        
    return AcceptEntityInput(weapon, "Kill");
} 


[Mitchell]

Because you are looping if DeleteClientWeapon returns false it will go back the the same weapon over and over creating an infinite loop. Some of these checks within DeleteClientWeapon should be in the method before, theres no need to constantly check if the player is alive or if the client owns the weapon, the weapon is in the client's slot, how would he not own it?
I'd also recommend starting the loop backwards so it starts with c4/grenades then moves to the primary.

[You9]

Thanks, working now but why backwards loop is better?

[Maxximou5]

Originally Posted by You9 Thanks, working now but why backwards loop is better?

I suppose because it's less likely you'd be using those slots (weapons), so when they're stripped it would be less likely to cause issues.
Maybe more to it, or different entirely.

[You9]

When i use RemoveAllWeapons(client, CS_SLOT_KNIFE); on the OnClientSpawn then some people still have AK-47's and stuff, weapons are not removed, someone know the issue?

[Mitchell]

Originally Posted by You9 When i use RemoveAllWeapons(client, CS_SLOT_KNIFE); on the OnClientSpawn then some people still have AK-47's and stuff, weapons are not removed, someone know the issue?

Are there any logs, also i'd recommend using RemovePlayerItem(client, weapon); instead of killing it.