[provocating]

Okay I finally got time to look at it again. I found how to fix it, at least for myself.

                  {
                                // 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.