[Spell] Mana Barrier

[Azelen]

What bug does the patch fix? What features does the patch add?

Implement Lady Deathwhisper's Mana Barrier Spell http://www.wowhead.com/spell=70842/mana-barrier

For which repository revision was the patch created?

11799

Is there a thread in the bug report section or at lighthouse? If yes, please add a link to the thread.

no

Who has been writing this patch? Please include either forum user names or email addresses.

me

Patch:

http://paste2.org/p/1671139

I am not sure if its correct its my first try Especially not sure about part

triggerTarget->EnergizeBySpell(triggerTarget, 70842, 0-damage, POWER_MANA);

Note: tested and works.

[michalpolko]

You don't need "0" in code, just "-" ;-)

This implementation looks nice.

Of course somebody will come here and say that it should be done via triggering some spell that gives mana and steals health and it should be called with custom basepoints; I would do it this way ;D But didn't find such spell.

[Azelen]

Ok just wasnt sure how to work with unint32

Edit: I was getting compiling error like "cannot apply sign to value of unsigned type".

Well wowhead doesnt have any spell that it triggers, and not just this spell i checked some other spells i found in close to this code part and all of them had no any triggered spells linked just "Triggering spell every %s secs".

[Schmoozerd]

This actually seems (first look) the way it is that we have no information about any additional related spell here.

About the details of the implementation I cannot say anything.

[Azelen]

Well when i was writing it i took http://www.wowhead.com/spell=23493 as example

[michalpolko]

oh, so if the parameter is unsigned then You can't do it this way seems like SetPower is the only option here.