Okay, I found the problem. I really didn't see it before I was about to re-write it.
You have passed on a symbol called "id" in your regen_loop, but you do not pass on anything called id in the task you set. I assume you want it to be a player id. It can't just find a player id out of no where. I re-wrote the whole thing, and it might not be the most optimized ways of doing it, but I believe this should get the job done.
As you have no id in the loop, you can't check nobody's armor, and that is probably why it wont work.
PHP Code:
/* Plugin generated by AMXX-Studio */
#include <superheromod>
new gHeroName[] = "Regen"
//bool for heroes
new bool:gHasRegen[SH_MAXSLOTS+1]
new bool:gHasWolv[SH_MAXSLOTS+1]
new bool:gHasDarth[SH_MAXSLOTS+1]
new bool:gHasRiddick[SH_MAXSLOTS+1]
//Hero ID's
new gHeroID
new WolvID
new darthID
new riddickID
//pcvars
new pcvarHPHeal
new pcvarAPHeal
public plugin_init()
{
register_plugin("SUPERHERO Regen", "1.0", "Exploited/Jelle")
new pcvarLevel = register_cvar("regen_level", "10")
pcvarHPHeal = register_cvar("regen_healhp", "30")
pcvarAPHeal = register_cvar("regen_healap", "30")
gHeroID = sh_create_hero(gHeroName, pcvarLevel)
sh_set_hero_info(gHeroID, "Regeneration", "High speed regeneration!")
set_task(1.0, "regen_loop", _, _, _, "b")
}
public plugin_cfg()
{
WolvID = sh_get_hero_id("Wolverine")
darthID = sh_get_hero_id("Darth Maul")
riddickID = sh_get_hero_id("Riddick")
}
public sh_hero_init(id, heroID, mode)
{
if ( gHeroID == heroID )
{
switch(mode)
{
case SH_HERO_ADD:
{
gHasRegen[id] = true
}
case SH_HERO_DROP:
{
gHasRegen[id] = false
}
}
}
else if ( WolvID == heroID )
{
switch(mode)
{
case SH_HERO_ADD:
{
gHasWolv[id] = true
}
case SH_HERO_DROP:
{
gHasWolv[id] = false
}
}
}
else if ( darthID == heroID )
{
switch(mode)
{
case SH_HERO_ADD:
{
gHasDarth[id] = true
}
case SH_HERO_DROP:
{
gHasDarth[id] = false
}
}
}
else if ( riddickID == heroID )
{
switch(mode)
{
case SH_HERO_ADD:
{
gHasRiddick[id] = true
}
case SH_HERO_DROP:
{
gHasRiddick[id] = false
}
}
}
}
public regen_loop()
{
if ( !sh_is_active() ) return
new players[SH_MAXSLOTS], playerCount, player, i
get_players(players, playerCount, "ah")
for ( i = 0; i < playerCount; i++ )
{
player = players[i]
//return if user has either one or more of the healing heroes
if ( gHasWolv[player] || gHasDarth[player] || gHasRiddick[player] ) return
if ( gHasRegen[player] )
{
sh_add_hp(player, get_pcvar_num(pcvarHPHeal) )
new userArmor = get_user_armor(player) + get_pcvar_num(pcvarAPHeal)
cs_set_user_armor(player, userArmor, CS_ARMOR_KEVLAR )
}
}
}
Why didn't I went with cs_get_user_armor when I use cs_set_user_armor? Well simply because I kept getting tag mismatch, as it seems like cs_get_user_armor can only get one type of armor, either west + helm or only west. Anyone knows about that?
EDIT:
I know if I don't add this, G-Dog and fr33m@n will be on my back about it, so here it is:
You can also do it this way in the hero init:
PHP Code:
public sh_hero_init(id, heroID, mode)
{
if ( gHeroID == heroID )
{
gHasRegen[id] = mode ? true : false
}
else if ( WolvID == heroID )
{
gHasWolv[id] = mode ? true : false
}
else if ( darthID == heroID )
{
gHasDarth[id] = mode ? true : false
}
else if ( riddickID == heroID )
{
gHasRiddick[id] = mode ? true : false
}
}
Then you just won't have the possibility to go to a forward if the user picks/drops the hero. If you want to use the hero as I posted, use this way of setting the Booleans to true or false, as that is the most efficient.
Linear Mode
