[Trigger] Mind to tell me why this is not mui?

[BeTaGod-]

It spawns some blades that flyes off from the caster until reaces the targetted location

Trigger init

• Redemption Orbit Start
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Dummy
  • Actions
    • -------- ---------------------------------------------------------------- --------
    • -------- Don't touch this settings --------
    • -------- ---------------------------------------------------------------- --------
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • RO_Skip Equal to 0
      • Then - Actions
        • Trigger - Turn on Redemption Orbit Loop <gen>
      • Else - Actions
    • Set RO_Skip = (RO_Skip + 1)
    • Set RO_Times = (RO_Times + 1)
    • Set RO_Off[RO_Times] = True
    • Set RO_Caster[RO_Times] = (Triggering unit)
    • Set RO_Player[RO_Times] = (Owner of RO_Caster[RO_Times])
    • Set RO_AbilityLevel[RO_Times] = (Level of (Ability being cast) for RO_Caster[RO_Times])
    • Set RO_Location[RO_Times] = (Position of RO_Caster[RO_Times])
    • Set RO_Location3[RO_Times] = (Target point of ability being cast)
    • Set RO_Distance[RO_Times] = ((Distance between RO_Location[RO_Times] and RO_Location3[RO_Times]) x 1.50)
    • Set RO_Speed[RO_Times] = (RO_Distance[RO_Times] / 100.00)
    • Set RO_Aoe[RO_Times] = 100.00
    • Set RO_Power[RO_Times] = (((Real(RO_AbilityLevel[RO_Times])) x 125.00) + 60.00)
    • Set RO_MissileCount[RO_Times] = (RO_AbilityLevel[RO_Times] x 4)
    • Set RO_Angle[RO_Times] = (360.00 / (Real(RO_MissileCount[RO_Times])))
    • For each (Integer A) from 1 to RO_MissileCount[RO_Times], do (Actions)
      • Loop - Actions
        • Set RO_Location2[RO_Times] = (RO_Location[RO_Times] offset by RO_Speed[RO_Times] towards (RO_Angle[RO_Times] x (Real((Integer A)))) degrees)
        • Unit - Create 1 Dummy for RO_Player[RO_Times] at RO_Location[RO_Times] facing RO_Location2[RO_Times]
        • Set RO_Dummy[RO_Times] = (Last created unit)
        • Animation - Change RO_Dummy[RO_Times]'s size to (160.00%, 160.00%, 160.00%) of its original size
        • Animation - Change RO_Dummy[RO_Times]'s animation speed to 230.00% of its original speed
        • Special Effect - Create a special effect attached to the chest of RO_Dummy[RO_Times] using Abilities\Weapons\SentinelMissile\SentinelMissile.mdl
        • Special Effect - Create a special effect attached to the chest of RO_Dummy[RO_Times] using Abilities\Spells\Undead\AbsorbMana\AbsorbManaBirthMissile.mdl
        • Unit Group - Add RO_Dummy[RO_Times] to RO_Group[RO_Times]
        • Custom script: call RemoveLocation(udg_RO_Location2[udg_RO_Times])
    • Custom script: call RemoveLocation(udg_RO_Location[udg_RO_Times])

Trigger Loop

• Redemption Orbit Loop
  • Events
    • Time - Every 0.02 seconds of game time
  • Conditions
  • Actions
    • -------- ---------------------------------------------------------------- --------
    • -------- Don't touch this settings --------
    • -------- ---------------------------------------------------------------- --------
    • For each (Integer RO) from 1 to RO_Times, do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • RO_Off[RO] Equal to True
          • Then - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • RO_Distance[RO] Less than or equal to 0.00
              • Then - Actions
                • Unit Group - Pick every unit in RO_Group[RO] and do (Actions)
                  • Loop - Actions
                    • Set RO_Location[RO] = (Position of (Picked unit))
                    • Set RO_Location2[RO] = (RO_Location3[RO] offset by (RO_Distance[RO] - RO_Speed[RO]) towards (Angle from RO_Location[RO] to RO_Location3[RO]) degrees)
                    • Unit - Move (Picked unit) instantly to RO_Location2[RO]
                    • Unit - Make (Picked unit) face RO_Location2[RO] over 0.00 seconds
                    • Animation - Change (Picked unit) flying height to 0.00 at 1200.00
                    • Unit - Add a 0.90 second Generic expiration timer to (Picked unit)
                    • Unit Group - Remove (Picked unit) from RO_Group[RO]
                    • Custom script: call RemoveLocation(udg_RO_Location[udg_RO])
                    • Custom script: call RemoveLocation(udg_RO_Location2[udg_RO])
                • Custom script: call RemoveLocation(udg_RO_Location3[udg_RO])
                • Set RO_Speed[RO] = 0.00
                • Set RO_Distance[RO] = 0.00
                • Set RO_Skip = (RO_Skip - 1)
                • Set RO_Off[RO] = False
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • RO_Skip Equal to 0
                  • Then - Actions
                    • Set RO_Times = 0
                    • Trigger - Turn off (This trigger)
                  • Else - Actions
              • Else - Actions
                • Set RO_Distance[RO] = (RO_Distance[RO] - RO_Speed[RO])
                • Unit Group - Pick every unit in RO_Group[RO] and do (Actions)
                  • Loop - Actions
                    • Set RO_Location[RO] = (Position of (Picked unit))
                    • Set RO_Location2[RO] = (RO_Location[RO] offset by RO_Speed[RO] towards ((Facing of (Picked unit)) + RO_Speed[RO]) degrees)
                    • Unit - Move (Picked unit) instantly to RO_Location2[RO]
                    • Animation - Change (Picked unit) flying height to ((Current flying height of (Picked unit)) + 1.00) at 0.00
                    • Custom script: call RemoveLocation(udg_RO_Location[udg_RO])
                    • Custom script: call RemoveLocation(udg_RO_Location2[udg_RO])
          • Else - Actions

 

[Urlock]

Because you should be using Jass.

 

[Adiktuz]

the recycling part should be this so that it won't even check the unused lower indexes... this is the recycle indexing method...

Set RO_Speed[RO] = RO_Speed[RO_Times]
Set RO_Distance[RO] = RO_Distance[RO_Times]
Set RO_Off[RO] = RO_Off[RO_Times]
Set RO = RO - 1
set RO_Times = RO_Times - 1

 

[BeTaGod-]

Mmmm still not working.. any other suggestion?

 

[Adiktuz]

btw, you don't need RO_Off now...

also change the If RO_Skip blahblah into If RO_times then remove the set RO_Times = 0 in the Do - actions

btw, which is not working?

 

[Maker]

The unit groups are null most likely, they don't exist.

• Custom script: if udg_RO_Group[udg_RO_Times] == null then
• Custom script: set udg_RO_Group[udg_RO_Times] = CreateGroup()
• Custom script: endif

 

[Adiktuz]

When using unit groups array in GUI make sure you set the amount to the max possible so they all get initialized

or better yet use Maker's actions