[Seta00]

If you're going to hook weapon_fire for pistols, DON'T DO IT!
The event gets called billions of times if the client holds his +attack button.
Hook bullet_impact instead.

Here an example from one of my plugins:

public Action:EventBulletImpact(Handle:event, const String:name[], bool:dontBroadcast) {
    new client = GetClientOfUserId(GetEventInt(event, "userid"))
        ,weapon = GetEntPropEnt(client, Prop_Send, "m_hActiveWeapon");
    
    if (weapon == specialWeapon) {       
        MyFunction(client); // MyFunction needs to be called ONCE for each weapon fire, so bullet_impact is the right way to do it.
    }
    
    return Plugin_Continue;
}

BULLET_IMPACT IS CREDIT TO PLUGIN!

[KyleS]

The same happens for the weapon_reload event >.<

[Seta00]

Use UserCmd or PlayerThink and check for ammo > lastAmmo to detect weapon reload

[blodia]

you could do that for pistol fire aswell though.

in my weaponmod i have some code to test if the pistol actually fired. although i don't use weapon_fire as it doesn't detect knife stab i used client postthink which fires just before weapon_fire and manually check if the clients weapon fired.

PHP Code:

if (GetGameTime() < GetEntPropFloat(WeaponIndex, Prop_Send, "m_flNextPrimaryAttack"))
{
    // didn't fire
    return;
} 


i don't know if that will work in weapon_fire but you could try it.

[Seta00]

Originally Posted by blodia you could do that for pistol fire aswell though

It's a lot more effort than bullet_impact ;)

[Bacardi]

Sticky: Bug Thread: Non-Beta Issues #77


*edit
one more thing about bullet impact, bullets penetrate player or wall corners or through box will give multiple event.. Like fire one time you mayeb get lucky 4 bullet impact event fired.
bad english

[M1NDFREAK]

Fixed by Valve (15.09.2011)

Pistols no longer continuously spam weapon_fire event when the +attack key is pressed. This has the side effect of no longer allowing the glock to continuously auto-fire in burst mode).

[Peace-Maker]

Keep in mind, that bullet_impact is called everytime a bullet hits a surface. So if you shoot through a door or similar it'll fire twice with different x, y and z coordinates. Once at the door and at the wall behind it.

[Mitchell]

weapon==special weapon, shouldnt that be a StrEqual?

[hleV]

Originally Posted by Mitchell weapon==special weapon, shouldnt that be a StrEqual?

No, because that's the ID of the weapon, not the name.