[Trigger] Why doesn't it work?

[Destroyer95]

I made a spell but it doesn't work. Can someone help me?

• Circle of Doom
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Circle of Doom
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Level of Circle of Doom for Caster) Equal to 3
      • Then - Actions
        • Set Angle = 0.00
        • Set Distance = 600
        • Set Caster = (Casting unit)
        • Set Caster_Point = (Position of Caster)
        • For each (Integer A) from 1 to 24, do (Actions)
          • Loop - Actions
            • Unit - Create 1 Dummy for (Owner of Caster) at (Caster_Point offset by (Real(Distance)) towards Angle degrees) facing (Position of (Triggering unit))
            • Unit - Add a 8.00 second Generic expiration timer to (Last created unit)
            • Set Angle = (Angle + 15.00)
        • Unit - Create 1 Casting Dummy for (Owner of Caster) at Caster_Point facing Default building facing degrees
        • Unit - Add Flames (Neutral Hostile) to (Last created unit)
        • Unit - Set level of Flames (Neutral Hostile) for (Last created unit) to 3
        • Unit - Add a 8.00 second Generic expiration timer to (Last created unit)
        • Unit - Order (Last created unit) to Human Archmage - Blizzard Caster_Point
      • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Level of Circle of Doom for Caster) Equal to 2
      • Then - Actions
        • Set Angle = 0.00
        • Set Distance = 475
        • Set Caster = (Casting unit)
        • Set Caster_Point = (Position of Caster)
        • For each (Integer A) from 1 to 24, do (Actions)
          • Loop - Actions
            • Unit - Create 1 Dummy for (Owner of Caster) at (Caster_Point offset by (Real(Distance)) towards Angle degrees) facing (Position of (Triggering unit))
            • Unit - Add a 8.00 second Generic expiration timer to (Last created unit)
            • Set Angle = (Angle + 15.00)
        • Unit - Create 1 Casting Dummy for (Owner of Caster) at Caster_Point facing Default building facing degrees
        • Unit - Add Flames (Neutral Hostile) to (Last created unit)
        • Unit - Set level of Flames (Neutral Hostile) for (Last created unit) to 2
        • Unit - Add a 8.00 second Generic expiration timer to (Last created unit)
        • Unit - Order (Last created unit) to Human Archmage - Blizzard Caster_Point
      • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Level of Circle of Doom for Caster) Equal to 1
      • Then - Actions
        • Set Angle = 0.00
        • Set Distance = 350
        • Set Caster = (Casting unit)
        • Set Caster_Point = (Position of Caster)
        • For each (Integer A) from 1 to 24, do (Actions)
          • Loop - Actions
            • Unit - Create 1 Dummy for (Owner of Caster) at (Caster_Point offset by (Real(Distance)) towards Angle degrees) facing (Position of (Triggering unit))
            • Unit - Add a 8.00 second Generic expiration timer to (Last created unit)
            • Set Angle = (Angle + 15.00)
        • Unit - Create 1 Casting Dummy for (Owner of Caster) at Caster_Point facing Default building facing degrees
        • Unit - Add a 8.00 second Generic expiration timer to (Last created unit)
        • Unit - Add Flames (Neutral Hostile) to (Last created unit)
        • Unit - Set level of Flames (Neutral Hostile) for (Last created unit) to 1
        • Unit - Order (Last created unit) to Human Archmage - Blizzard Caster_Point
      • Else - Actions

 

[Reaper2008]

First pick up all the If - Then - Else into an Integer B loop 1-4. Also, how doesn't this work? Does nothing? Does half? Does wrong?

 

[Destroyer95]

Its supposed to make a tentacle prison around the caster, and then damage all units inside it.

It doesn't work at all. Does absolutley nothing.

 

[Yixx]

You need to set the variable 'Caster' before the If/Then/Else, because you use it in the comparison,,
Also it could be done alot easier without any if/then/else, but with a calculation instead,,

 

[Reaper2008]

Yup, yixx is right, just noticed. Do both the loop B and set before the If - Then -Else, then we'll see if there's anything else.

 

[Destroyer95]

Thx alot guys

 

[chilla_killa]

Don't forget to use custom scripts to remove the locations to prevent memory leaks, just look in the Thread "Thing that leak"