[10504][fix bug] урон петом по игроку на нейтральной територии

[Chipleo]

@@ -533,7 +533,7 @@ uint32 Unit::DealDamage(Unit *pVictim, uint32 damage, CleanDamage const* cleanDa
 
     //You don't lose health from damage taken from another player while in a sanctuary
     //You still see it in the combat log though
-    if(pVictim != this && GetTypeId() == TYPEID_PLAYER && pVictim->GetTypeId() == TYPEID_PLAYER)
+    if(pVictim != this && GetCharmerOrOwnerPlayerOrPlayerItself() && pVictim->GetCharmerOrOwnerPlayerOrPlayerItself()) //kia
     {
         const AreaTableEntry *area = GetAreaEntryByAreaID(pVictim->GetAreaId());
         if(area && area->flags & AREA_FLAG_SANCTUARY)       //sanctuary

[SeT]

Вносить вместе с комментарием?
И в мангосе, в этой функции нет такого кода.

[Chipleo]

uint32 Unit::DealDamage(Unit *pVictim, uint32 damage, CleanDamage const* cleanDamage, DamageEffectType damagetype, SpellSchoolMask damageSchoolMask, SpellEntry const *spellProto, bool durabilityLoss)
{
    if (!pVictim->isAlive() || pVictim->IsTaxiFlying() || pVictim->GetTypeId() == TYPEID_UNIT && ((Creature*)pVictim)->IsInEvadeMode())
        return 0;

    //You don't lose health from damage taken from another player while in a sanctuary
    //You still see it in the combat log though
    if(pVictim != this && GetTypeId() == TYPEID_PLAYER && pVictim->GetTypeId() == TYPEID_PLAYER)
    {
        const AreaTableEntry *area = GetAreaEntryByAreaID(pVictim->GetAreaId());
        if(area && area->flags & AREA_FLAG_SANCTUARY)       //sanctuary
            return 0;
    }

- это оригинальный код. Мой патч взят из модифицированого кода с добавлнением логов боя и т.д., суть бага остается неизменной: например в Шатрате можно натравить пета на вражеского игрока и игрок не может ничего с петом сделать. А комментарий не нужен, я оставляю его чтобы знать что мною изменено и не соответствует основному коду.

[Vladimir]

GetCharmerOrOwnerPlayerOrPlayerItself ущет указатель по guid который для проверки совершенно не нужен. Там есть специально функция IsCharmerOrOwnerPlayerOrPlayerItself которая не ищет указатель и ограничивается тестом guid-ов.

Добавил в [10504] с этим изменением во всех местах где эта проверка делается (3). Спасибо.

В mangos-0.12 перенесу при следующем моем проходе по переносу свежих коммитов из master.