[yousef]

Deagle Round

Admin Commands:
deagle_round : starts a deagle round all players will have deagle weapon with 10 bullets and cant buy any weapon or pick up any weapon. when rounds finished every thing will be back normal

Cvars:
deagle_ammo "10" ( default: 10 )
sets deagle ammo
deagle_wait "20" ( default: 20 )
this cvar will make admins cant active deagle round when passed 20 seconds in round

Modules:
Fun
Engine
Cstrike

Credits:
Exolent[jNr] disable buy commands
Pokemonmaster disable pick up weapons

[sami_spt]

Post the link for colorchat include so ppl can download.

plugin can be made better

[yousef]

Originally Posted by sami_spt Post the link for colorchat include so ppl can download.

Done, Thanks

[ConnorMcLeod]

Problems :

Trie is destroyed at round end, so you can buy is deagle round is set on another round than very 1st one.

If players open buy menu before deagle round is activated, they can buy weapons with the menu once weapons have been stripped and new weapons have been given.


Must :

Use cvar pointers
Don't retrieve 2 times the same cvar value in the same function (deagle_wait)
Change the way to set deagle ammo, actual way is not reliable.


Should :

Use engine client_kill forward instead of FM since engine is already included. (then FM is not required anymore).
You don't need to check teams in client_command forward.

[yousef]

Originally Posted by ConnorMcLeod Problems : Trie is destroyed at round end, so you can buy is deagle round is set on another round than very 1st one. If players open buy menu before deagle round is activated, they can buy weapons with the menu once weapons have been stripped and new weapons have been given. Must : Use cvar pointers Don't retrieve 2 times the same cvar value in the same function (deagle_wait) Change the way to set deagle ammo, actual way is not reliable. Should : Use engine client_kill forward instead of FM since engine is already included. (then FM is not required anymore). You don't need to check teams in client_command forward.

Check plugin now, thanks for help

[ConnorMcLeod]

Trie was a good way because you were lowering chars before check commands, now you only register the lower case and it can easily be bypassed.
Just put back the Trie but do never destroy it.

[yousef]

Originally Posted by ConnorMcLeod Trie was a good way because you were lowering chars before check commands, now you only register the lower case and it can easily be bypassed. Just put back the Trie but do never destroy it.

Done.
Questions:
1. I dont have to do TrieDestroy ?
2. What if i added this ?

PHP Code:

public plugin_end( )
{
    TrieDestroy( g_tBuyCommands );
} 


[ConnorMcLeod]

tries are automatically destroyed, you don't need to put this, but there is no problem if you destroy it there.

You could check deagleround value as first check on client_command.


You must use deagle_wait cvar pointer
you don't need to convert its integer value to a float since systime is integer.

You may register "weapon_shield" touch as well as weaponbox and armoury_entity
In touch callback, you should at least check if iToucher is in players range (1 to maxplayers)

You must use deagle_ammo cvar pointer

find_ent_by_owner is not a good way to retrieve weapon index.
Also, as you strip weapons before you give deagle, you should retrieve deagle index with give_item return value, then if value is <= 0, you can try to retrieve it with find_ent_by_owner, and then if value is > 0, you can set ammo on it.

[yousef]

Originally Posted by ConnorMcLeod tries are automatically destroyed, you don't need to put this, but there is no problem if you destroy it there. You could check deagleround value as first check on client_command. You must use deagle_wait cvar pointer you don't need to convert its integer value to a float since systime is integer. You may register "weapon_shield" touch as well as weaponbox and armoury_entity In touch callback, you should at least check if iToucher is in players range (1 to maxplayers) You must use deagle_ammo cvar pointer find_ent_by_owner is not a good way to retrieve weapon index. Also, as you strip weapons before you give deagle, you should retrieve deagle index with give_item return value, then if value is <= 0, you can try to retrieve it with find_ent_by_owner, and then if value is > 0, you can set ammo on it.

I tried to use cvar pointer but it didnt work first when i used it with "deagle_wait" it didnt tell me that you cant activate deagle round after 20 seconds i played like 1 min and didnt tell me that .. and second i used it with deagle_ammo it gave me 96 bullets and it cvar deagle_ammo was 10
and when i do like this

PHP Code:

cs_set_weapon_ammo( Players, deagleammo ); 


plugin show me error in console

PHP Code:

L 08/17/2013 - 16:57:39: [CSTRIKE] Non-player entity 1 out of range
L 08/17/2013 - 16:57:39: [AMXX] Displaying debug trace (plugin "deagle_round.amxx")
L 08/17/2013 - 16:57:39: [AMXX] Run time error 10: native error (native "cs_set_weapon_ammo")
L 08/17/2013 - 16:57:39: [AMXX]    [0] deagle_round.sma::DRound (line 113) 


thats why i used find_ent_by_owner anyways, check plugin now and thanks for your help.

[WhyThis]

why did you don't add color chat in stock -_-
it will be better !