[Trigger] A trouble with trigger...

[blah900]

Can someone please tell me why this won't work? This is an ability with a stormbolt as a dummy spell. The dummy caster's ability is based on the spell impale. When the spell is cast on a target, it's supposed to cast impale six times on a target, but won't continue after the first one. I use WE unlimited. Please help ><

• Druid Wrath of Nature
  • Events
    • Unit - A unit Begins casting an ability
  • Conditions
    • (Ability being cast) Equal to Wrath of Nature (Druid)
  • Actions
    • Unit - Create 1 Dummy Caster for (Owner of (Casting unit)) at ((Position of (Target unit of ability being cast)) offset by 50.00 towards 25.00 degrees) facing (Position of (Target unit of ability being cast))
    • Unit - Add Druid Dummy Ability (Wrath of Nature) to (Last created unit)
    • Unit - Order (Last created unit) to Undead Crypt Lord - Impale (Position of (Target unit of ability being cast))
    • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
    • Wait 1.00 seconds
    • Unit - Create 1 Dummy Caster for (Owner of (Casting unit)) at ((Position of (Target unit of ability being cast)) offset by 50.00 towards 25.00 degrees) facing (Position of (Target unit of ability being cast))
    • Unit - Add Druid Dummy Ability (Wrath of Nature) to (Last created unit)
    • Unit - Order (Last created unit) to Undead Crypt Lord - Impale (Position of (Target unit of ability being cast))
    • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
    • Wait 1.00 seconds
    • Unit - Create 1 Dummy Caster for (Owner of (Casting unit)) at ((Position of (Target unit of ability being cast)) offset by 50.00 towards 25.00 degrees) facing (Position of (Target unit of ability being cast))
    • Unit - Add Druid Dummy Ability (Wrath of Nature) to (Last created unit)
    • Unit - Order (Last created unit) to Undead Crypt Lord - Impale (Position of (Target unit of ability being cast))
    • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
    • Wait 1.00 seconds
    • Unit - Create 1 Dummy Caster for (Owner of (Casting unit)) at ((Position of (Target unit of ability being cast)) offset by 50.00 towards 25.00 degrees) facing (Position of (Target unit of ability being cast))
    • Unit - Add Druid Dummy Ability (Wrath of Nature) to (Last created unit)
    • Unit - Order (Last created unit) to Undead Crypt Lord - Impale (Position of (Target unit of ability being cast))
    • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
    • Wait 1.00 seconds
    • Unit - Create 1 Dummy Caster for (Owner of (Casting unit)) at ((Position of (Target unit of ability being cast)) offset by 50.00 towards 25.00 degrees) facing (Position of (Target unit of ability being cast))
    • Unit - Add Druid Dummy Ability (Wrath of Nature) to (Last created unit)
    • Unit - Order (Last created unit) to Undead Crypt Lord - Impale (Position of (Target unit of ability being cast))
    • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
    • Wait 1.00 seconds
    • Unit - Create 1 Dummy Caster for (Owner of (Casting unit)) at ((Position of (Target unit of ability being cast)) offset by 50.00 towards 25.00 degrees) facing (Position of (Target unit of ability being cast))
    • Unit - Add Druid Dummy Ability (Wrath of Nature) to (Last created unit)
    • Unit - Order (Last created unit) to Undead Crypt Lord - Impale (Position of (Target unit of ability being cast))
    • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
    • Wait 1.00 seconds

 

[Just_Spectating]

well, you cant impale a unit more than once at a time if thats what your talking about.

and after a wait, the "casting unit" memory is lost, so it forgets what unit casts the ability. you will have to use a variable. also, the event "starts the effect of an ability" is much more efficient.

 

[PurplePoot]

Or you can use Triggering Unit, Just_Spectating. It's better anyways.

Oh, and you want Starts the Effect of an Ability. And not cause it's more efficient (it isn't to my knowledge), but because better conditions cause it.

I also added

• tags to your post.
• You should look into loops, too.
• (note: this needs a variable called TempPoint and another called TempPoint2, both of which are Points, and a unit variable called TempUnit)
• [trigger]Druid Wrath of Nature
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Wrath of Nature (Druid)
  • Actions
    • Custom script: local unit udg_TempUnit
    • Set TempUnit = (Target Unit of Ability Being Cast)
    • For Each Integer A from 1 to 6, do actions
      • Loop - Actions
        • Set TempPoint = (Position of TempUnit)
        • Set TempPoint2 = (TempPoint offset by 50 towards 25 degrees)
        • Unit - Create 1 Dummy Caster for (Owner of (Triggering Unit)) at TempPoint2 facing TempPoint
        • Unit - Add Druid Dummy Ability (Wrath of Nature) to (Last created unit)
        • Unit - Order (Last created unit) to Undead Crypt Lord - Impale TempPoint
        • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        • Custom script: call RemoveLocation(udg_TempPoint)
        • Custom script: call RemoveLocation(udg_TempPoint2)
        • Wait 1.00 seconds
    • Set TempUnit = No Unit

 

[blah900]

err what do those custom scripts mean? Sorry, never used custom scripts in my life before... lol...

 

[Just_Spectating]

they remove memory leaks, which, if you have enough leaks, will lag the game.