How is this possible!!!?

I am getting impossible behavior from an if statement. Observe:


That is just the critical parts, the full trigger is here if you want.

Hidden information: Trigger: ManablastStops Events Unit - A unit Stops casting an ability Unit - A unit Finishes casting an ability Conditions (Ability being cast) Equal to Mana Blast Actions Custom script: local integer udg_i Set u = (Triggering unit) Custom script: set udg_i = GetAttachedInt(udg_u, "owner") -------- REMOVE THE CASTER UNIT -------- Game - Display to (All players) the text: blasterstop Unit Group - Remove u from blastingnodes[i] Custom script: call CleanAttachedVars(udg_u) Unit - Remove u from the game Game - Display to (All players) the text: (String((Number of units in blastingnodes[i]))) -------- NO MORE NODES MANA BLASTING, STOP HERO -------- If (All Conditions are True) then do (Then Actions) else do (Else Actions) If - Conditions (Number of units in blastingnodes[i]) Equal to 0 Then - Actions Game - Display to (All players) the text: herostopping Unit - Pause u Unit - Order Hero[i] to Stop Unit - Unpause u Else - Actions

What this prints out is

blasterstop
1
herostopping

So it is saying that the number of units in the unit group == 1 and then it is doing stuff inside the if statement that requires the number of units to be == 0!

Learn JASS. GUI merely removes you from actual code.

The if statement is in a different function. Use a global, or use JASS. Don't use locals in GUI unless you know what you are doing.

Why the hell are you even using a local there? It has no purpose.

Doh. That is not the first time that one has got me. I know JASS, GUI is just easier most of the time. But then it does silly stuff like that. What business does it have making a seperate function for the if statement?

I added the local variable because of all the triggers flying around triggering each other and screwing up the globals. This trigger was firing off from some actions in another trigger using the global i variable and I didn't what it to get changed. But now that I look at the triggering trigger it seems I don't use udg_i after this one gets called. Maybe I needed it at some point but I guess I don't now.

you only need locals where there is some kind of wait or delay involved.

That's not true. And apparently I do need that variable to be local, it is changing around as the trigger runs for who knows what reason.

Phoenix, consider this example that can screw up globals with no waits (too lazy to make some real triggers):

When trigger 1 stops the unit trigger 2 will immediately fire and change udg_u to "some crazy unit" and then trigger 1 will end up moving "some crazy unit" instead of the original unit.