Lightning Nova

[Moderator]

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

Lightning Nova | Reviewed by Maker | 30th May 2013

NEEDS FIX

You are not using K_Group destroy it Temp group is destroyed but no created again The death type of the dummies should be Can't raise, does not decay The dummies give too much visibiliy The units picked in 225 range function doesn't do anything Does it belong here? Currently pathing blockers and things like that interrupt the movement of the dummies You could use a variable for owner of triggering unit to avoid repeated function calls Remove the AND from the unit group pick Place the conditions directly under the If - Conditions The dummies do not need Invulnerability ability, they have Locust It would be better if the caster dealt the damage

[tr]

 

[deathismyfriend]

Plz show us the triggers.

I took a look at the trigger u have.
This can be made more efficient.
Anything u use twice or more should be stored into a variable for efficiency and speed.
Get rid of the and condition. only time u need to use the and condition is inside an ir condition.
You leak a location in the very beginning.
When u use ur unit group loop the TargetLoc is null so it will bug up.
I think thats all i just took a quick look at it.
This spell is also GUI not JASS

 

[Losam]

Nice spell.

 

[-SmaSHeR-]

Nice spell.

well thank you