[Crash] Improving a Spell

[Yugata]

So while I was testing my map, I got an error.

Error

What does this mean? Is there anything I can learn directly from this? What actually causes this error to happen?

I think I know the cause of the error though. This exact same error happened everytime I used this spell on a group of units.

Spell Tooltip

So basically this runs off everytime the hero attacks a unit, damaging a total of 6 times over 3 seconds. When the crash happens, I'm hitting maybe 20 units with this about 2-3 times.
I really like this spell as it fits the hero, so I was wondering if anyone could help me improve the triggering for it so that it won't crash anymore.
The spell runs everytime the hero's attack hits successfully while the spell is toggled on. It runs off of looking_for_help's DDS system.

Triggers

• Static Vortex
  • Events
    • Game - PDD_damageEventTrigger becomes Equal to 1.00
  • Conditions
    • (PDD_source has buff Static Vortex ) Equal to True
    • PDD_damageType Equal to PDD_PHYSICAL
  • Actions
    • Set Temp_P = (Position of PDD_target)
    • Set Static_Vortex_Loop_Max = (Static_Vortex_Loop_Max + 1)
    • Set Static_Vortex_C[Static_Vortex_Loop_Max] = PDD_source
    • Unit - Create 1 Razor (Static Vortex) for (Owner of PDD_source) at Temp_P facing Default building facing degrees
    • Set Static_Vortex_FX[Static_Vortex_Loop_Max] = (Last created unit)
    • Animation - Change Static_Vortex_FX[Static_Vortex_Loop_Max]'s animation speed to 33.33% of its original speed
    • Set Static_Vortex_Duration[Static_Vortex_Loop_Max] = 3.00
    • Set Static_Vortex_Timer[Static_Vortex_Loop_Max] = 0.50
    • Set Static_Vortex_DMG[Static_Vortex_Loop_Max] = ((87.50 x (Real((Level of Razor (Static Vortex) for Static_Vortex_C[Static_Vortex_Loop_Max])))) + (((Real((Agility of Static_Vortex_C[Static_Vortex_Loop_Max] (Include bonuses)))) x 0.10) + ((Real((Intelligence of Static_Vortex_C[Static_Vortex_Loop_Max] (Incl
    • Custom script: set bj_lastCreatedLightning = AddLightningEx("MFPB", true, GetLocationX(udg_Temp_P), GetLocationY(udg_Temp_P), GetLocationZ(udg_Temp_P) + 1000, GetLocationX(udg_Temp_P), GetLocationY(udg_Temp_P), GetLocationZ(udg_Temp_P))
    • Set Static_Vortex_Lightning[Static_Vortex_Loop_Max] = (Last created lightning effect)
    • Custom script: call RemoveLocation (udg_Temp_P)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Static_Vortex_Loop_Max Equal to 1
      • Then - Actions
        • Trigger - Turn on Static Vortex Loop <gen>
      • Else - Actions

• Static Vortex Loop
  • Events
    • Time - Every 0.10 seconds of game time
  • Conditions
  • Actions
    • For each (Integer Static_Vortex_Loop) from 1 to Static_Vortex_Loop_Max, do (Actions)
      • Loop - Actions
        • Set Static_Vortex_Duration[Static_Vortex_Loop] = (Static_Vortex_Duration[Static_Vortex_Loop] - 0.10)
        • Set Static_Vortex_Timer[Static_Vortex_Loop] = (Static_Vortex_Timer[Static_Vortex_Loop] - 0.10)
        • Lightning - Change color of (Last created lightning effect) to (1.00 1.00 1.00) with (Static_Vortex_Duration[Static_Vortex_Loop] / 3.00) alpha
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • Static_Vortex_Timer[Static_Vortex_Loop] Less than or equal to 0.00
          • Then - Actions
            • Set Temp_P = (Position of Static_Vortex_FX[Static_Vortex_Loop])
            • Set Temp_UGroup = (Units within 275.00 of Temp_P)
            • Unit Group - Pick every unit in Temp_UGroup and do (Actions)
              • Loop - Actions
                • Set Temp_PickedUnit = (Picked unit)
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • (Temp_PickedUnit is A structure) Not equal to True
                    • (Temp_PickedUnit is Magic Immune) Not equal to True
                    • (Temp_PickedUnit is alive) Equal to True
                    • (Temp_PickedUnit belongs to an enemy of (Owner of Static_Vortex_C[Static_Vortex_Loop])) Equal to True
                  • Then - Actions
                    • Unit - Cause Static_Vortex_C[Static_Vortex_Loop] to damage Temp_PickedUnit, dealing Static_Vortex_DMG[Static_Vortex_Loop] damage of attack type Spells and damage type Normal
                  • Else - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • Static_Vortex_Duration[Static_Vortex_Loop] Less than or equal to 0.00
              • Then - Actions
                • Unit - Kill Static_Vortex_FX[Static_Vortex_Loop]
                • Lightning - Destroy Static_Vortex_Lightning[Static_Vortex_Loop]
                • -------- DE-INDEX --------
                • Set Static_Vortex_C[Static_Vortex_Loop] = Static_Vortex_C[Static_Vortex_Loop_Max]
                • Set Static_Vortex_FX[Static_Vortex_Loop] = Static_Vortex_FX[Static_Vortex_Loop_Max]
                • Set Static_Vortex_Duration[Static_Vortex_Loop] = Static_Vortex_Duration[Static_Vortex_Loop_Max]
                • Set Static_Vortex_Timer[Static_Vortex_Loop] = Static_Vortex_Timer[Static_Vortex_Loop_Max]
                • Set Static_Vortex_DMG[Static_Vortex_Loop] = Static_Vortex_DMG[Static_Vortex_Loop_Max]
                • Set Static_Vortex_Lightning[Static_Vortex_Loop] = Static_Vortex_Lightning[Static_Vortex_Loop_Max]
                • Set Static_Vortex_Loop = (Static_Vortex_Loop - 1)
                • Set Static_Vortex_Loop_Max = (Static_Vortex_Loop_Max - 1)
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • Static_Vortex_Loop_Max Less than or equal to 0
                  • Then - Actions
                    • Trigger - Turn off Static Vortex Loop <gen>
                  • Else - Actions
              • Else - Actions
                • Set Static_Vortex_Timer[Static_Vortex_Loop] = 0.50
          • Else - Actions

 

[Maker]

You are referencing last created lightning in the looping trigger. If it is null, the game could crash. You want to reference the lightning at the correct index.

 

[Yugata]

Yay, someone found something wrong! I was worried that I'd have to readjust my original post to make more sense.