[Trigger] Failed in MUI Situation - Why?

[Gnuoy]

So this runs when players are defeated and moves their heroes to 'heaven' - more of a cosmetic thing. However seems "Rezing_Hero_Hev_Var" got rewrote half way through the trigger. I thought only 1 trigger could run at a time? Is the solution to just make more variables? thanks for any feedback.

• Heroes Hev Rez Red
  • Events
    • Time - Royal_Hev_Rez_Timer[1] expires
  • Conditions
  • Actions
    • Trigger - Turn off Red King Stay in <gen>
    • Trigger - Turn off AI Hero Dies Red <gen>
    • Set VariableSet Ran_Num_Rez_Var = (Random integer number between 1 and Rez_Pt_Count)
    • Set VariableSet Rezing_Hero_Hev_Var = Red_King
    • For each (Integer A) from 1 to 6, do (Actions)
      • Loop - Actions
        • Hero - Drop the item from slot (Integer A) of Rezing_Hero_Hev_Var.
    • Hero - Instantly revive Rezing_Hero_Hev_Var at Rez_Pt_Var[Ran_Num_Rez_Var], Show revival graphics
    • Unit - Move Rezing_Hero_Hev_Var instantly to Rez_Pt_Var[Ran_Num_Rez_Var]
    • Custom script: call RemoveLocation(udg_Rez_Pt_Var[udg_Ran_Num_Rez_Var])
    • Unit - Make Rezing_Hero_Hev_Var Invulnerable
    • Unit - Change ownership of Rezing_Hero_Hev_Var to Neutral Passive and Retain color
    • Unit - Change color of Rezing_Hero_Hev_Var to Snow
    • Custom script: call SetUnitPropWindow(udg_Rezing_Hero_Hev_Var, 0.00)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Ran_Num_Rez_Var Not equal to Rez_Pt_Count
      • Then - Actions
        • Set VariableSet Rez_Pt_Var[Ran_Num_Rez_Var] = Rez_Pt_Var[Rez_Pt_Count]
      • Else - Actions
    • Set VariableSet Rez_Pt_Count = (Rez_Pt_Count - 1)
    • Set VariableSet Ran_Num_Rez_Var = (Random integer number between 1 and Rez_Pt_Count)
    • Set VariableSet Rezing_Hero_Hev_Var = Red_Prince
    • For each (Integer A) from 1 to 6, do (Actions)
      • Loop - Actions
        • Hero - Drop the item from slot (Integer A) of Rezing_Hero_Hev_Var.
    • Hero - Instantly revive Rezing_Hero_Hev_Var at Rez_Pt_Var[Ran_Num_Rez_Var], Show revival graphics
    • Unit - Move Rezing_Hero_Hev_Var instantly to Rez_Pt_Var[Ran_Num_Rez_Var]
    • Custom script: call RemoveLocation(udg_Rez_Pt_Var[udg_Ran_Num_Rez_Var])
    • Unit - Make Rezing_Hero_Hev_Var Invulnerable
    • Unit - Change ownership of Rezing_Hero_Hev_Var to Neutral Passive and Retain color
    • Unit - Change color of Rezing_Hero_Hev_Var to Snow
    • Custom script: call SetUnitPropWindow(udg_Rezing_Hero_Hev_Var, 0.00)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Ran_Num_Rez_Var Not equal to Rez_Pt_Count
      • Then - Actions
        • Set VariableSet Rez_Pt_Var[Ran_Num_Rez_Var] = Rez_Pt_Var[Rez_Pt_Count]
      • Else - Actions
    • Set VariableSet Rez_Pt_Count = (Rez_Pt_Count - 1)

 

[Pyrogasm]

I thought only 1 trigger could run at a time?

Yes, but if something in trigger A causes trigger B to fire, B interrupts A and A will only resume once B has finished (or hit a wait). This also applies to a trigger C being fired because of something B did, and so on ad infinitum. If A sets a variable to x, but then B sets it to y... in the part of A that runs after B, the variable will be y not x.

Something in this trigger is causing another trigger to fire which is overwriting that variable. However, there's also this line that explicitly changes the variable halfway through:

• Set Rezing_Hero_Hev_Var = Red_Prince

 

[Gnuoy]

well what happened was two players got eliminated at nearly the exact same moment, which is rare but i still want it to work in those situations. it seems like one of the heroes got moved to 'heaven' but the ownership action didn't seem to fire. for the other 3 heroes it seemed to work fine.

 

[Wrda]

Simultaneity doesn't exist, it's always an order of operations.
I think this mostly can be solved by the technique of shadowing global variables.

• Actions
  • Custom script: local udg_SomeVariable
  • Set VariableSet SomeVariable = (Random integer number between 1 and 10)

This sort of thing. Now, I don't know which variables are been overwritten, so I would just do this logic to every variable you're using there, unless you find out the culprit.

 

[Uncle]

Simultaneity doesn't exist, it's always an order of operations.
I think this mostly can be solved by the technique of shadowing global variables.

• Actions
  • Custom script: local udg_SomeVariable
  • Set VariableSet SomeVariable = (Random integer number between 1 and 10)

This sort of thing. Now, I don't know which variables are been overwritten, so I would just do this logic to every variable you're using there, unless you find out the culprit.

Keep in mind that shadowed variables are inaccessible in Condition blocks so lines like these won't work with them:

• If - Conditions
  • Ran_Num_Rez_Var Not equal to Rez_Pt_Count