some small bugs

[Den]

Две не большие проблеммы:
1) Начинаем кастить открываем лут с моба - каст должен прерыватся.
2) Открываем лут с моба и начинаем кастить ( к примеру камень портала) - лут должен закрывается.
С пунктом №1 решение есть:

PHP код:

diff --git a/src/game/Player.cpp b/src/game/Player.cpp
index 9812578..1e893d1 100644
--- a/src/game/Player.cpp
+++ b/src/game/Player.cpp
@@ -7845,6 +7845,13 @@ void Player::SendLoot(ObjectGuid guid, LootType loot_type)
     if (uint64 lguid = GetLootGUID())
         m_session->DoLootRelease(lguid);
 
+    // prevent cast spell - player looted
+    if (loot_type == LOOT_CORPSE)
+    {      
+        if (IsNonMeleeSpellCasted(true))
+            InterruptNonMeleeSpells(true);
+    }        
+        
     Loot    *loot = 0;
     PermissionTypes permission = ALL_PERMISSION; 


[Den]

Заметил не так давно баг ( нетак давно потому, что некогда небыло и нет сервера ) на последних ~50 ревизиях:
Убиваем моба, делаем логаут, загружаемся, смотрим, труп моба который до логаута был IsFriendlyTo, стал IsHostileTo. Есть ещё такой момент , если персонаж будет в группе или рейде, то после логаута с мобом всё ок и лут можно брать. Откатился до 10123 ревизии, проблемма существует, но немного не так, после логаута моб IsFriendlyTo, но лута как бы нет. Видимо баг существует уже давно...

add
Ещё одна мелочь, игрок сделавший логаут и при этом находящийся в группе - не должен исключаться из неё.
в WorldSession.cpp void WorldSession::LogoutPlayer(bool Save)
вместо:

if(_player->GetGroup() && !_player->GetGroup()->isRaidGroup() && m_Socket)

поставить:

if(!_player->GetGroup() && !_player->GetGroup()->isRaidGroup() && m_Socket)

Конечно проблемма исчезает, но правильно ли ?

[Den]

Типы урона, такие как: утопание, усталость, лава, слизь - должны тикать не раз в секунду , а в две:

diff --git a/src/game/Player.cpp b/src/game/Player.cpp
index 90131f2..8ead314 100644
--- a/src/game/Player.cpp
+++ b/src/game/Player.cpp
@@ -1048,7 +1048,7 @@ void Player::HandleDrowning(uint32 time_diff)
             // Timer limit - need deal damage
             if (m_MirrorTimer[BREATH_TIMER] < 0)
             {
-                m_MirrorTimer[BREATH_TIMER]+= 1*IN_MILLISECONDS;
+                m_MirrorTimer[BREATH_TIMER]+= 2*IN_MILLISECONDS;
                 // Calculate and deal damage
                 // TODO: Check this formula
                 uint32 damage = GetMaxHealth() / 5 + urand(0, getLevel()-1);
@@ -1084,7 +1084,7 @@ void Player::HandleDrowning(uint32 time_diff)
             // Timer limit - need deal damage or teleport ghost to graveyard
             if (m_MirrorTimer[FATIGUE_TIMER] < 0)
             {
-                m_MirrorTimer[FATIGUE_TIMER]+= 1*IN_MILLISECONDS;
+                m_MirrorTimer[FATIGUE_TIMER]+= 2*IN_MILLISECONDS;
                 if (isAlive())                                            // Calculate and deal damage
                 {
                     uint32 damage = GetMaxHealth() / 5 + urand(0, getLevel()-1);
@@ -1117,7 +1117,7 @@ void Player::HandleDrowning(uint32 time_diff)
             m_MirrorTimer[FIRE_TIMER]-=time_diff;
             if (m_MirrorTimer[FIRE_TIMER] < 0)
             {
-                m_MirrorTimer[FIRE_TIMER]+= 1*IN_MILLISECONDS;
+                m_MirrorTimer[FIRE_TIMER]+= 2*IN_MILLISECONDS;
                 // Calculate and deal damage
                 // TODO: Check this formula
                 uint32 damage = urand(600, 700);

[evilstar]

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

add
Ещё одна мелочь, игрок сделавший логаут и при этом находящийся в группе - не должен исключаться из неё.
в WorldSession.cpp void WorldSession::LogoutPlayer(bool Save)
вместо:

if(_player->GetGroup() && !_player->GetGroup()->isRaidGroup() && m_Socket)

поставить:

if(!_player->GetGroup() && !_player->GetGroup()->isRaidGroup() && m_Socket)

Конечно проблемма исчезает, но правильно ли ?

На оффе до Катаклизма при логауте игрок покидал обычную группу (иногда и не покидал), но в рейде оставался. Сейчас эта ситуация немного меняется, сейчас даже при логауте игрок остается в группе.

[Den]

По поводу: Убиваем моба, делаем логаут, загружаемся, смотрим, труп моба который до логаута был IsFriendlyTo, стал IsHostileTo.
Если не ошибаюсь, то просматривать надо, этих два места ?
1) Player.cpp

PHP код:

bool Player::isAllowedToLoot(Creature* creature)
{
    // never tapped by any (mob solo kill)
    if (!creature->HasFlag(UNIT_DYNAMIC_FLAGS, UNIT_DYNFLAG_TAPPED))
        return false;

    if (Player* recipient = creature->GetLootRecipient())
    {
        if (recipient == this)
            return true;

        if (Group* otherGroup = recipient->GetGroup())
        {
            Group* thisGroup = GetGroup();
            if (!thisGroup)
                return false;

            return thisGroup == otherGroup;
        }
        return false;
    }
    else
        // prevent other players from looting if the recipient got disconnected
        return !creature->HasLootRecipient();
} 


2) Object.cpp

PHP код:

// Gamemasters should be always able to select units - remove not selectable flag
                else if (index == UNIT_FIELD_FLAGS && target->isGameMaster())
                {
                    *data << (m_uint32Values***91;index***93; & ~UNIT_FLAG_NOT_SELECTABLE);
                }
                // hide lootable animation for unallowed players
                else if (index == UNIT_DYNAMIC_FLAGS && GetTypeId() == TYPEID_UNIT)
                {
                    if (!target->isAllowedToLoot((Creature*)this))
                        *data << (m_uint32Values***91;index***93; & ~(UNIT_DYNFLAG_LOOTABLE | UNIT_DYNFLAG_TAPPED_BY_PLAYER));
                    else
                    {
                        // flag only for original loot recipent
                        if (target->GetObjectGuid() == ((Creature*)this)->GetLootRecipientGuid())
                            *data << m_uint32Values***91;index***93;;
                        else
                            *data << (m_uint32Values***91;index***93; & ~(UNIT_DYNFLAG_TAPPED | UNIT_DYNFLAG_TAPPED_BY_PLAYER));
                    }
                }
                else
                {
                    // send in current format (float as float, uint32 as uint32)
                    *data << m_uint32Values***91;index***93;;
                } 


[Den]

Некоторые проки от чантов на офе - не снимаются правой кнопкой мыши (у нас снимаются).
Проверил только: Берсерк, Мангуст, Рыцарь.
Что ж получается, нужно пихать конкретные спелы в void WorldSession::HandleCancelAuraOpcode( WorldPacket& recvPacket) ?

Выходит, что на офе , все проки аур от чантов, професии зачарование - не снимаются пкм ?

[Den]

Может быть это смешно, но после отмены логаута игрок не должен вставать.
У нас встаёт. В MiscHandler.cpp void WorldSession::HandleLogoutCancelOpcode( WorldPacket & /*recv_data*/ )
убераем ?
//! Stand Up
GetPlayer()->SetStandState(UNIT_STAND_STATE_STAND);
А тем временем, на офе есть баг у друида, с обликом птицы, после отмены логаута она подлетает , а потом снова садится, а у нас нету такого О черт, спалился ...