[patch] Мобы и критические удары спеллами

[PSZ]

Мобы спеллами критовать не могут. Об этом даже выскакивает подсказка при загрузке клиента.

diff --git a/src/game/Unit.cpp b/src/game/Unit.cpp
index 3b36892..d8743ad 100644
--- a/src/game/Unit.cpp
+++ b/src/game/Unit.cpp
@@ -9375,6 +9375,10 @@ int32 Unit::SpellBaseDamageBonusTaken(SpellSchoolMask schoolMask)
 
 bool Unit::IsSpellCrit(Unit *pVictim, SpellEntry const *spellProto, SpellSchoolMask schoolMask, WeaponAttackType attackType)
 {
+    // mobs can't crit with spells at all
+    if (GetObjectGuid().IsCreature())
+        return false;
+
     // not critting spell
     if((spellProto->AttributesEx2 & SPELL_ATTR_EX2_CANT_CRIT))
         return false;

[Insider42]

А петов это не затронет случаем?

[wk23]

bool IsCreature() const { return GetHigh() == HIGHGUID_UNIT; }
bool IsPet() const { return GetHigh() == HIGHGUID_PET; }
bool IsVehicle() const { return GetHigh() == HIGHGUID_VEHICLE; }
bool IsCreatureOrPet() const { return IsCreature() || IsPet(); }

[PSZ]

Сообщение от Insider42

А петов это не затронет случаем?

Не должно. Поэтому и использовал GetObjectGuid().IsCreature(), а не обычное GetTypeId() == TYPEID_UNIT. GetTypeId() возвращает TYPEID_UNIT и для петов.

        static TypeID GetTypeId(HighGuid high)
        {
            switch(high)
            {
                case HIGHGUID_ITEM:         return TYPEID_ITEM;
                //case HIGHGUID_CONTAINER:    return TYPEID_CONTAINER; HIGHGUID_CONTAINER==HIGHGUID_ITEM currently
                case HIGHGUID_UNIT:         return TYPEID_UNIT;
                case HIGHGUID_PET:          return TYPEID_UNIT;
                case HIGHGUID_PLAYER:       return TYPEID_PLAYER;
                case HIGHGUID_GAMEOBJECT:   return TYPEID_GAMEOBJECT;
                case HIGHGUID_DYNAMICOBJECT:return TYPEID_DYNAMICOBJECT;
                case HIGHGUID_CORPSE:       return TYPEID_CORPSE;
                case HIGHGUID_MO_TRANSPORT: return TYPEID_GAMEOBJECT;
                case HIGHGUID_VEHICLE:      return TYPEID_UNIT;
                // unknown
                default:                    return TYPEID_OBJECT;
            }
        }