is this trigger ok?

[rax_7]

team 1 area
    Events
        Unit - A unit enters team 1 shop <gen>
    Conditions
        Or - Any (Conditions) are true
            Conditions
                (Owner of (Triggering unit)) Equal to Player 2 (Blue)
                (Owner of (Triggering unit)) Equal to Player 3 (Teal)
                (Owner of (Triggering unit)) Equal to Player 5 (Yellow)
                (Owner of (Triggering unit)) Equal to Player 6 (Orange)
                (Owner of (Triggering unit)) Equal to Player 7 (Green)
                (Owner of (Triggering unit)) Equal to Player 9 (Gray)
                (Owner of (Triggering unit)) Equal to Player 10 (Light Blue)
                (Owner of (Triggering unit)) Equal to Player 11 (Dark Green)
                (Owner of (Triggering unit)) Equal to Player 12 (Brown)
    Actions
        Unit - Move (Triggering unit) instantly to (Random point in team 1 area <gen>)
        Special Effect - Create a special effect at (Position of (Entering unit)) using Abilities\Spells\NightElf\Blink\BlinkTarget.mdl

its like any enemy units that enters that area will instantly move away.
the "Or" in that condition keeps bugging me. do i need to put another condition above that one?

 

[Emu.Man00]

Looks fine to me. I never used or.. found it useless, considering from my experiances having all of them there without any 'or' would achieve the same effect

 

[worldofDeath]

Got some leaks might want to correct them..


(Line: 16 ) (Word: 8 ) Position Leak
Unit - Move (Triggering unit) instantly to (Random point in team 1 area <gen>)
Unit - Move (Triggering unit) instantly to ^Leak
(Suggested fix) Set L = (Random point in team 1 area <gen>)
(Suggested fix) Custom Script: Call RemoveLocation(udg_L)

(Line: 17 ) (Word: 4 ) Special Effect Leak
Special Effect - Create a special effect at (Position of (Entering unit)) using Abilities\Spells\NightElf\Blink\BlinkTarget.mdl
Special Effect - ^Leak
(Suggested fix) Special Effect - Destroy (Last created special effect)

 

[Choppa]

And this might be easier for your action to check all the players...

Actions
    Player Group - Pick every player in (All players controlled by a User player) and do (Actions)
        Loop - Actions
            If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                If - Conditions
                    Or - Any (Conditions) are true
                        Conditions
                            (Owner of (Triggering unit)) Equal to (Picked player)
                            (Owner of (Triggering unit)) Not equal to Player 1 (Red)
                Then - Actions
                    YOUR ACTIONS
                Else - Actions

 

[AceHart]

> having all of them there without any 'or' would achieve the same effect

???
That would mean the unit is owned by all those players at once. Can't work...