GUI Triggering Issue

Pardon the poor triggering, but there's something that's screwing up in this...
That is... when I change it ever so slightly.

ATM, this trigger works fine at the very end with a "Do Nothing" in the else actions. If I change that with a skip all remaining actions (Which I want there in case another player casts the spell so the damage won't accidentally stack in the case that one is disrupted and another is set on top of it right after the sleep is disrupted for double damage), the damage will not occur and the rest of the trigger is SKIPPED.

I thought at first it might be due to the fact that sleep uses 3 buffs (So I included all 3 with an or condition up there), but that did not solve my problems either... Any ideas?

And what is your problem? That action called "Skip remaining actions" really skips the rest of the trigger?

My problem is that if I replace the Do Nothing w/ Skip Remaining Actions, the damage and stuff never occurs, even if the unit has the buff...

I think you may want to start over and could do it without a loop. What does this spell do exactly? Stuns and damages the target every second until the target reaches a certain low amount of HP? And I assume the spell itself is only a stun so you are adding the damage over time component with a trigger?

EDIT dunno what i was thinking... you would need a loop

Try changing the "Starts the effect..." with "finishes casting" or add small wait on first line. I guess the ability being cast is some sort of sleep...the sleep effect probably starts AFTER the first pass through the loop => the condition fails on the first pass => the else statement is executed => it is not a problem when there is Do Nothing, but with "Skip..." it terminates the trigger. I hope you understand.

I tried changing it to when a unit finishes casting and that still didn't help. The same effect (No damage) occurred.

Add cca 0.5 sec wait on the first line of the trigger.

Try this. I replaced your variable with the event response (Target unit of ability being cast), so now your spell should be multi-instancable.

EDIT: This means you should be able to have have multiple units casting this spell, which I think was your original reason for trying to add a "skip remaining actions"

That doesn't help... at all. My spell is already the level of MUI I want it to be, capable of having 1 per player.

I don't think you understand my problem entirely Fluff...

This is a sleep spell. Let's say Player 1 casts this spell on Player 2, which lasts 9 seconds.

2 seconds in after casting the spell, Player 3 wakes up Player 2 & casts the same spell. As a result, Player 1's damage (What would still be going on) & Player 3's damage are now affecting Player 2.

The whole point of "Skip Remaining Actions" was for that case. (So the damages wouldn't bug and accidentally stack).

Try the wait or I will have to try it myself :P

Renamed thread to something slightly more descriptive.

The complexity of your triggering is reaching the level where you would have an easier time writing and getting help with scripting JASS directly.

What you want is actually simpler than this. You will need two triggers.

Trigger 1:
When the ability is cast, add the target to a unit group

Trigger 2:
Every second, loop through the unit group. If the a unit in the group doesn't have the buff, remove it from the group, otherwise deal damage to it.

This will work well because it completely avoids using waits and checking for double-casting, etc.

Strilanc...

1. Your method doesn't support leveling damage (X/X/X damage per second with level).

2. Your method also will not credit the caster for a kill if the target dies under this.

You could fix the first problem by simply making 3 groups, or using a group array.

Then use a couple arrays instead of a group. You'd need a target array and a caster array. Arrays will probably even be faster than groups because you'll know the index when you want to remove an element, letting you do it in O(1) time instead of O(n).