Trigger makes my game crash?

[Krakenn99]

• Fire Bounce
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Fire Bounce [Q]
  • Actions
    • Set Caster_FireBounce = (Triggering unit)
    • Set temp_point1_FireBounce = (Position of Caster_FireBounce)
    • Unit - Create 1 Dummy Fire Bounce lvl1 for (Owner of Caster_FireBounce) at temp_point1_FireBounce facing Default building facing degrees
    • Set Dummy_FireBounce = (Last created unit)
    • Unit - Order Dummy_FireBounce to Attack Once (Target unit of ability being cast)
    • Unit - Remove Dummy_FireBounce from the game
    • Custom script: call RemoveLocation(udg_temp_point1_FireBounce)

First cast ... everthing is ok. But when I cast it again the game crashes. No errors no nothing just closes. Why?

 

[ILH]

Try not to remove the dummy, but add expiration timer instead.

 

[JaysProjex]

• Fire Bounce
  • Unit - Order Dummy_FireBounce to Attack Once (Target unit of ability being cast)

First cast ... everthing is ok. But when I cast it again the game crashes. No errors no nothing just closes. Why?

The Dummy_FireBounce doesn't always have a target

 

[Krakenn99]

The Dummy_FireBounce doesn't always have a target

But the ability is based of Storm Bolt so there is no way for it to have no target? And could that really make the game crash!?

Oh no it is from the map! It's not because of the spell

 

[RobertMKD]

Try deleting the custom script action, the trigger should work without it too

 

[Krakenn99]

Try deleting the custom script action, the trigger should work without it too

Custom script is for removing leaks and it is not from the spells. It is from the map itself :|

 

[LordDz]

Change the attacking dummy to have timed life instead. If you just want it to deal one attack, set it's attack animation to 0.0 and it's attack cooldown to 200. It'll attack once and then wait 200 seconds until it attacks again.

And you don't need temp_point1_FireBounce variables for your trigger, just rename them to tempPoint1, tempPoint2, etc and use them for all triggers

 

[Krakenn99]

Change the attacking dummy to have timed life instead. If you just want it to deal one attack, set it's attack animation to 0.0 and it's attack cooldown to 200. It'll attack once and then wait 200 seconds until it attacks again.

And you don't need temp_point1_FireBounce variables for your trigger, just rename them to tempPoint1, tempPoint2, etc and use them for all triggers

But if I cast miltipule spells with that variable won't it bug out?

 

[Cokemonkey11]

But if I cast miltipule spells with that variable won't it bug out?

In this case it won't bug out because this spell is instantaneous (warcraft 3 doesn't have parallel execution)

Regardless, the optimization is futile and worthless.

I don't see any actual error in your trigger, but try the timed life + attack cooldown thing.