Weapons War v7.5 (26 Weapons) - Page 20

zmd94 · 10-06-2015 , 14:03 Re: Weapons War v7.1 (26 Weapons)

1. Sorry, do you mean just change below:

Code:

    get_string(1, g_sWeaponWar, charsmax(g_sWeaponWar))
    if(strlen(g_sWeaponWar) < 1)
    {
        log_error(AMX_ERR_NATIVE, "cs_war_set native is incorrect. Plugin can't register custom war with an empty name")
        return 0
    }

-->

Code:

    get_string(1, g_sWeaponWar, charsmax(g_sWeaponWar))
    if(g_sWeaponsWar[0] == EOS)
    {
        log_error(AMX_ERR_NATIVE, "cs_war_set native is incorrect. Plugin can't register custom war with an empty name")
        return 0
    }

2. Next, which native do you mean?

Quote:

Again, in cs_war_round_end you can work directly with WeaponCannotDrop[i](in the for I mean). While you can argue that this is for readability it does not make sense to call another native in the loop for nothing.

3.

Quote:

cs_fw_war_spawn is pointless, you are calling it from Ham, why should someone not use directly Hamsandwich ?

cs_fw_war_spawn forward is useful to any developers is they want to hook player spawn event during the war round.

**HamletEagle** · 10-06-2015 , 14:05 Re: Weapons War v7.1 (26 Weapons)

1.Yes.
2.As I said two times in the review, no native, work directly with WeaponsData[i][WeaponID] in the for(in RegisterHam or whatever you are doing).
3.Ham_Spawn + cs_is_war_round.

zmd94 · *Last edited by zmd94; 10-07-2015 at 00:05.*

Alright.

zmd94 · *Last edited by zmd94; 10-07-2015 at 06:24.*

Weapons War v7.2

(26 Weapons)

Status:
UP-TO-DATE (7/10/2015)

Changelog:

Code:

v7.2
- [FIXED] Remove unnecessary natives and forward.
- [FIXED] cs_war_set native. Add code to get weapon id associated with a custom weapon name.
- [FIXED] cs_start_war and cs_start_vote natives. Add code to determine whether cs_war_set native is already being used. If not, it will give error.
- [ADDED] Option to block rodio sound or message of "Fire in the hole!" during HE-grenade war.
- [FIXED] Example using of custom cs_war.inc in custom_war.sma and war_trail.sma files.
- [FIXED] Find leader code.
- [ADDED] New senteces!

Credit:

Quote:

1. xPaw (https://forums.alliedmods.net/showpo...88&postcount=4)

New senteces:

Code:

CSWAR_NEW_LEADER = New points leader!
CSWAR_SAME_LEADER = Same points leader from previous!
CSWAR_WHOIS_LEADER = %s is new points leader!

zmd94 · 10-13-2015 , 23:05 Re: Weapons War v7.2 (26 Weapons)

Any news, comments or suggestions?

addons_zz · 10-13-2015 , 23:59 Re: Weapons War v7.2 (26 Weapons)

Quote:

Originally Posted by zmd94

Any news, comments or suggestions?

It is an awesome plugin

zmd94 · 10-14-2015 , 00:38 Re: Weapons War v7.2 (26 Weapons)

I appreciate it. ;)

Klatypaus · 10-15-2015 , 11:03 Re: Weapons War v7.2 (26 Weapons)

zmd The whole plugin is well coded and works perfectly fine. But the bug which I pointed out earlier still exists. That is, players who use auto buy scripts(not all scripts work) can still buy other weapons in the buy zone. I suggest you to disable the buy zone completely like in Simple Knife Warmup plugin

zmd94 · 10-15-2015 , 19:08 Re: Weapons War v7.2 (26 Weapons)

Alright. I just want your confirmation, so the player will only can use the auto-buy scripts in buy zone?

zmd94 · *Last edited by zmd94; 10-16-2015 at 11:18.*

Klatypaus, after tested the plugin many times, there will be no problem if player use the auto-buy scripts in buy zone. This is because if the weapon that they are currectly holding is not the same with current weapon war, the weapon will become unusable or unfunctional.