[xPaw]

Kevlar Pickup Fix
By xPaw

.: Description :.

For example, some plugin gave you kevlar + helmet, and you walk on kevlar on ground, you will pickup it, but no effect, so you can walk on all, and they will disappear, unfair for other players ye ? Or for example you have 50 AP, you can walk on armor and it will give you 100

[crazyeffect]

I have seen this already...

EDIT: Yeah, you released it already in Suggestions


Way to go BugFixer

[ConnorMcLeod]

There is no need to use entity_get_string( iEntity, EV_SZ_model,..) since you need to use cs_get_armoury_type after.

I don't think kevlars disappear in such situations, only players are not able to pick them up.

[xPaw]

Originally Posted by ConnorMcLeod There is no need to use entity_get_string( iEntity, EV_SZ_model,..) since you need to use cs_get_armoury_type after. I don't think kevlars disappear in such situations, only players are not able to pick them up.

Yes it did, i have a plugin wich gives item_assaultsuit, and on ground was kevlar, i walked on it, and pickuped all of them oO

[ConnorMcLeod]

That plugin (the one that gives the item) doesn't seems to update player armortype pdata.

[xPaw]

give_item( id, "item_assaultsuit" );

[KadiR]

item_assaultsuit is a buggy entity in cs.

[anakin_cstrike]

Fuck, i hate you xPawn, i had this ideea too just a few days ago

[ConnorMcLeod]

You messed up the plugin. cs_get_armoury_type doesn't return CSW_ values but CSA_.
How i would code it :

PHP Code:

#include <amxmodx>
#include <cstrike>
#include <engine>

public plugin_init()
{
    register_plugin("Kevlar pickup fix", "1.0", "xPaw")

    register_touch("armoury_entity", "player", "FwdWeaponBoxTouch")
}

public FwdWeaponBoxTouch(iEntity, id)
{
    if( is_user_alive(id) )
    {    
        static iType
        iType = cs_get_armoury_type(iEntity) - 15

        static const bitArmorType = _:CS_ARMOR_KEVLAR|_:CS_ARMOR_VESTHELM
    
        if( iType & bitArmorType )
        {
            static CsArmorType:iArmorType, iArmor
            iArmor = cs_get_user_armor(id, iArmorType)

            if( _:iArmorType < iType || (iArmor < 100 && iType == _:iArmorType) )
            {
                cs_set_user_armor(id, 0, CS_ARMOR_NONE)
            }
        }
    }
} 


The bug with item_assaultsuit shoud be fixed in any plugin using it but setting the armotype offset, or by replacing give_item with cs_set_user_armor.

[Hawk552]

This plugin is well done.

If you would like any information regarding possible adjustments you could make or things you could do to make this better, please feel free to post here or PM me.

Approved.