A. Good idea.
B. bad script.
B1. If a user switches team, he will still have unlimited nades but will be on the opposite team.
B2. If a user joins a team that has unlimited nades, he won't get them.
B3. client_PostThink to set their number of nades is pretty harsh on the server compared to what it needs to be. Try hooking this:
Code:
register_event("StatusIcon","givenades","be","1=1")
And for B1+2 you need to remake the script. Make 2 global variables, one that is set if a whole team should be given ammo (variable[5]) and one like it is now but only used if a specific player is targeted for unlimited ammo. For example, this would be the line in the function that is the forward from the "StatusIcon" hook:
Code:
if(has_unlimited[id] || if(team_has_unlimited[get_user_team(id)])