Mui conversion help

[WuHT]

After reading a hearty thread by Tom Jones.. i decided to check if this trigger is MUI

G Rhinok Attack
    Events
        Unit - A unit Is attacked
    Conditions
        ((Attacked unit) is A structure) Equal to False
        (Unit-type of (Attacking unit)) Equal to Rhinokosaur G
    Actions
        Wait 0.25 seconds
        Set Rhinokattack = (Position of (Attacking unit))
        Unit - Create 1 Dummy for (Owner of (Attacking unit)) at Rhinokattack facing Default building facing degrees
        Unit - Add G Impaler  to (Last created unit)
        Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
        Unit - Order (Last created unit) to Undead Crypt Lord - Impale (Attacked unit)

Rhinoksaur = unit
G Impaler = Impale custom ability
The whole point is to have a "lurker" like attack.

The wait part i forgot why i needed in the first place..maybe cosmetic but does impale's "air" time (which i set to a miniscule amount) mess up targetting ?
If i did need a wait...would it be better if "G Impaler" was a spell based on channel with a wait time.... and then when "G impaler" is cast i have another trigger that adds ANOTHER dummy to cast impale? <--dummy for a dummy might cause more problems ?

There was also a fat thread on assigning up to (500?) units a special value, and recalling those values to do stuff. Problem is, i'm using a lot of dummys, summons, regular spawns and implementing this "might" seem like a lot of work.

 

[elmstfreddie]

Attacking Unit doesn't exist after the wait

.. I think

 

[Trollvottel]

its much work but you should make your triggers MUI. its very useful check uareanoobs mui tutorial.

 

[Tinki3]

It looks to be MUI, though, have you tested it with alot of units?

You could always store "Attacking unit" into a local-global variable for use after the wait.

 

[WuHT]

lol dont be so vague.

are you saying to remove the wait or ... it should be fine ?

 

[Tinki3]

>are you saying to remove the wait or ... it should be fine ?

It should be fine.

Tested this trigger with 40 units:

Test
    Events
        Unit - A unit Is attacked
    Conditions
        (Unit-type of (Attacking unit)) Equal to Footman
    Actions
        Wait 0.25 seconds
        Set Temp_Point = (Position of (Attacked unit))
        Special Effect - Create a special effect at Temp_Point using Abilities\Spells\Human\ThunderClap\ThunderClapCaster.mdl
        Special Effect - Destroy (Last created special effect)
        Custom script:   call RemoveLocation(udg_Temp_Point)

No problems.