[fix][7515] Polymorph: Pig, Turtle, etc

[Auntie Mangos]

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

It was possible to polymorph more than one target at a time using different versions of the polymorph spell. Using any of the polymorph spells (Polymorph Rank 1, 2, 3, 4, Turtle, Pig, Serpent, Black Cat) should break the effect of the other ones.

For which repository revision was the patch created?

7503

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=3889 (a bit old, but I couldn't find any newer; however, the problem still exists in rev. 7503)

diff --git a/src/game/SpellMgr.cpp b/src/game/SpellMgr.cpp
index 1f6a746..9dc6deb 100644
--- a/src/game/SpellMgr.cpp
+++ b/src/game/SpellMgr.cpp
@@ -530,6 +530,7 @@ bool IsSingleTargetSpells(SpellEntry const *spellInfo1, SpellEntry const *spellI
    switch(spec1)
    {
        case SPELL_JUDGEMENT:
+         case SPELL_MAGE_POLYMORPH:
            if(GetSpellSpecific(spellInfo2->Id) == spec1)
                return true;
            break;

Tested it with 2 mages on 3 mobs, and it seems to be working properly.

[Sephiroth1983]

Will this 1 line generic fix, fix it for real at last? I never noticed the problem, always used turtle only, but this problem recurred at least 3 times for now - hope is the last.

[VladimirMangos]

In [7515]. Thank you

[DangeR]

Nice fix! Thanks a lot DonTomika.