[berni]

if (GetEntPropEnt(FindEntityByClassname(attacker , "weapon_usp"), Prop_Send, "m_bSilencerOn") == 1)

That's never gonna work without checking the owner of the weapon at least (and this way it's not really clean).

Originally Posted by Annihilator [PHP]#pragma semicolon 1 Works when testing on a small server, but doesn't work on a populated server. No error logs. Any ideas? :/

When I hate something more than someone coding dirty, it is when someone ignores me, do what I mentioned before !

[Annihilator]

PHP Code:

(StrEqual(WeaponName, "weapon_usp") && (GetEntPropEnt(GetEntPropEnt(attacker, Prop_Data, "m_hActiveWeapon"), Prop_Send, "m_bSilencerOn") == 1)) 


Or did I get it wrong again?

[berni]

Add some proper validation checking, remove unnecessary code and do some code cleanup (readable code is very important) then you are fine:

PHP Code:

public Action:TakeDamageHook(victim, &attacker, &inflictor, &Float:damage, &damagetype, &weapon, Float:damageForce[3], Float:damagePosition[3])
{
    // We don't have to check if the victim is valid, because otherwise this hook wouldn't have been called.
    if (attacker < 0 || attacker > MaxClients || !IsClientInGame(attacker)) {
        return Plugin_Continue;
    }

    if (GetClientTeam(victim) == CS_TEAM_T && GetClientTeam(attacker) == CS_TEAM_CT) {
        decl String:weaponClass[32];
        GetEntityClassname(weapon, weaponClass, sizeof(weaponClass));

        if (StrEqual(weaponClass, "weapon_usp")) {
            // We use GetEntProp to get a cell and cast it to a boolean, because there is no boolean version
            new bool:m_bSilencerOn = bool:GetEntProp(weapon, Prop_Send, "m_bSilencerOn");

            if (m_bSilencerOn) {
                damage = 0.0;
                PrintToChatAll("\x01Oh noes! \x03%N \x01shot \x04%N \x01!", attacker, victim);

                return Plugin_Changed;
            }
        }
    }

    return Plugin_Continue;
}