[10926] Remove player-only limit for HandleAuraTransform where miscvalue is 0

Add error output for unhandled spells plus minor code cleanup

Signed-off-by: NoFantasy <nofantasy@nf.no>
This commit is contained in:
NoFantasy 2010-12-28 11:56:57 +01:00
parent 6a8f83379f
commit 1031f3b5d2
2 changed files with 11 additions and 14 deletions

View file

@ -3139,14 +3139,9 @@ void Aura::HandleAuraTransform(bool apply, bool Real)
// special case (spell specific functionality) // special case (spell specific functionality)
if (m_modifier.m_miscvalue == 0) if (m_modifier.m_miscvalue == 0)
{ {
// player applied only
if (target->GetTypeId() != TYPEID_PLAYER)
return;
switch (GetId()) switch (GetId())
{ {
// Orb of Deception case 16739: // Orb of Deception
case 16739:
{ {
uint32 orb_model = target->GetNativeDisplayId(); uint32 orb_model = target->GetNativeDisplayId();
switch(orb_model) switch(orb_model)
@ -3195,25 +3190,27 @@ void Aura::HandleAuraTransform(bool apply, bool Real)
} }
break; break;
} }
// Murloc costume case 42365: // Murloc costume
case 42365: target->SetDisplayId(21723); break; target->SetDisplayId(21723);
// Honor the Dead break;
case 65386: case 65386: // Honor the Dead
case 65495: case 65495:
{ {
switch(target->getGender()) switch(target->getGender())
{ {
case GENDER_MALE: case GENDER_MALE:
target->SetDisplayId(29203); // Chapman target->SetDisplayId(29203); // Chapman
break; break;
case GENDER_FEMALE: case GENDER_FEMALE:
case GENDER_NONE: case GENDER_NONE:
target->SetDisplayId(29204); // Catrina target->SetDisplayId(29204); // Catrina
break; break;
} }
break; break;
} }
default: break; default:
sLog.outError("Aura::HandleAuraTransform, spell %u does not have creature entry defined, need custom defined model.", GetId());
break;
} }
} }
else else

View file

@ -1,4 +1,4 @@
#ifndef __REVISION_NR_H__ #ifndef __REVISION_NR_H__
#define __REVISION_NR_H__ #define __REVISION_NR_H__
#define REVISION_NR "10925" #define REVISION_NR "10926"
#endif // __REVISION_NR_H__ #endif // __REVISION_NR_H__