Spell Problem, please help me

why doesnt this trigger works ? (the units won't take any damage, pretty odd)

E:
Unit - A unit Begins casting an ability

C:
(Ability being cast) Equal to Chock Wave
(Unit-type of (Casting unit)) Equal to Black Wizard [|c00A0A050Enlightened|r] w


A:
Set BlackWizardCaster2 = (Casting unit)

Environment - Create a 5.00 second Depression ripple deformation at (Position of BlackWizardCaster2) with starting radius 50.00, ending radius 900.00, and depth 200.00, using 5.00 second ripples spaced 94.44 apart

Unit Group - Pick every unit in (Units within 50.00 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 900.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 144.44 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 810.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 238.88 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 720.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 333.32 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 630.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 427.76 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 540.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 522.20 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 450.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 616.64 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 360.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 711.08 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 270.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 805.52 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 180.00))

Wait 0.50 seconds

Unit Group - Pick every unit in (Units within 900.00 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 90.00))

cmon people... isnt there any good spell maker theese days ?

Is the ability not triggering at all? If so, that's where you should start. Throw in a "Game, display to all players" As the first action, and display some text. If the text doesn't appear, then it's your event/condition that isn't being met.

If the effects are being done, but the units are suffering no "damage" there is only one place the problem could be: The casting unit is not correctly being set to the variable you are using. If you have indeed copied this straight from the editor, I don't see that as being possible.
One thing to try is to set a 'point' variable instead of a caster variable, and then output the X value and Y value of that point at the end of the trigger in a Display Text to see where the point is, and/or create a special effect at that point.

By the way, I don't think you are doing this trigger very efficiently. You realize that the units within 50 will take ALL the different damages listed below, right? That's one heck of a lot of damage (Though your map could be scaled way up).

What *I* would do: Unit group: Add unit group to unit group (Effected units). The unit group to add is 'all units within 900 of 'casting point'.
Then, as each ring of damage is dealt, remove those units from the effected group. This way not unit will be effected more then once. Just a thought.

i know it works because it makes the ripple at the right position.. but the units won't take any damage...

and i have checked that the owning player is equal to player 9 (gray)

HI!

Ok do this.

Before - Unit Group - Pick every unit in (Units within 711.08 of (Position of BlackWizardCaster2) matching ((Owner of (Picked unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 270.00))

After - Unit Group - Pick every unit in (Units within 711.08 of (Position of BlackWizardCaster2) matching ((Owner of ( matching unit)) Equal to Player 9 (Gray))) and do (Unit - Set life of (Picked unit) to ((Life of (Picked unit)) - 270.00))

Your matching the unit properties, so you need to change it, you only pick the unit when your doing the actions. NOT the condtion.

ahhhh thank u!!!

i owe u big time :D

thx for helping a noob in trouble :S

Yea, just dont say there are no good spell makers around here. I see plenty. You just have to be patient, and your not a nobb, i made that mistake once too. Its very common.

Gah, you answered this ALREADY? Garr... I did the same error in my triggers... I went back, looked again... And I was like "Wtf am I, a moron?". Anyway, good to see it solved Chemo ;).

Regards
Dead-Inside

Ummmm...OK. Just remember, you said that i was a bad triggerer. Dead-Inside. I wont let that down. :-D

Man when did I... EVER... Say you were bad with triggers? From what I recall it was the other way around...

You said that i wasnt the bomb at triggers.