Some bugs in my spell trigger code

[SkyXyden]

Guys, I need some help in my spell trigger, I couldn't figured a correct way to code them properly >.>

Here are the three spell I had problems

Fire Cannon - Deals damage in a line, stunning units as it passes

Problems: The spells conflicts with another spell 'Inferno' (shown below), causing the spell to loop 30 times instead of 9.

AST Fire Cannon
    Events
        Unit - A unit Starts the effect of an ability
    Conditions
        (Ability being cast) Equal to AST 2 Fire Cannon (Cast Point)
    Actions
        Set TempSkillPoint[1] = (Position of (Triggering unit))
        Set TempSkillPoint[2] = (Target point of ability being cast)
        For each (Integer A) from 1 to 9, do (Actions)
            Loop - Actions
                Set TempSkillPoint[3] = (TempSkillPoint[1] offset by (100.00 + (200.00 x (Real((Integer A))))) towards (Angle from TempSkillPoint[1] to TempSkillPoint[2]) degrees)
                Unit - Create 1 Dummy Unit for (Owner of (Triggering unit)) at TempSkillPoint[3] facing Default building facing degrees
                Animation - Change (Last created unit)'s size to (400.00%, 400.00%, 400.00%) of its original size
                Unit - Add AST 2 Fire Cannon (Blast Point) to (Last created unit)
                Unit - Set level of AST 2 Fire Cannon (Blast Point) for (Last created unit) to (Level of AST 2 Fire Cannon (Cast Point) for (Triggering unit))
                Unit - Order (Last created unit) to Orc Tauren Chieftain - War Stomp
                Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
                Wait 0.10 seconds
                Custom script:   call RemoveLocation (udg_TempSkillPoint[3])
        Custom script:   call RemoveLocation (udg_TempSkillPoint[1])
        Custom script:   call RemoveLocation (udg_TempSkillPoint[2])

Inferno Spell Code

AST Inferno
    Events
        Unit - A unit Starts the effect of an ability
    Conditions
        (Ability being cast) Equal to AST 1 Inferno 
    Actions
        Set FireCrossPoint = (Target point of ability being cast)
        Special Effect - Create a special effect at FireCrossPoint using Abilities\Spells\Other\Awaken\Awaken.mdl
        Special Effect - Destroy (Last created special effect)
        For each (Integer A) from 1 to 30, do (Actions)
            Loop - Actions
                Set FireCrossUnits[1] = (FireCrossPoint offset by 300.00 towards (90.00 + ((Real((Integer A))) x 10.00)) degrees)
                Set FireCrossUnits[2] = (FireCrossPoint offset by 300.00 towards (180.00 + ((Real((Integer A))) x 10.00)) degrees)
                Set FireCrossUnits[3] = (FireCrossPoint offset by 300.00 towards (270.00 + ((Real((Integer A))) x 10.00)) degrees)
                Set FireCrossUnits[4] = (FireCrossPoint offset by 300.00 towards (0.00 + ((Real((Integer A))) x 10.00)) degrees)
                Unit - Create 1 Dummy Unit for (Owner of (Triggering unit)) at FireCrossUnits[1] facing FireCrossUnits[3]
                Unit - Add AST 1 Inferno (Blast Point) to (Last created unit)
                Unit - Set level of AST 1 Inferno (Blast Point) for (Last created unit) to (Level of AST 1 Inferno  for (Triggering unit))
                Unit - Order (Last created unit) to Neutral Pandaren Brewmaster - Breath Of Fire FireCrossUnits[3]
                Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
                -------- ---- --------
                Unit - Create 1 Dummy Unit for (Owner of (Triggering unit)) at FireCrossUnits[2] facing FireCrossUnits[4]
                Unit - Add AST 1 Inferno (Blast Point) to (Last created unit)
                Unit - Set level of AST 1 Inferno (Blast Point) for (Last created unit) to (Level of AST 1 Inferno  for (Triggering unit))
                Unit - Order (Last created unit) to Neutral Pandaren Brewmaster - Breath Of Fire FireCrossUnits[4]
                Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
                -------- ---- --------
                Unit - Create 1 Dummy Unit for (Owner of (Triggering unit)) at FireCrossUnits[3] facing FireCrossUnits[1]
                Unit - Add AST 1 Inferno (Blast Point) to (Last created unit)
                Unit - Set level of AST 1 Inferno (Blast Point) for (Last created unit) to (Level of AST 1 Inferno  for (Triggering unit))
                Unit - Order (Last created unit) to Neutral Pandaren Brewmaster - Breath Of Fire FireCrossUnits[1]
                Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
                -------- ---- --------
                Unit - Create 1 Dummy Unit for (Owner of (Triggering unit)) at FireCrossUnits[4] facing FireCrossUnits[2]
                Unit - Add AST 1 Inferno (Blast Point) to (Last created unit)
                Unit - Set level of AST 1 Inferno (Blast Point) for (Last created unit) to (Level of AST 1 Inferno  for (Triggering unit))
                Unit - Order (Last created unit) to Neutral Pandaren Brewmaster - Breath Of Fire FireCrossUnits[2]
                Unit - Add a 1.00 second Generic expiration timer to (Last created unit)
                Wait 0.01 seconds
                Custom script:   call RemoveLocation (udg_FireCrossUnits[1])
                Custom script:   call RemoveLocation (udg_FireCrossUnits[2])
                Custom script:   call RemoveLocation (udg_FireCrossUnits[3])
                Custom script:   call RemoveLocation (udg_FireCrossUnits[4])
        Custom script:   call RemoveLocation (udg_FireCrossPoint)

 

[Never_Quit]

the problem lies in that if the
object editor
ability - ability activate/turn on "string"

is the same then it will register as the same ability on the system level

 

[istar]

the order strings should have no effects here
for eg.

(Ability being cast) Equal to AST 1 Inferno

it checks for ability id, whether orderstring is the same have no effect
also, as far as the trigger goes, you used different order strings

idk, but it seems there is no conflict with the 2 spells
try use integer B instead....

edit: yea, probably you need use integer B
by setting the loop 1 to 9, you are actually setting 2 global variable to 1 and 9 respectively
you have waits in the loop and the global variable is reassigned to 30
either use local variable(no need worry, integers dont leak) or use integer loop B