1- If you use "Position of (anything)" you leak, just like you're doing in the Group Declaration
2- You should create the dummy at the TargetPoint, not a Triggering Unit's Position
3- You have to destroy TargetPoint at the end of the trigger, else you leak.
4- You have to configure the dummy to make it cast instantly:
Art - Animation cast backswing: 0.00
Art - Animation cast point: 0.00
Stats - Mana Initial amount: 9999
Stats - Mana Maximum: 9999
Stats - Mana Regeneration: 9999
Movement - Type: Flying
5- You have to configure teh Mass Inner Fire (Dummy) to make it instant-castable
Art - Animation names: --- (none)
Stats - Cast Range: 9999
Stats - Casting time: 0.00
Stats - Cooldown: 0.00
Stats - Mana Cost: 0
Stats - Targets Allowed: Air, Ground.
If you don't like 4 and 5, you have to :
6- Create the dummy INSIDE the loop and add Expiration timer INSIDE the loop, and add the Mass Inner Fire (Dummy) ability INSIDE the loop.
Right now you're not using the "Point" inside the loop for anything.
7- Learn basic jass. This is a peace of cake with locals
Create a trigger with the Event "A unit starts the effect of an ability" and the condition "Ability Being Cast equal to Mass Inner Fire" and in the actions create several "Do Nothing" actions. Then turn that trigger into Custom Script. There you'll see your "Do Nothing" Actions, replace them with this:
JASS:
// To see the ID of elements in the Object Editor press CTRL+D.
local unit u = GetTriggerUnit()
local real x = GetSpellTargetX()
local real y = GetSpellTargetY()
local unit dummy = CreateUnit(GetTriggerPlayer(), 'xxxx', x, y, 0) // Replace 'xxxx' with the Id of your Dummy unit
local unit fog
call UnitApplyTimedLife(dummy, 'BTLF', 1)
call GroupEnumUnitsInRange(bj_lastCreatedGroup, 500, 0, 0 null)
loop
set fog = FirstOfGroup(bj_lastCreatedGroup)
exitwhen fog == null
if IsUnitEnemy(fog, GetTriggerPlayer()) == true then
call IssueTargetOrderById(dummy, 'zzzz' , fog) // Replace 'zzzz' with the Id of your Mass Inner Fire (Dummy) Id
endif
call GroupRemoveUnit(bj_lastCreatedGroup, fog)
endloop
set u = null
set dummy = null
That JASS script is an optimized form (easier, faster, lighter, smoother) of the GUI trigger you're trying to create:
-
Melee Initialization
-
Events
-
Unit - A unit Starts the effect of an ability
-
Conditions
-
(Ability being cast) Equal to Animate Dead
-
Actions
-
Set tempPoint1 = (Target point of ability being cast)
-
Set tempPlayer = (Triggering player)
-
Set tempGroup = (Units within 500.00 of tempPoint1 matching (((Matching unit) belongs to an enemy of tempPlayer) Equal to False))
-
Unit Group - Pick every unit in tempGroup and do (Actions)
-
Loop - Actions
-
Set tempUnit = (Picked unit)
-
Set tempPoint2 = (Position of tempUnit)
-
Unit - Create 1 Footman for tempPlayer at tempPoint2 facing Default building facing degrees
-
Unit - Add a 0.50 second Generic expiration timer to (Last created unit)
-
Unit - Order (Last created unit) to Human Priest - Inner Fire tempUnit
-
Custom script: call RemoveLocation(udg_tempPoint2)
-
Custom script: call RemoveLocation(udg_tempPoint1)
-
Custom script: call DestroyGroup(udg_tempGroup)
Wait 1.00 seconds
