00:12, 4th Nov 2012
Magtheridon96: Approved.
Well done.

edit
Re-evaluated to 4/5 because the code is really clean and readable

You did not explain how it does damage .

You should put in the tooltip the damage mechanism of this spell...

Also, I'd prefer a timer event than a periodic one... but nothing big really I think... unless the user wants the interval to be changeable in-game...

btw, since you've gone through putting comments, I think the first AoE comment should be changed to Pick Tree AOE instead of just AoE. I know that the variable after that is self-explanatory but since you've already done the part of putting comments, why not make it more direct?

same thing for the special effects comment, If I'll just read it, I would think that it is the special effect for the field itself...

Also, I think it's better to make the AoE, changeable per level and as such, make the scale of the shield unit changeable per level too... doing this for the height and speed is optional...

Instead of this:
set bj_lastCreatedEffect = AddSpecialEffectTarget(udg_FF_SFX, udg_FF_NearbyUnits[udg_FF_Index], udg_FF_Attachment)
Custom script: call DestroyEffect(bj_lastCreatedEffect)
you could use this:
call DestroyEffect(AddSpecialEffectTarget(udg_FF_SFX, udg_FF_NearbyUnits[udg_FF_Index], udg_FF_Attachment))

Instead of creating so many tree remover,just create a peasant at the start of the game,hide it,cache it,add locust to it.

Skip remaining actions
You can remove this one because it just skips nothing

You could add the unit type of the field to the dummy remover

Move the removelocation loc2 inside the pick unit block,cause you are leaking ( number of units in that group - 1) positions every 0.03 seconds.
Unit groups are just like integer loops,as you can see,when you put this integer or variable:
set count = count + 1
The group is picking the units,then applying actions to it.This is the same to integers thats why you leak locations.
In short,unit groups are loops

Dont use unit in the condition of the dummy remover,use unittype instead

You could also make this a channeling spell


You should read this,then this spell will be approved

I agree with radamantus except, I'd prefer this the current way (non-channeling)... but adding a method to do it in a channeling way is more than welcome... though it might be a bit hard since in your current set-up, a single unit can have multiple instances of this spell running...

Radamantus said:

Instead of this:
set bj_lastCreatedEffect = AddSpecialEffectTarget(udg_FF_SFX, udg_FF_NearbyUnits[udg_FF_Index], udg_FF_Attachment)
Custom script: call DestroyEffect(bj_lastCreatedEffect)
you could use this:
call DestroyEffect(AddSpecialEffectTarget(udg_FF_SFX, udg_FF_NearbyUnits[udg_FF_Index], udg_FF_Attachment))

Click to expand...

LOOK

theoretically, those two lines are really just equal to the other one liner... can you show the actual trigger?

:V
Why?

Oh yeah,didnt notice that one.Why doomhammer?

Radamantus said:

Instead of creating so many tree remover,just create a peasant at the start of the game,hide it,cache it,add locust to it.

Click to expand...

Then What???

Can you make the duration configurable from the initialization trigger?

Also, (Dying unit) -> (Triggering unit)

But when I put the Duration on the Config Trig.. The Spell only works Once.

._.

Config:
Set Spell_Duration[1] = 8.00
Set Spell_Duration[2] = 10.00
Set Spell_Duration[3] = 12.00

Cast:
Set FF_Duration[FF_Index] = Spell_Duration[FF_Level]

Wow! It blinded me!
5/5

Told you the sfx is working

I love you Rada Rada as A Friend

I ain't Gay You Know.

harvestCustom script: set udg_FF_TreeRemover = CreateUnit(Player(15), 'hpea', 0, 0, 0)
need to see this command completely ....
this action does not appear in the file

^ Please don't spam. Reporting.

ohh sorry

I want to cry!! T_T