Zombie XP system

Let's start with the fixes now.

1.
date
you accidentally wrote "date" instead of "data"

-- > 

data
2. zombie_xp.txt

some corrections

NO_ITEMS = ^4[ZX] ^3There ^4No ^3Items
ZX_DEAD_MSG = ^4[ZX] ^3Dead Player ^4 Can't But Items
ZX_ITEMS_DISABLED = ^4[ZX] Extra Items Is ^3Disabled
-->

NO_ITEMS = ^4[ZX] ^3There ^4Are ^3No ^4Items
ZX_DEAD_MSG = ^4[ZX] ^3Dead Players ^4 Can't Buy Items
ZX_ITEMS_DISABLED = ^4[ZX] Extra Items Are ^3Disabled


---

3. zombie_xp.sma 

- 0.1 : 
Why are u defining these? You already have ZP_TEAM_x in zombieplague.inc

#define TEAM_ZOM (1<<0)
#define TEAM_HUM (1<<1)
- 0.2 :

delete amxmisc from includes

- 0.3: 

Why do you confuse yourself finding task ID ?

new ids = id - TASK_HUD_REMOVE
-- >
change ids -- > id everywhere in that task and do this in other tasks.

id -= TASK_HUD_REMOVE
- 0.4: 

ShowSyncHudMsg(ids ,zx_hud , "[Zombie XP system]^n[XP : %i | MAXED]
-- >

just a typo

ShowSyncHudMsg(ids ,zx_hud , "[Zombie XP system]^n[XP : %i | MAX]
- 0.5 

in func_damage add a global check, "is_user_valid(attacker)" at the beginning of the func, so it won't throw an invalid player ID error. And change idinflictor to inflictor.

public func_damage(victim, idinflictor, attacker, Float:damage, damagebits)

{

if (playerlvl[attacker] >= 5 && zp_get_user_human(attacker))

SetHamParamFloat(4 , damage * 1)



if (playerlvl[attacker] >= 15 && zp_get_user_human(attacker))

SetHamParamFloat(4 , damage * 1.1)



if (playerlvl[attacker] >= 20 && zp_get_user_human(attacker))

SetHamParamFloat(4 , damage * 1.3)



if (playerlvl[attacker] >= 30 && zp_get_user_human(attacker))

SetHamParamFloat(4 , damage * 1.5 )



if (playerlvl[attacker] >= 5 && zp_get_user_zombie(attacker))

SetHamParamFloat(4 , damage + 5.0)



if (playerlvl[attacker] >= 10 && zp_get_user_zombie(attacker))

SetHamParamFloat(4 , damage + 20.0)



if (playerlvl[attacker] >= 20 && zp_get_user_zombie(attacker))

SetHamParamFloat(4 , damage + 40.0)



if (playerlvl[attacker] >= 30 && zp_get_user_zombie(attacker))

SetHamParamFloat(4 , damage + 50.0)

}
-->
public func_damage(victim, inflictor, attacker, Float:damage, damagebits)

{



if(is_user_valid(attacker)) {

if (playerlvl[attacker] >= 5 && zp_get_user_human(attacker))

SetHamParamFloat(4 , damage * 1)



if (playerlvl[attacker] >= 15 && zp_get_user_human(attacker))

SetHamParamFloat(4 , damage * 1.1)



if (playerlvl[attacker] >= 20 && zp_get_user_human(attacker))

SetHamParamFloat(4 , damage * 1.3)



if (playerlvl[attacker] >= 30 && zp_get_user_human(attacker))

SetHamParamFloat(4 , damage * 1.5 )



if (playerlvl[attacker] >= 5 && zp_get_user_zombie(attacker))

SetHamParamFloat(4 , damage + 5.0)



if (playerlvl[attacker] >= 10 && zp_get_user_zombie(attacker))

SetHamParamFloat(4 , damage + 20.0)



if (playerlvl[attacker] >= 20 && zp_get_user_zombie(attacker))

SetHamParamFloat(4 , damage + 40.0)



if (playerlvl[attacker] >= 30 && zp_get_user_zombie(attacker))

SetHamParamFloat(4 , damage + 50.0)

}



return HAM_IGNORED;

}

- 0.6

Don't register natives with "1" parameter, just leave it blank. 

- 0.7 

in natives, use get_param(parameter_number) instead of variables (id, index)

public native_zx_get_user_level(id)

{

if(!is_user_valid(id))

{

log_error(AMX_ERR_NATIVE , "[ZX] Invaild Player (%d)" , id)

return -1

}

return playerlvl[id]

}
--> 

do this in every native.

public native_zx_get_user_level(index)

{

       index = get_param(1)

if(!is_user_valid(index))

{

log_error(AMX_ERR_NATIVE , "[ZX] Invaild Player (%d)" , index)

return -1

}

return playerlvl[index]

}
- 0.8

create a 1.0 frequency set_task and check there everything you need instead of making a lot of tasks.

- 0.9 

use formatex instead of format.

- 1.0

your nvault save/retrieve methods are like the old one. Look at my plugin LXS.

- 1.1 

remove IP/Name data save.

Anyways about the methods, it's your choice.
Formatex is a requirement. I know there's places where you can't use formatex but you should use it where you can.
You didn't understand what I said. You don't need these defines. They're already included in zombieplague.inc /* Teams for zp_register_extra_item() */
#define ZP_TEAM_ZOMBIE (1<<0)
#define ZP_TEAM_HUMAN (1<<1)
#define ZP_TEAM_NEMESIS (1<<2)
#define ZP_TEAM_SURVIVOR (1<<3)
#define ZP_TEAM_SNIPER (1<<4)
#define ZP_TEAM_ASSASSIN (1<<5)

Natives on style 1 are buggy. Change it to 0 like I said.
SteamID doesn't change, only if you use CT Shield.

Yes. You forgot a ")". What's the point of this check? Checking if user is zombie and not zombie !?

if (zp_get_user_zombie(id) && !(team & ZP_TEAM_ZOMBIE))

I see, add credits to Mercylezz for the natives, you didn't make them yourself did you?

NO Credits!!

New Link download pliz ?