[fezh]

Spanish Translation:

[es]
MINIGUN_ADMIN_GIVE_ALL = El administrador te a dado una Minigun con %d balas!
MINIGUN_ADMIN_GIVE = Administrador dio %s %d Miniguns
MINIGUN_ADMIN_BUY = Solo los adminsitradores pueden comprar esto!
MINIGUN_ALIVE = Solo puedes comprar una Minigun estando vivo.
MINIGUN_ALREADY = Ya tienes una Minigun.
MINIGUN_NO_MONEY = No tienes dinero suficiente!!!
MINIGUN_BUY = Has comprado una Minigun por $%i.
MINIGUN_FOR_BUY = Para comprar una Minigun, escribe /minigun en el chat.
MINIGUN_PRICE = Precio Minigun- $%d

[joaquimandrade]

CLLlAgOB, this code needs a full re-write.

As an example, the function remowegun is like to hunt a bunny with a militar tank. You can make it as this instead:

PHP Code:

public remowegun(id) 
{
    fm_strip_user_gun(id,CSW_M249)
} 


[CLLlAgOB]

Originally Posted by joaquimandrade CLLlAgOB, this code needs a full re-write. As an example, the function remowegun is like to hunt a bunny with a militar tank. You can make it as this instead: PHP Code: public remowegun(id)  {     fm_strip_user_gun(id,CSW_M249) }

You mean what better to use the module fakemeta?

[eforie]

"You now how cuth a moment a human state zombie? "
i don`t know

[CLLlAgOB]

Originally Posted by joaquimandrade CLLlAgOB, this code needs a full re-write. As an example, the function remowegun is like to hunt a bunny with a militar tank. You can make it as this instead: PHP Code: public remowegun(id)  {     fm_strip_user_gun(id,CSW_M249) }

Fix and use fakemeta

[joaquimandrade]

Originally Posted by CLLlAgOB You mean what better to use the module fakemeta?

It's not about the module is about the way of doing each part of the plugin. In that function you are removing every weapon of the player and then giving him again those that are not a minigun. What you should do instead? Simply remove the minigun. Like you have it, not only it doesn't lots of unneeded things, as it doesn't care about the weapons' ammo.

Another example:

You are using event_start_freezetime to check if a player that survived the round has a minigun and to show a message to every player. Instead hook Ham_Spawn pre and check if the player is alive. And hook Ham_Spawn post and show him the message. This way you won't miss players that spawned after freeze time, you will not make checks for player that didn't survived the round and, you increase the cross-mod compatible code, since the HLTV event is CS specific.

I also doubt that you need this hook: fwEvGameWillRestartIn

You are not using pfn_touch

You have g_MaxPlayers but you don't use it always you can and when you use it you use it wrong:

for(id = 1; id < g_MaxPlayers; id++) {

(Should be <= )

get_user_msgid("StatusText") should be cached.

You code style varies from function to function, what shows that it is composed of several pieces glued together instead of you making it your style and your way.

[merkava]

Polish translate.

[pl]
MINIGUN_ADMIN_GIVE_ALL = Admin dal Ci obrotowke z %d nabojami!
MINIGUN_ADMIN_GIVE = Admin dal %s %d obrotowke.
MINIGUN_ADMIN_BUY = Tylko admin moze kupic ta bron!
MINIGUN_ALIVE = Mozesz kupic ta bron jesli jestes zywy.
MINIGUN_ALREADY = Masz obrotowke!
MINIGUN_NO_MONEY = Masz za malo kasy!
MINIGUN_BUY = Kupiles obrotowke za $%i.
MINIGUN_FOR_BUY = Zeby kupic obrotowke, wpisz w sayu /minigun
MINIGUN_PRICE = Koszt obrotowki - $%d

[CLLlAgOB]

Originally Posted by joaquimandrade for(id = 1; id < g_MaxPlayers; id++) { (Should be <= ) get_user_msgid("StatusText") should be cached.

if use <= id get max vavle 33
mb this
for(id = 0; id < g_MaxPlayers; id++)
id = min 1 max 32

[TitANious]

It should be 33 if i'm right new g_MaxPlayers[ 33 ] = get_maxplayers ()

[Arkshine]

A player's index can be 1 to 32 maximum. ( 0 is not valid ) But 32 because it's a 32 slots. If you have 16 slots, it will be 16 max. So the right usage is to loop between 1 and max_client included. so it should be "id <=".