Everyone seems to want to be fancy and make macros that do something on Tuesday, but another thing on a Wednesday following a Thursday. Yeah, you can figure out super fancy reasons to add more to stuff, but for normal people, start off at the beginning.
Does the target option actually target the correct person?
Does the spell actually go off?
Do a /target [options] and see who get targeted. Once that is working, add in more.
Do a /cast Spell (no options) and see if it goes off correctly.
Shouldn't be much more after a target and spell.
I don't like to put 32 options on one line, I like seperate lines so even people that don't understand what is going on, will still see what is happening.

/cast [option 1] Spell 1
/cast [option 2] Spell 2
/cast [no options] Spell 3
That way, if spell 2 isn't going off, then you know exactly where to look.
One simple thing to remember, put your most limiting condition first. If you don't, it might get one first and the others won't have a chance to be done. For example:
/cast [mod:ctrl] Spell 1
/cast [mod:ctrl,alt] Spell 2
Guess what happens when you press CTRL and ALT and push the key. It goes off with spell 1 everytime. It does the first condition that is true.


I just use the target, help and focus-target-target options for my healing. It lets me heal anyone (not in group like BGs) or emergency someone in my group but mostly (2nd part) it just heals whoever has aggro. No fancy button for each player, no fancy worry about whoever has aggro, just heal whoever the mob is beating on.

Hope that helps.