[CrazY.]

Hey Depresie , The code that you gave me is to check which grenade was thrown?

[HamletEagle]

Use grenade_throw from csx.

[Depresie]

Yes, that is called when the grenade is thrown..

[JusTGo]

some how FM_SetModel s called twice so i don't advice using it in this case.

[Depresie]

It is called twice, first to set the grenade p_model then the w_model...
The script i posted is safe to be used, it is called only when the grenade is thrown, not when deployed or dropped

[edon1337]

Originally Posted by Depresie You could also detect grenade launching using FM_SetModel, check there for w_grenade model then check the classname of the entity not to be a weaponbox PHP Code: register_forward(FM_SetModel, "fw_SetModel") public fw_SetModel(iEntity, const szModel[]) {     if(!pev_valid(iEntity))         return FMRES_IGNORED     if(strlen(szModel) < 8)         return FMRES_IGNORED     if(szModel[7] != 'w' || szModel[8] != '_')         return FMRES_IGNORED     if(equal(szModel, "models/w_weaponbox.mdl"))         return FMRES_IGNORED     static szClass[32]     entity_get_string(iEntity, EV_SZ_classname, szClass, charsmax(szClass))     if(!equal(szClass, "weaponbox"))         return FMRES_IGNORED                  // Grenade launched ( you can check for model to get what type of grenade it is, ex w_flashbang, w_smokegrenade, w_hegrenade

Poor way. Using grenade_throw from csx as Hamlet stated is fine.

PHP Code:

public grenade_throw( index,greindex,wId ) {

    if(wId != CSW_HEGRENADE) 
    return PLUGIN_CONTINUE;
    
    // code //
} 


[JusTGo]

Originally Posted by Depresie It is called twice, first to set the grenade p_model then the w_model... The script i posted is safe to be used, it is called only when the grenade is thrown, not when deployed or dropped

afaik FM_SetModel isn't called when p model is set also from my testes its callled twice even if i m checking for w_ models.

[HamletEagle]

Originally Posted by JusTGo afaik FM_SetModel isn't called when p model is set also from my testes its callled twice even if i m checking for w_ models.

It's called twice because it firstly sets weaponbox.mdl then it sets the model corresponding to the weapon packed into the box. You did something wrong if it's still called twice.

[JusTGo]

Originally Posted by HamletEagle It's called twice because it firstly sets weaponbox.mdl then it sets the model corresponding to the weapon packed into the box. You did something wrong if it's still called twice.

damn i registred the forward twice... amxmodx compiler should give a warning or somthing for such things xD.

[klippy]

Originally Posted by JusTGo amxmodx compiler should give a warning or somthing for such things xD.

Not going to happen.