Trigger help please!

[xericho]

Im doing the a spell which is kinda like zeus' wrath from Dota, it does damage all the heroes in the map, but the problem is it also damages the caster
tell me whats wrong with this trigger.

Unit Group - Pick every unit in (Units in (Playable map area) matching (((Picked unit) belongs to an enemy of (Owner of Caster)) Equal to True)) and do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
((Picked unit) is A Hero) Equal to True
Then - Actions
Unit - Cause Caster to damage (Picked unit), dealing (180.00 x (Real((Level of Crush for TC_Caster)))) damage of attack type Spells and damage type Normal

 

[FhelZone]

You must add in the conditions, Owner of picked unit is an ally of owner of triggering unit. Its a boolean comparison.

 

[Ashcat]

Here, lets simplify the trigger, and remove that leak while we're at it.

Global Hero Bomb
    Events
        Unit - A unit Starts the effect of an ability
    Conditions
        (Ability being cast) Equal to Animate Dead
    Actions
        Set Unit_Group_Variable = (Units in (Playable map area) matching ((((Matching unit) is A Hero) Equal to True) and ((((Matching unit) is alive) Equal to True) and (((Owner of (Casting unit)) is an enemy of (Owner of (Matching unit))) Equal to True))))
        Unit Group - Pick every unit in Unit_Group_Variable and do (Actions)
            Loop - Actions
                Unit - Cause (Casting unit) to damage (Picked unit), dealing 500.00 damage of attack type Spells and damage type Normal
        Custom script:   call DestroyGroup (udg_Unit_Group_Variable)

 

[FhelZone]

Well that should work, but does groups leak?