problem w/ a skill trigger

[meercat]

im having a problem with a skill. Im making a skill named Tidal Crush, which creates waves from four diferent directions (Namely West, East, North and South) that will meet in a center point. When the tides meets in the center, it will create a crushing force that stuns enemies caught. Now my problem is, everytime I try to cast the skill; it does nothing........ Can anyone help me?

Here are the trigers.

• Tidal Crush
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Tidal Crush
  • Actions
    • Set SkillPos = (Position of (Casting unit))
    • Set SkillPointPos = (Target point of issued order)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Level of Tidal Crush for (Casting unit)) Equal to 1
      • Then - Actions
        • Unit - Create 1 Tidal Crush (Level 1) for (Owner of (Casting unit)) at (SkillPos offset by (-5.00, 0.00)) facing Default building facing degrees
        • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (-5.00, 0.00))
        • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        • Unit - Create 1 Tidal Crush (Level 1) for (Owner of (Casting unit)) at (SkillPos offset by (5.00, 0.00)) facing Default building facing degrees
        • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (5.00, 0.00))
        • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        • Unit - Create 1 Tidal Crush (Level 1) for (Owner of (Casting unit)) at (SkillPos offset by (0.00, 5.00)) facing Default building facing degrees
        • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (0.00, 5.00))
        • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        • Unit - Create 1 Tidal Crush (Level 1) for (Owner of (Casting unit)) at (SkillPos offset by (0.00, -5.00)) facing Default building facing degrees
        • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (0.00, -5.00))
        • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        • Unit - Create 1 Tidal Crush (Level 1) for (Owner of (Casting unit)) at SkillPointPos facing Default building facing degrees
        • Unit - Order (Last created unit) to Orc Tauren Chieftain - War Stomp
        • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
      • If - Conditions
        • (Level of Tidal Crush for (Casting unit)) Equal to 2
        • Then - Actions
          • Unit - Create 1 Tidal Crush (Level 2) for (Owner of (Casting unit)) at (SkillPos offset by (-5.00, 0.00)) facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (-5.00, 0.00))
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Unit - Create 1 Tidal Crush (Level 2) for (Owner of (Casting unit)) at (SkillPos offset by (5.00, 0.00)) facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (5.00, 0.00))
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Unit - Create 1 Tidal Crush (Level 2) for (Owner of (Casting unit)) at (SkillPos offset by (0.00, 5.00)) facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (0.00, 5.00))
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Unit - Create 1 Tidal Crush (Level 2) for (Owner of (Casting unit)) at (SkillPos offset by (0.00, -5.00)) facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (0.00, -5.00))
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Unit - Create 1 Tidal Crush (Level 2) for (Owner of (Casting unit)) at SkillPointPos facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - War Stomp
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
      • If - Conditions
        • (Level of Tidal Crush for (Casting unit)) Equal to 3
        • Then - Actions
          • Unit - Create 1 Tidal Crush (Level 3) for (Owner of (Casting unit)) at (SkillPos offset by (-5.00, 0.00)) facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (-5.00, 0.00))
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Unit - Create 1 Tidal Crush (Level 3) for (Owner of (Casting unit)) at (SkillPos offset by (5.00, 0.00)) facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (5.00, 0.00))
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Unit - Create 1 Tidal Crush (Level 3) for (Owner of (Casting unit)) at (SkillPos offset by (0.00, 5.00)) facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (0.00, 5.00))
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Unit - Create 1 Tidal Crush (Level 3) for (Owner of (Casting unit)) at (SkillPos offset by (0.00, -5.00)) facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - Shockwave (SkillPos offset by (0.00, -5.00))
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
          • Unit - Create 1 Tidal Crush (Level 3) for (Owner of (Casting unit)) at SkillPointPos facing Default building facing degrees
          • Unit - Order (Last created unit) to Orc Tauren Chieftain - War Stomp
          • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        • Else - Actions

 

[Diablo-dk]

You used Set SkillPointPos = (Target point of issued order) replace it with (Target point of Ability being cast).

Also by the way, you leak alot of locations.
Point with polar offset also returns a location, meaning you have to store it into a variable and then remove it after you don't need it anymore with:

• Custom script: call RemoveLocation(udg_<LocationName>)

 

[Dr Super Good]

• Unit - A unit Starts the effect of an ability
• Set SkillPointPos = (Target point of issued order)

You set "SkillPointPos" to "null" since the spell is not triggered on a unit issues an order targeting a poin but on "A unit Starts the effect of an ability" and thus you must get the target point of ability being cast to get the point where the user targeted. Maybe think before using actions that do not match the event you use to trigger the trigger.