Sky Arrow[GUI][MUI][Leakless]

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

01:52, 8th Jun 2012
Pharaoh_: Fix the changes proposed by the reviews. 4 triggers for 1 spell is a bit too much.

 

[Tank-Commander]

Need recyling, also don't use custom values - it can cause confliction with other systems

 

[Ryro]

but i use custom value for dummy

Sorry my english very low

 

[maddeem]

[MUI] [Leakless] should be a given, no need to put that in the description.
I would suggest you loop it at 0.03 because movement still looks smooth at 0.03.
Turn off the loop when no units are in your arrow group.
You don't need

• Custom script: call RemoveLocation (udg_SA_Point[1])

throughout the loop. Just put it below the whole if statement (not under else)
You could have done the IF/Then for the damage much better...

 

[Matt]

Hi there. Great spell, I learned a lot from reading these triggers, but I have a few concerns.

First: I noticed your dummies can be raised/do decay. This has negative interaction in places, the first and foremost of which (aside from leaking corpses) is that the missiles persist for much longer than intended. You give them a 2-second lifespan, but because decaying units may still exist in groups, the "dead" missile continues flying until it completely decays. This led to a hilarious event wherein I accidentally scouted from one end of my map to the other with a missed missile. The fix is obviously to just change them to Can't Be Raised/Do Not Decay.

Second: Depending on the castable arc of a unit, your missile can end up being cast at funny/unintended angles. A simple fix for this appears to be to set the facing of the spawned dummy arrow at "facing point of ability being cast" rather than "facing of casting unit".

Third: Just a question, what does setting the Animation - Change (Last created unit) flying height to SA_MissileHeight at 1000000000.00 actually do? That's a huge number, and I haven't actually dealt with flying heights and stuff before, so it was odd to see one billion anythings.

Other than that, great spell, has performed flawlessly, and it's opened my eyes to yet another trick that can be used to make interesting spells. Thank you so much, great resource.

 

[eubz]

use this to check if your dummy is dead.

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • ((<your unit>) is dead) Equal to True
  • Then - Actions
    • -------- your actions --------
  • Else - Actions