[Spell] Dummy unit is sometimes not removed.

[Dinodin]

I dunno why but this simple passive ability named "Double Attack" sometimes doesn't remove the dummy unit and I can't see the reason for it...

• Double Attack
  • Events
    • Unit - A unit Is attacked
  • Conditions
  • Actions
    • Set VariableSet Caster_Loc = (Position of (Triggering unit))
    • Set VariableSet DA_Attacker = (Attacking unit)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Random real number between 1.00 and 100.00) Less than or equal to (10.00 x (Real((Level of Double Attack for DA_Attacker))))
      • Then - Actions
        • Unit - Create 1 Dummy (Double Attack) for (Owner of DA_Attacker) at Caster_Loc facing Caster_Loc
        • Unit - Add a 3.00 second Generic expiration timer to (Last created unit)
        • Unit - Order (Last created unit) to Undead Necromancer - Unholy Frenzy DA_Attacker
        • Trigger - Turn off (This trigger)
        • Wait 1.00 seconds
        • Trigger - Turn on (This trigger)
      • Else - Actions
    • Custom script: call RemoveLocation(udg_Caster_Loc)

 

[Duckfarter]

The event Unit - A unit Is attacked occurs exactly then, when a unit is attacked, not after a successful attack. Are you sure youre not mistaking the dummy not being removed with a new dummy being created exactly when its supposed to according to the event?

 

[A]mun]

Apart from what Duckfarter said, you are creating Caster_Loc everytime there is an attack, needlessly.
Add a condition, checking if the attacker even has double attack.

Also this is clear case for a reusable dummy caster, no need to create and remove it all the time.