Why target unit of ability being cast not works properly in here? I have already solved, just wanted to share

[JFAMAP]

This is not working (not passing Then)

• MagnataurQ Copy
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Reckless Hit //
  • Actions
    • Unit - Create 1 DUMMY [JFA] for (Owner of (Casting unit)) at (Position of (Casting unit)) facing Default building facing degrees
    • Unit - Add Fireball // master to (Last created unit)
    • Unit - Set level of Fireball // master for (Last created unit) to 23
    • Unit - Order (Last created unit) to Neutral - Firebolt (Target unit of ability being cast)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Target unit of ability being cast) is A Hero) Equal to True
      • Then - Actions
        • Game - Display to (All players) for 1.00 seconds the text: YES
      • Else - Actions
        • Game - Display to (All players) for 1.00 seconds the text: NO

But this does.

• MagnataurQ Copy
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Reckless Hit //
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Target unit of ability being cast) is A Hero) Equal to True
      • Then - Actions
        • Game - Display to (All players) for 1.00 seconds the text: YES
      • Else - Actions
        • Game - Display to (All players) for 1.00 seconds the text: NO
    • Unit - Create 1 DUMMY [JFA] for (Owner of (Casting unit)) at (Position of (Casting unit)) facing Default building facing degrees
    • Unit - Add Fireball // master to (Last created unit)
    • Unit - Set level of Fireball // master for (Last created unit) to 23
    • Unit - Order (Last created unit) to Neutral - Firebolt (Target unit of ability being cast)

What is the mystery behind it? It was bug for the previous versions or am i missing something imporant in here?

 

[Tristronic]

Not sure, does the fireball find the target? Either way if you set the target to a variable at the top then you can just use the variable

Edit:
I would be somewhat surprised if this caused the issue, I should hope it doesn't get interpreted as a comment but I recommend not naming anything "Fireball // master"
or more specifically, never using "//" in a name.
For the sake of ruling it out I would change that name, maybe "Fireball - master" instead?

When the second trigger fires, does the whole trigger work or just the debug msg? If the second trigger also fails to cast the fireball, maybe that's a hint toward the "don't use comment tags (//) in names" theory ?

 

[Nichilus]

The (Target unit of ability being cast) is a global variable, meaning it can get overwritten.
This is what happens:

1. Unit casts an ability
2. Ability is Reckless Hit
   1. At this point, the (Target unit of ability being cast) refers to the target of Reckless Hit
3. You create a dummy, give it fireball, order it to cast the fireball on some target
4. Since the dummy is casting a spell, it fires a new "Unit casts an ability" event
   1. At this point, the (Target unit of ability being cast) refers to the target of the Fireball spell
5. After any trigger that satisfies the event has been finished, the (Target unit of ability being cast) is nulled
6. Now you get back to your trigger. The If/then/else does not work because there is no longer any (Target unit of ability being cast)

 

[Tristronic]

Would an event filter prevent this?
Edit: I mean if you weren't using GUI

 

[Nichilus]

I am not sure, I have no experience with event filters. If it works with those filters, I imagine you would need to use them in all your triggered spells.
But this is a very specific use-case as I believe this only works with dummies - since a properly setup dummy has no turn rate, no movement and has 0.00 cast point, it can "inject" its own cast event into the currently running cast event trigger. But setting for example cast point to 0.001 will prevent such behavior.

 

[JFAMAP]

Thank you both of you for your comments. Is it as it supposed to be or bug?

 

[Nichilus]

Is it as it supposed to be or bug?

That's hard to say, but it's not the only case where triggers behave this way.
Actually, I think this behavior is in many cases beneficial - for example damage detection systems make use of this behavior.
On your side to prevent this issue, all you have to do is store the unit in your own variable, so it's easy fix/prevention on your side.