GetSpellTargetUnit() malfunction in trigger chains

[DracoL1ch]

We have common trigger for SPELL_EFFECT, which looks up for any spell and related events.

if GetSpellAbilityId()=='AbiA' then

endif
call BJDebugMsg("Spell casted, caster = "+GetUnitName(GetTriggerUnit())+"; target = "+GetUnitName(GetSpellTargetUnit()))

Lets say we use custom spell A. On cast, it created a dummy who orders spell B onto the target.

call IssueTargetOrder(dummy,"slow",GetSpellTargetUnit())

Guess what will we see in the end?

I didn't run THIS exact test, but recovering it from my tests on doto:

Spell casted, caster = Dummy, target = MyTarget
Spell casted, caster = MyCaster, target =

GetSpellTargetUnit lose it's value once other spell event fires. Unlike (at least I didn't notice any other natives to do so) any other, who keep it's value until the trigger's end, like cached.

That's definitely a bug?

 

[IcemanBo]

That's good to know.
There are some working correctly like GetTriggerUnit(), and some can be overwritten by other triggers like a global variable, and some can't be used after wait.
It seems that this one is one of these that dont work correctly-- here is also an almost filled list with infos about responses. http://www.hiveworkshop.com/threads/event-response-myths.253665/#post2543973

 

[Kaijyuu]

The only event response I trust to be local is in fact GetTriggerUnit(). If I need to use any of the others after a wait I'll preserve them in my own local variable.

 

[WaterKnight]

GetTriggerUnit() is no different. In casting events it will show the same behavior as GetSpellAbilityUnit(). The event is the culprit.