[patch][8483] Glyph of the Penguin (58310) fix

[bobaz]

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

It fixes the effect of Glyph of the Penguin.

For which repository revision was the patch created?

8480+

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

Me - Dr Bobaz

Patch:

@@ -3026,12 +3026,17 @@ void Aura::HandleAuraTransform(bool apply, bool Real)
                m_target->SetDisplayId(16358);
                sLog.outError("Auras: unknown creature id = %d (only need its modelid) Form Spell Aura Transform in Spell ID = %d", m_modifier.m_miscvalue, GetId());
            }
            else
            {
-                // Will use the default model here
-                m_target->SetDisplayId(ci->DisplayID_A[0]);
+				Unit* caster = GetCaster();
+				if((caster) && ((caster->HasAura(52648)) && (GetSpellProto()->SpellIconID == 82)))
+					// Glyph of the Penguin
+					m_target->SetDisplayId(24698);
+				else
+                    // Will use the default model here
+					m_target->SetDisplayId(ci->DisplayID_A[0]);

                // Dragonmaw Illusion (set mount model also)
                if(GetId()==42016 && m_target->GetMountID() && !m_target->GetAurasByType(SPELL_AURA_MOD_INCREASE_FLIGHT_SPEED).empty())
                    m_target->SetUInt32Value(UNIT_FIELD_MOUNTDISPLAYID,16314);
            }

EDIT: I don't understand why highlight changes [0] to symbols, but it in both cases should be "m_target->SetDisplayId(ci->DisplayID_A[0]);" at the line.

[yavi]

Tested, works.

[morderek]

working!!

thanks!

[VladimirMangos]

1) you check for expected polumorph overwrite touch to many spells (need check family also)

2) I think you select wrong model. Exist polymorph spell 59634 that have in misc creature id 31128 with model 26452.

With this fixes in [8483]. Thank you