What is wrong with this simple trigger...

Spontaneous Combustion

Events

Unit - A unit Begins casting an ability

Conditions

(Ability being cast) Equal to (==) Spontaneous Combustion

Actions

Unit Group - Add (Target unit of ability being cast) to spontaneouscombustionunitgroup[(Player number of (Owner of (Casting unit)))]

Spontaneous Combustion Die

Events

Unit - A unit Dies

Conditions
Actions

Do Multiple ActionsFor each (Integer A) from 1 to 12, do (Actions)

Loop - Actions

Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions)

If - Conditions

((Dying unit) is in spontaneouscombustionunitgroup[(Integer A)]) Equal to (==) True

Then - Actions

Unit - Remove Spontaneous Combustion buff from (Dying unit)
Set dyingunitpoint = (Position of (Dying unit))
Set spontaneouscombustionunitrange = (Units within 250.00 of dyingunitpoint matching (((Matching unit) belongs to an enemy of (Player((Integer A)))) Equal to (==) True))
Special Effect - Create a special effect at dyingunitpoint using Abilities\Spells\Orc\AncestralSpirit\AncestralSpiritCaster.mdl
Special Effect - Destroy (Last created special effect)
Unit Group - Pick every unit in spontaneouscombustionunitrange and do (Actions)

Loop - Actions

Unit - Cause hero[(Integer A)] to damage (Picked unit), dealing (50.00 x (Real((Level of Spontaneous Combustion for hero[(Integer A)])))) damage of attack type Spells and damage type Normal

Point - Remove dyingunitpoint
Unit Group - Destroy unit group spontaneouscombustionunitrange

Else - Actions

Its suppose to:

Put unit in a unit group in an array number of the casting unit's
When a unit dies, it will see if the dying unit is in any of the groups
If it is, special effects and units within range takes damage
There...