Spell help please.. if anyone can :)

Hiyas
Im trying to make a volcano, but it wont clean up the effects after itself - anyone spot my mistake? Im going blind staring at the damn triggers now..

I want to clean the special effects that are in VolcanoEffects[VolcanoValue], and remove the units in VolcanoGroup[VolcanoUnit].

One last thing (that i havent looked at very much myself, so i cant really ask for help) is the fact that the units "Volcano Barrage" pick one random target, and keep firering at it. I'd like them to fire randomly all the durn time, in the set area :)

Edit: Oops, i better add: - i left out some of the triggers, so dont look at the events/conditions. They fire allright, it's just the code in them that dont work like i want it to.

VolcanoEffects - Special Effect Array
VolcanoGroup - Unit-Group Array
VolcanoUnit - Integer
VolcanoValue - Integer

Action trigger:
Volcano action
Events
Conditions
Actions
Countdown Timer - Start VolcanoTimer as a One-shot timer that will expire in 10.50 seconds
Environment - Create a 2.00 second Permanent crater deformation at (Center of VolcanoAttack <gen>) with radius 200.00 and depth -300.00
Camera - Shake the camera for Player 1 (Red) with magnitude 5.00
Wait 2.00 seconds
For each (Integer A) from 1 to 20, do (Actions)
Loop - Actions
Special Effect - Create a special effect at (Random point in VolcanoEffects <gen>) using Doodads\LordaeronSummer\Props\SmokeSmudge\SmokeSmudge1.mdl
Set VolcanoEffects[VolcanoValue] = (Last created special effect)
Set VolcanoValue = (VolcanoValue + 1)
Camera - Stop swaying/shaking the camera for Player 1 (Red)
For each (Integer A) from 1 to 5, do (Actions)
Loop - Actions
Unit - Create 1 Volcano Barrage for Player 1 (Red) at (Center of VolcanoAttack <gen>) facing (Random point in (Playable map area))
Set VolcanoGroup[VolcanoUnit] = (Last created unit)
Set VolcanoUnit = (VolcanoUnit + 1)
Animation - Change (Last created unit)'s size to (0.00%, 0.00%, 0.00%) of its original size
Set VolcanoUnit = 0
For each (Integer B) from 1 to 100, do (Actions)
Loop - Actions
Unit - Order VolcanoGroup[VolcanoUnit] to Attack Ground (Random point in VolcanoAttack <gen>)
Set VolcanoUnit = (VolcanoUnit + 1)
Unit - Order VolcanoGroup[VolcanoUnit] to Attack Ground (Random point in VolcanoAttack <gen>)
Set VolcanoUnit = (VolcanoUnit + 1)
Unit - Order VolcanoGroup[VolcanoUnit] to Attack Ground (Random point in VolcanoAttack <gen>)
Set VolcanoUnit = (VolcanoUnit + 1)
Unit - Order VolcanoGroup[VolcanoUnit] to Attack Ground (Random point in VolcanoAttack <gen>)
Set VolcanoUnit = (VolcanoUnit + 1)
Unit - Order VolcanoGroup[VolcanoUnit] to Attack Ground (Random point in VolcanoAttack <gen>)
Set VolcanoUnit = 0

Cleanup Trigger:
Volcano cleanup
Events
Conditions
Actions
Game - Display to (All players) the text: (VolcanoValue: + (String(VolcanoValue)))
Game - Display to (All players) the text: (VolcanoUnit: + (String(VolcanoUnit)))
Environment - Create a 1.00 second Permanent crater deformation at VolcanoPoint with radius 200.00 and depth 300.00
For each (Integer VolcanoUnit) from 0 to VolcanoUnit, do (Actions)
Loop - Actions
Unit - Remove VolcanoGroup[VolcanoUnit] from the game
For each (Integer VolcanoValue) from 0 to VolcanoValue, do (Actions)
Loop - Actions
Game - Display to (All players) the text: Reached the Destroy...
Special Effect - Destroy VolcanoEffects[VolcanoValue]