Issue with leveling an ability through GUI.

[BarzahdX]

I have a trigger, so that when you learn this ability, your kills spawn a dummy caster to give you a negative value cripple. It works for level one(I was so glad) but it doesn't increase in level, I have the trigger to set increase the level, but it doesn't work.. The abilities that are issuable through triggers, can those not be leveled up? Here are the codes for the first level that works.

Victory Rush Var 1
    Events
        Unit - A unit Learns a skill
    Conditions
        (Learned Hero Skill) Equal to Victory Rush 
        (Learned skill level) Equal to 1
    Actions
        Set Ferocious_Wengido_VR[1] = (Learning Hero)

Which chains of to:

Victory Rush Kill 1
    Events
        Unit - A unit Dies
    Conditions
        (Killing unit) Equal to Ferocious_Wengido_VR[1]
    Actions
        Set Caster_Position[3] = (Position of (Killing unit))
        Unit - Create 1 "Cripple"(Victory Rush) Dummy Caster for (Owner of Ferocious_Wengido_VR[1]) at Caster_Position[3] facing Caster_Position[3]
        Set VR_Dummy = (Last created unit)
        Unit - Add a 2.00 second Generic expiration timer to VR_Dummy
        Unit - Set level of Cripple for VR_Dummy to 1
        Unit - Order VR_Dummy to Undead Necromancer - Cripple Ferocious_Wengido_VR[1]
        Custom script:   call RemoveLocation (udg_Caster_Position[3])

That works exactly as intended, but when he levels up his ability--the Cripple doesn't increase.. This is what I have set. This is for level 2.

Victory Rush Var 2
    Events
        Unit - A unit Learns a skill
    Conditions
        (Learned Hero Skill) Equal to Victory Rush 
        (Learned skill level) Equal to 2
    Actions
        Set Ferocious_Wengido_VR[2] = (Learning Hero)

Which chains to:

Victory Rush Kill 2
    Events
        Unit - A unit Dies
    Conditions
        (Killing unit) Equal to Ferocious_Wengido_VR[2]
    Actions
        Set Caster_Position[3] = (Position of (Killing unit))
        Unit - Create 1 "Cripple"(Victory Rush) Dummy Caster for (Owner of Ferocious_Wengido_VR[2]) at Caster_Position[3] facing Caster_Position[3]
        Set VR_Dummy = (Last created unit)
        Unit - Add a 2.00 second Generic expiration timer to VR_Dummy
        Unit - Set level of Cripple for VR_Dummy to 2
        Unit - Order VR_Dummy to Undead Necromancer - Cripple Ferocious_Wengido_VR[2]
        Custom script:   call RemoveLocation (udg_Caster_Position[3])

Please help me guys--I know you have my answer.. You always do.

 

[Expelliarmus]

Victory Rush Var 1
You won't need the first trigger.

Victory Rush Kill 1
    Events
        Unit - A unit Dies
    Conditions
        (Killing unit) Equal to Ferocious_Wengido_VR[1]
    Actions
        Set Caster_Position[3] = (Position of (Killing unit))
        Unit - Create 1 "Cripple"(Victory Rush) Dummy Caster for (Owner of Ferocious_Wengido_VR[1]) at Caster_Position[3] facing Caster_Position[3]
        Set VR_Dummy = (Last created unit)
        Unit - Add a 2.00 second Generic expiration timer to VR_Dummy
        Unit - Set level of Cripple for VR_Dummy to 1
        Unit - Order VR_Dummy to Undead Necromancer - Cripple Ferocious_Wengido_VR[1]
        Custom script:   call RemoveLocation (udg_Caster_Position[3])

Replace Killing Unit Equal to ... to
Level of Victory for Killing Unit Greater 0

Set Caster_Position[3]
You won't need this. Replace the array with just normal point and remove the leak without the array.

As for the doesn't work, I think that when you level up, the hero is still set the first trigger as well as the second.

 

[BarzahdX]

Victory Rush
    Events
        Unit - A unit Dies
    Conditions
        (Unit-type of (Killing unit)) Equal to Ferocious Wendigo
        (Level of Victory Rush  for (Killing unit)) Greater than 0
    Actions
        Set Caster_Position[3] = (Position of (Killing unit))
        Unit - Create 1 "Cripple"(Victory Rush) Dummy Caster for (Owner of (Killing unit)) at Caster_Position[3] facing Caster_Position[3]
        Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        Unit - Set level of Cripple for (Last created unit) to (Level of Victory Rush  for (Killing unit))
        Unit - Order (Last created unit) to Undead Necromancer - Cripple (Killing unit)
        Custom script:   call RemoveLocation (udg_Caster_Position[3])

Good ideas, saves room, and looks cleaner, thank you for that.. Is this better?

About to test it now, although I doubt it will work. I didn't change the array in Caster_Position because I use multiple arrays for many triggers, if I got rid of the array--I'd have to edit a lot of triggers. It shouldn't actually change the effectiveness of the trigger, should it?

 

[BarzahdX]

Awesome--although I don't know why, it makes no logical sence--BUT, it works now. Thank you for your help.