What The Crap

First person to find the stupid mistake i've made in these lines of code and tell me how to fix them gets a cookie. For some reason, it damages the caster, not the target.

Im not sure, but i think the problem is that you are using (Casting Unit) in the triggers. I reccommend using (Triggerimg Unit). Its easier, and faster, and aint lost after a wait. Also, wait commands is inaccurate, maybye you should run a timer and deal the damage when it expires.

Ill try make the spell just as you have your triggers, and then i try figure it out.
EDIT: Sry im a bit lazy right now, may you just give me a map with the spell, and ill see what i can do?

That trick can only be done once in GUI per trigger.
Therefore it's likely causing issues.

Ultra Ninja Edit:
I did some tests, and found out some really weird behaviours while using "fake local-global variables":
Test Map:
- Cast Banish on Mountain King
This means, you must not use a global variable and turn it into a local, like that:
Instead, you just
And turn every function that uses this unit into custom text.
zeroXD is correct too. In order to prevent funny problems with Waits, use as much "Triggering Unit" as you can.

How do you use timers for waits anyway? I've been hunting around the JASS tuts but it all looks like gibberish to me.

I think it should work the same way you used the
Instead, you start a timer with a Expiration-Time of
(or something)
and give the Mind Crush FX Trigger and event
(or something)
If you want to use JASS, make a new Trigger, put in the GUI-Function you want to JASS and turn that trigger into custom scripts. There you will see something like
The udg_spellcaster is the variable you used. Just replace that with a local variable, meaning, something without "udg_" at begin. Then paste it as a Custom Script.
This means, the must be without "udg_" too.

So in your case it would look like.

The problem with the global->local trick is that it can only be done with one variable, do it with multiple and they will all somehow share the same memory adress.

The problem with doing it that way UnMi is that i want this spell to be multi-instancable. So i need it to use a local timer, also as far as i'm aware local variables can only be used inside a trigger, not transferred from trigger to trigger, so i can't have the pause or whatever outside the trigger, as that leaves a possibility for the two global variables i use to transfer the target and caster from trigger to trigger will be overwritten mid-cast and cause all kinds of havoc.

Not with the local handle vars thing you were asking that other day.

Wha?

...
If you want to make a multi-instancable timer, you will have to use 100% JASS. It's not hard to implement though, all you would do is copy paste.
The problem, as zeroXD mentioned, is that Wait actually eats some delay, thus delaying the damaging point too.

I'm already using 100% JASS, i figured it'd be easier to just make the trigger in GUI, convert to JASS and then switch the bits i needed to be locals and so on into what they should be.

Uh, so where was your problem again? >>;

I don't know how to use a local timer to activate the actual damage function.

Ideally i'd prefer if it did a 'wait until target has buff Mind Crush' as that assures that the damage will only occur when the bolt actually hits as opposed to when it should hit.

Oh, now I see what you meant by
Would you please copy paste the whole trigger code of both triggers in JASS with [jass] tags? (Yes, because I am lazy.)