Okay I finally got time to look at it again. I found how to fix it, at least for myself.
Code:
{
// don't cast this on necro's, their health to mana
// spell eats up the rune spell and it just keeps
// getting recast over and over
}
else
{
SpellOnTarget(thespell, g->members[i]);
}
Basically just removing the first section and letting it cast. At least on my server this is working, otherwise it is getting caught in a loop. I am sure there is a more elegant solution but this will work for me, for now.