[soumyadip77]

PHP Code:

public plugin_init()
{
   register_plugin(PLUGIN, VERSION, AUTHOR);
   RegisterHookChain(RG_CBasePlayer_Killed, "CBasePlayer_Killed", .post = 1)
}

public CBasePlayer_Killed(pevVictim, pevAttacker)
{
    if(is_user_connected(pevAttacker) || is_user_connected(pevVictim) || pevAttacker != pevVictim || get_member(pevAttacker, m_iTeam) != get_member(pevVictim, m_iTeam))
    {
        if(rg_get_weaponbox_id(pevAttacker) == WEAPON_KNIFE)
        {
            new nAttacker[ MAX_NAME_LENGTH ], nVictim[ MAX_NAME_LENGTH ];
            get_user_name(pevAttacker, nAttacker, charsmax(nAttacker));
            get_user_name(pevVictim, nVictim, charsmax(nVictim));

            client_print_color(0, 0, "%s ^3%s ^1Knifed ^3%s ^1LOL ^4Sharam Kar Le ^3:p", PREFIX, nAttacker, nVictim);
        }
    }
} 


[Bugsy]

Your ||'s should be &&'s since if your attacker is connected, no further conditions are being checked. You want all to be true to advance to the next code block.

[soumyadip77]

Ohk, Anything wrong in get weapon id as server crash while I am killing some one.

Originally Posted by Bugsy Your ||'s should be &&'s since if your attacker is connected, no further conditions are being checked. You want all to be true to advance to the next code block.

[Napoleon_be]

The following might fix ur problem. Check if the attacker is actually a valid player.

PHP Code:

if(iAttacker != 0)
{
    // Valid id.
} 


or this would be even better:

PHP Code:

if(1 <= MAX_PLAYERS <= 32)
{
    // Valid id.
} 


[Natsheh]

Originally Posted by soumyadip77 Ohk, Anything wrong in get weapon id as server crash while I am killing some one.

Yeah that's because rg_get_weaponbox_id(pevAttacker) is expecting a valid weapon entity index and not a player index.

[CrazY.]

Replace rg_get_weaponbox_id(pevAttacker) with get_user_weapon(pevAttacker).

You should probably hook DeathMsg instead, because it may happen that the victim dies to a grenade while the attacker is holding a knife.

Originally Posted by Bugsy I've never scripted or know anything about WC3. Do some testing for yourself to see what happens in the DeathMsg event. DeathMsg PHP Code: register_event( "DeathMsg" , "DeathMsgEvent" , "a" ); public DeathMsgEvent() {      new iKiller = read_data( 1 );      new iVictim = read_data( 2 );      new iIsHeadshot = read_data( 3 );      new szWeapon[ 20 ]; read_data( 4 , szWeapon , charsmax( szWeapon ) );            client_print( 0 , print_chat , "%d %d %d %s" , iKiller , iVictim , iIsHeadshot , szWeapon ); }

[Natsheh]

You can use get_user_attacker(victim, CSW_WPNID); to catch the weapon.

[soumyadip77]

ohk. I know this method as I am rearning and exploring Reapi more so trying to with Reapi do take it wrong way please.