[patch] Heartierce scripted effect

[Auntie Mangos]

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

The patch adds correct dummy effect for dagger http://www.wowhead.com/item=49982 and it's heroic version http://www.wowhead.com/item=50641

For which repository revision was the patch created?

10312

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

http://getmangos.eu/community/viewtopic.php?id=14584&highlight=Heartpierce

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

Me

http://paste2.org/p/937541

Description:

Original dbc data is wrong, there's other item's triggered spell in it. Spellfamily is paladin, that's a bit confusing too. Proc chance is 100% and according to wowhead this item has no internal cooldown.

Maybe I've forgotten something to mention, you can read original bugreport thread.

Tested ingame, works fine.

[Atømic]

Hello,

I have test your script, but whit compilation.. error : return false;

Maybe change by return SPELL_AURA_PROC_FAILED; whit it's okey' !

PS : Sorry my english

[dpedroia15]

Have you tested it ingame? I've written the same exact thing essentially, but it kept proccing Blessing of Light instead of the effect.

[fullburned]

@Atømic thx, i've missed that, first post updated with new link.

Have you tested it ingame? I've written the same exact thing essentially, but it kept proccing Blessing of Light instead of the effect.

Be sure to put the code under SPELLFAMILY_PALADIN case.

[dpedroia15]

That did it. I should have guessed because I had to put my Tiny Abomination in a Jar fix under the paladin spellfamily.

Why does blizzard make it tricky on us ?

[dpedroia15]

By the way it works perfectly fine^^