Help with spell please

[rogchert]

• Blade of doom
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Blade of Doom
  • Actions
    • Set bladeofdoomtarget = (Target point of ability being cast)
    • Set Bladeofdoomcast = (Position of (Triggering unit))
    • Special Effect - Create a special effect at Bladeofdoomcast using Abilities\Spells\Human\FlakCannons\FlakTarget.mdl
    • Special Effect - Create a special effect attached to the chest of (Target unit of ability being cast) using Abilities\Weapons\Rifle\RifleImpact.mdl
    • Unit - Cause (Triggering unit) to damage (Target unit of ability being cast), dealing (150.00 + (100.00 x (Real((Level of Blade of Doom for (Triggering unit)))))) damage of attack type Spells and damage type Death
    • Unit - Move (Triggering unit) instantly to bladeofdoomtarget
    • Wait 0.15 seconds
    • Special Effect - Create a special effect at bladeofdoomtarget using Abilities\Spells\Human\FlakCannons\FlakTarget.mdl
    • Unit Group - Pick every unit in (Units within 500.00 of (Position of (Triggering unit))) and do (Actions)
      • Loop - Actions
        • Unit Group - Add (Picked unit) to Unitgroupblink
    • Unit Group - Add (Random unit from Unitgroupblink) to Unitgroupblink2
    • Set bladeofdoomtarget = (Position of (Random unit from Unitgroupblink2))
    • Special Effect - Create a special effect attached to the chest of (Random unit from Unitgroupblink2) using Abilities\Weapons\Rifle\RifleImpact.mdl
    • Unit - Cause (Triggering unit) to damage (Random unit from Unitgroupblink2), dealing ((0.80 x 150.00) + (100.00 x (Real((Level of Blade of Doom for (Triggering unit)))))) damage of attack type Spells and damage type Death
    • Unit - Move (Triggering unit) instantly to bladeofdoomtarget
    • Unit Group - Remove all units from Unitgroupblink2
    • Wait 0.15 seconds
    • Special Effect - Create a special effect at bladeofdoomtarget using Abilities\Spells\Human\FlakCannons\FlakTarget.mdl
    • Unit Group - Pick every unit in (Units within 500.00 of (Position of (Triggering unit))) and do (Actions)
      • Loop - Actions
        • Unit Group - Add (Picked unit) to Unitgroupblink
    • Unit Group - Add (Random unit from Unitgroupblink) to Unitgroupblink2
    • Set bladeofdoomtarget = (Position of (Random unit from Unitgroupblink2))
    • Special Effect - Create a special effect attached to the chest of (Random unit from Unitgroupblink2) using Abilities\Weapons\Rifle\RifleImpact.mdl
    • Unit - Cause (Triggering unit) to damage (Random unit from Unitgroupblink2), dealing ((0.50 x 150.00) + (100.00 x (Real((Level of Blade of Doom for (Triggering unit)))))) damage of attack type Spells and damage type Death
    • Unit - Move (Triggering unit) instantly to bladeofdoomtarget
    • Unit Group - Remove all units from Unitgroupblink2
    • Unit Group - Remove all units from Unitgroupblink
    • Wait 0.15 seconds
    • Unit - Move (Triggering unit) instantly to Bladeofdoomcast
    • Custom script: call RemoveLocation(udg_bladeofdoomtarget)
    • Custom script: call RemoveLocation(udg_Bladeofdoomcast)

I need help with this. This is supposed to be a blink strike which hits 3 different targets. But it's not working good.
When I try the spell in-game the spell sometimes targets the caster which is really annoying. If someone could help me with this.
Also if anyone knows a cool special effect for hitting the target or sees any other mistake in the trigger please mention it so i can fix it. also if someone knows a way to make the trigger shorter please tell. If this is a noob Question u should understand one thing. i am a noob with editor ( not working so long with it.)

sorry if there already was a thread about this, yet i couldn't find one. If there was you can delete this one.

 

[Hoernchen]

Search before posting. The spell you are trying to make is known as omnislash, and there is a bunch of tutorials on how to make on.
And of course your caster will damage himself...
Your trigger says:

Unit Group - Pick every unit in (Units within 500.00 of (Position of (Triggering unit))) and do (Actions)

This mean it will pick all units within 500 of caster. That includes flying, invincible, friendly, dead and of course the caster itself. You did not say Pick every unit matching (Matching unit is alive) and isnt a friend and is a ground unit.

Also, your spell leaks. Use custom script: call RemoveLocation and call DestroyGroup to clean your leaks. If you don't know what a leak is, read about it on the tutorials.