[11446] resurrect player - if it not in raid group

[Den]

Название темы
Суть проблемы:
Если труп персонажа в рейдовом подземелье, а рейдовая группа расформирована, то воскресится нельзя.
Древний баг, если не изменяет память , то припоминаю ещё с версии 2.4.3
Как далжно работать:
Призраком заходим на портал и игрок воскрешается, только если без группы или в ней, кроме рейдовой группы.

Работает, но есть какие-то сомнения... может быть, есть ошибки ?

PHP код:

diff --git a/src/game/MiscHandler.cpp b/src/game/MiscHandler.cpp
index e7a1d27..6fd10fc 100644
--- a/src/game/MiscHandler.cpp
+++ b/src/game/MiscHandler.cpp
@@ -806,6 +806,25 @@ void WorldSession::HandleAreaTriggerOpcode(WorldPacket & recv_data)
                 GetPlayer()->SendTransferAborted(at->target_mapId, TRANSFER_ABORT_DIFFICULTY, GetPlayer()->GetDifficulty(mapEntry->IsRaid()));
             return;
         }
+
+        // resurrect player - if it not in raid group
+        if (!GetPlayer()->isAlive())
+        {
+            Group* group = GetPlayer()->GetGroup();
+            if (!group || !group->isRaidGroup())
+            {
+                Corpse *corpse = GetPlayer()->GetCorpse();
+                uint32 corpsemapid = corpse->GetMapId();
+                if (corpsemapid != GetPlayer()->GetMapId())
+                {
+                    if (MapEntry const* corpseMapEntry = sMapStore.LookupEntry(corpsemapid))
+                    {
+                        if (corpseMapEntry->IsDungeon() && corpseMapEntry->ghost_entrance_map >= 0)
+                            GetPlayer()->ResurrectPlayer(0.5f);
+                    }
+                }
+            }
+        }
     }
 
     GetPlayer()->TeleportTo(at->target_mapId, at->target_X, at->target_Y, at->target_Z, at->target_Orientation, TELE_TO_NOT_LEAVE_TRANSPORT); 


[Den]

Позабыты две мелочи:
1) если призраком пытатся зайти в рейдовое подземелье - то далжно писать, что трупа нет в подземелье.
2) если труп есть - воскрешаемся и ничего не пишится.
У нас же в двух случаях пишится, что нет рейдовой группы.
Подрозумеваются рейдовые подземелья.

PHP код:

diff --git a/src/game/MapManager.cpp b/src/game/MapManager.cpp
index 52ed4a2..5ed339e 100644
--- a/src/game/MapManager.cpp
+++ b/src/game/MapManager.cpp
@@ -168,11 +168,14 @@ bool MapManager::CanPlayerEnter(uint32 mapid, Player* player)
                 Group* group = player->GetGroup();
                 if (!group || !group->isRaidGroup())
                 {
-                    // probably there must be special opcode, because client has this string constant in GlobalStrings.lua
-                    // TODO: this is not a good place to send the message
-                    player->GetSession()->SendAreaTriggerMessage("You must be in a raid group to enter %s instance", mapName);
-                    DEBUG_LOG("MAP: Player '%s' must be in a raid group to enter instance of '%s'", player->GetName(), mapName);
-                    return false;
+                    if (player->isAlive())
+                    {
+                        // probably there must be special opcode, because client has this string constant in GlobalStrings.lua
+                        // TODO: this is not a good place to send the message
+                        player->GetSession()->SendAreaTriggerMessage("You must be in a raid group to enter %s instance", mapName);
+                        DEBUG_LOG("MAP: Player '%s' must be in a raid group to enter instance of '%s'", player->GetName(), mapName);
+                        return false;
+                    }
                 }
             }
         }
diff --git a/src/game/MiscHandler.cpp b/src/game/MiscHandler.cpp
index e7a1d27..bca21da 100644
--- a/src/game/MiscHandler.cpp
+++ b/src/game/MiscHandler.cpp
@@ -806,6 +806,26 @@ void WorldSession::HandleAreaTriggerOpcode(WorldPacket & recv_data)
                 GetPlayer()->SendTransferAborted(at->target_mapId, TRANSFER_ABORT_DIFFICULTY, GetPlayer()->GetDifficulty(mapEntry->IsRaid()));
             return;
         }
+
+        // resurrect player - if it not in raid group
+        if (!GetPlayer()->isAlive())
+        {
+            Group* group = GetPlayer()->GetGroup();
+            if (!group || !group->isRaidGroup())
+            {
+                Corpse *corpse = GetPlayer()->GetCorpse();
+                uint32 corpsemapid = corpse->GetMapId();
+                if (corpsemapid != GetPlayer()->GetMapId())
+                {
+                    if (MapEntry const* corpseMapEntry = sMapStore.LookupEntry(corpsemapid))
+                    {
+                        if (corpseMapEntry->IsDungeon() && corpseMapEntry->ghost_entrance_map >= 0)
+                            GetPlayer()->ResurrectPlayer(0.5f);
+                        return;
+                    }
+                }
+            }
+        }
     }
 
     GetPlayer()->TeleportTo(at->target_mapId, at->target_X, at->target_Y, at->target_Z, at->target_Orientation, TELE_TO_NOT_LEAVE_TRANSPORT); 


[TOM_RUS]

Насколько я знаю, игрок должен воскрешается до телепорта в инст, а не после него как у нас... (но естественно после проверок на существования трупа в данном конкретном инсте и т.д.).

[Fedia22]

если он состоит в рейде то он воскрешается и портуется в этот рейд
если он не состоит в рейде а труп в данном подземелье то при подходе к порталу он воскрешается и никуда не заходит (как сказал Том_Рус)

[Йоха]

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

если он не состоит в рейде а труп в данном подземелье то при подходе к порталу он воскрешается и никуда не заходит (как сказал Том_Рус)

подтверждаю

[Den]

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

если он не состоит в рейде а труп в данном подземелье то при подходе к порталу он воскрешается и никуда не заходит (как сказал Том_Рус)

Данный патч был сделан именно для этого.

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

если он состоит в рейде то он воскрешается и портуется в этот рейд

Да, самому заметить это невозможно, поэтому проверил с двух акаунтав, очень хорошо видно, как воскрешается и только потом портуется в рейд.

Поэтому нужно править в других местах.

[Den]

А если, сделаем как на офе, вот так ?

PHP код:

diff --git a/src/game/MapManager.cpp b/src/game/MapManager.cpp
index 0e05860..1001901 100644
--- a/src/game/MapManager.cpp
+++ b/src/game/MapManager.cpp
@@ -170,11 +170,14 @@ bool MapManager::CanPlayerEnter(uint32 mapid, Player* player)
                 Group* group = player->GetGroup();
                 if (!group || !group->isRaidGroup())
                 {
-                    // probably there must be special opcode, because client has this string constant in GlobalStrings.lua
-                    // TODO: this is not a good place to send the message
-                    player->GetSession()->SendAreaTriggerMessage("You must be in a raid group to enter %s instance", mapName);
-                    DEBUG_LOG("MAP: Player '%s' must be in a raid group to enter instance of '%s'", player->GetName(), mapName);
-                    return false;
+                    if (player->isAlive())      // no area message for ghost player
+                    {
+                        // probably there must be special opcode, because client has this string constant in GlobalStrings.lua
+                        // TODO: this is not a good place to send the message
+                        player->GetSession()->SendAreaTriggerMessage("You must be in a raid group to enter %s instance", mapName);
+                        DEBUG_LOG("MAP: Player '%s' must be in a raid group to enter instance of '%s'", player->GetName(), mapName);
+                        return false;
+                    }
                 }
             }
         }
diff --git a/src/game/MiscHandler.cpp b/src/game/MiscHandler.cpp
index 557c6a8..ef53d3c 100644
--- a/src/game/MiscHandler.cpp
+++ b/src/game/MiscHandler.cpp
@@ -806,6 +806,51 @@ void WorldSession::HandleAreaTriggerOpcode(WorldPacket & recv_data)
                 GetPlayer()->SendTransferAborted(at->target_mapId, TRANSFER_ABORT_DIFFICULTY, GetPlayer()->GetDifficulty(mapEntry->IsRaid()));
             return;
         }
+
+        if (mapEntry->IsDungeon() || mapEntry->IsRaid())
+        {
+            if (!GetPlayer()->isAlive())
+            {
+                Group* group = GetPlayer()->GetGroup();
+                Corpse *corpse = GetPlayer()->GetCorpse();
+                uint32 corpsemapid = corpse->GetMapId();
+
+                if (group)
+                {
+                    if (corpsemapid != GetPlayer()->GetMapId())
+                    {
+                        if (MapEntry const* corpseMapEntry = sMapStore.LookupEntry(corpsemapid))
+                        {
+                            if (corpseMapEntry->IsDungeon() && corpseMapEntry->ghost_entrance_map >= 0)
+                            {
+                                GetPlayer()->ResurrectPlayer(0.5f);
+                                GetPlayer()->SpawnCorpseBones();
+
+                                if (!group->isRaidGroup() && mapEntry->IsRaid())
+                                    return;
+                            }
+                        }
+                    }
+                }
+                else
+                {
+                    if (corpsemapid != GetPlayer()->GetMapId())
+                    {
+                        if (MapEntry const* corpseMapEntry = sMapStore.LookupEntry(corpsemapid))
+                        {
+                            if (corpseMapEntry->IsDungeon() && corpseMapEntry->ghost_entrance_map >= 0)
+                            {
+                                GetPlayer()->ResurrectPlayer(0.5f);
+                                GetPlayer()->SpawnCorpseBones();
+
+                                if (mapEntry->IsRaid())
+                                    return;
+                            }
+                        }
+                    }
+                }
+            }
+        }
     }
 
     GetPlayer()->TeleportTo(at->target_mapId, at->target_X, at->target_Y, at->target_Z, at->target_Orientation, TELE_TO_NOT_LEAVE_TRANSPORT);
diff --git a/src/game/MovementHandler.cpp b/src/game/MovementHandler.cpp
index 412382e..2c05c20 100644
--- a/src/game/MovementHandler.cpp
+++ b/src/game/MovementHandler.cpp
@@ -169,17 +169,6 @@ void WorldSession::HandleMoveWorldportAckOpcode()
         GetPlayer()->m_taxi.ClearTaxiDestinations();
     }
 
-    // resurrect character at enter into instance where his corpse exist after add to map
-    Corpse *corpse = GetPlayer()->GetCorpse();
-    if (corpse && corpse->GetType() != CORPSE_BONES && corpse->GetMapId() == GetPlayer()->GetMapId())
-    {
-        if( mEntry->IsDungeon() )
-        {
-            GetPlayer()->ResurrectPlayer(0.5f);
-            GetPlayer()->SpawnCorpseBones();
-        }
-    }
-
     if (mInstance)
     {
         Difficulty diff = GetPlayer()->GetDifficulty(mEntry->IsRaid()); 


[Den]

Патч для 11426.
Поубирал левые пробелы.
Хотелось бы услышать комментарии... Может это вообще хак, и стоит подождать пока кто-нибудь из мангос тим возьмется за исправление ?

PHP код:

diff --git a/src/game/AuctionHouseMgr.cpp b/src/game/AuctionHouseMgr.cpp
diff --git a/src/game/MapManager.cpp b/src/game/MapManager.cpp
index cece28f..5930b1e 100644
--- a/src/game/MapManager.cpp
+++ b/src/game/MapManager.cpp
@@ -173,11 +173,14 @@ bool MapManager::CanPlayerEnter(uint32 mapid, Player* player)
                 Group* group = player->GetGroup();
                 if (!group || !group->isRaidGroup())
                 {
-                    // probably there must be special opcode, because client has this string constant in GlobalStrings.lua
-                    // TODO: this is not a good place to send the message
-                    player->GetSession()->SendAreaTriggerMessage("You must be in a raid group to enter %s instance", mapName);
-                    DEBUG_LOG("MAP: Player '%s' must be in a raid group to enter instance of '%s'", player->GetName(), mapName);
-                    return false;
+                    if (player->isAlive())      // no area message for ghost player
+                    {
+                        // probably there must be special opcode, because client has this string constant in GlobalStrings.lua
+                        // TODO: this is not a good place to send the message
+                        player->GetSession()->SendAreaTriggerMessage("You must be in a raid group to enter %s instance", mapName);
+                        DEBUG_LOG("MAP: Player '%s' must be in a raid group to enter instance of '%s'", player->GetName(), mapName);
+                        return false;
+                    }
                 }
             }
         }
diff --git a/src/game/MiscHandler.cpp b/src/game/MiscHandler.cpp
index d03cb8f..3d70e41 100644
--- a/src/game/MiscHandler.cpp
+++ b/src/game/MiscHandler.cpp
@@ -806,6 +806,49 @@ void WorldSession::HandleAreaTriggerOpcode(WorldPacket & recv_data)
                 GetPlayer()->SendTransferAborted(at->target_mapId, TRANSFER_ABORT_DIFFICULTY, GetPlayer()->GetDifficulty(mapEntry->IsRaid()));
             return;
         }
+
+        // ressurect to teleport in instance
+        if (mapEntry->IsDungeon() || mapEntry->IsRaid())
+        {
+            if (!GetPlayer()->isAlive())
+            {
+                Group* group = GetPlayer()->GetGroup();
+                Corpse* corpse = GetPlayer()->GetCorpse();
+                uint32 corpsemapid = corpse->GetMapId();
+                MapEntry const* corpseMapEntry = sMapStore.LookupEntry(corpsemapid);
+
+                if (group)
+                {
+                    if (corpsemapid != GetPlayer()->GetMapId())
+                    {
+                        if (corpseMapEntry->IsDungeon() && corpseMapEntry->ghost_entrance_map >= 0)
+                        {
+                            GetPlayer()->ResurrectPlayer(0.5f);
+                            GetPlayer()->SpawnCorpseBones();
+
+                            // not area message
+                            if (!group->isRaidGroup() && mapEntry->IsRaid())
+                                return;
+                        }
+                    }
+                }
+                else
+                {
+                    if (corpsemapid != GetPlayer()->GetMapId())
+                    {
+                        if (corpseMapEntry->IsDungeon() && corpseMapEntry->ghost_entrance_map >= 0)
+                        {
+                            GetPlayer()->ResurrectPlayer(0.5f);
+                            GetPlayer()->SpawnCorpseBones();
+
+                            // not area message
+                            if (mapEntry->IsRaid())
+                                return;
+                        }
+                    }
+                }
+            }
+        }
     }
 
     GetPlayer()->TeleportTo(at->target_mapId, at->target_X, at->target_Y, at->target_Z, at->target_Orientation, TELE_TO_NOT_LEAVE_TRANSPORT);
diff --git a/src/game/MovementHandler.cpp b/src/game/MovementHandler.cpp
index 7ecd4bd..ea77930 100644
--- a/src/game/MovementHandler.cpp
+++ b/src/game/MovementHandler.cpp
@@ -169,17 +169,6 @@ void WorldSession::HandleMoveWorldportAckOpcode()
         GetPlayer()->m_taxi.ClearTaxiDestinations();
     }
 
-    // resurrect character at enter into instance where his corpse exist after add to map
-    Corpse *corpse = GetPlayer()->GetCorpse();
-    if (corpse && corpse->GetType() != CORPSE_BONES && corpse->GetMapId() == GetPlayer()->GetMapId())
-    {
-        if( mEntry->IsDungeon() )
-        {
-            GetPlayer()->ResurrectPlayer(0.5f);
-            GetPlayer()->SpawnCorpseBones();
-        }
-    }
-
     if (mInstance)
     {
         Difficulty diff = GetPlayer()->GetDifficulty(mEntry->IsRaid());
diff --git a/src/game/Player.cpp b/src/game/Player.cpp
index b3bf4c0..d455b41 100644
--- a/src/game/Player.cpp
+++ b/src/game/Player.cpp
@@ -4446,13 +4446,6 @@ void Player::BuildPlayerRepop()
     // there must be SMSG.STOP_MIRROR_TIMER
     // there we must send 888 opcode
 
-    // the player cannot have a corpse already, only bones which are not returned by GetCorpse
-    if(GetCorpse())
-    {
-        sLog.outError("BuildPlayerRepop: player %s(%d) already has a corpse", GetName(), GetGUIDLow());
-        MANGOS_ASSERT(false);
-    }
-
     // create a corpse and place it at the player's location
     Corpse *corpse = CreateCorpse();
     if(!corpse) 


[Vladimir]

Вы зачем часть // the player cannot have a corpse already,
убили? Она должна выполняться при любых раскладах или баг какойто в другом месте.

Со входом призраком в dungeon - понятно - воскрешаемся и если можем заходим.
Непонятен случай с многоуровневыми подземельями - там то труп на уровень глубже и вроде коскрешение только перед необъодимым вторым телепортом. Как в таких случаях ситуация зарруливается на оффе?

[LordJZ]

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

Вы зачем часть // the player cannot have a corpse already,
убили? Она должна выполняться при любых раскладах или баг какойто в другом месте.

Со входом призраком в dungeon - понятно - воскрешаемся и если можем заходим.
Непонятен случай с многоуровневыми подземельями - там то труп на уровень глубже и вроде коскрешение только перед необъодимым вторым телепортом. Как в таких случаях ситуация зарруливается на оффе?

При входе в не тот вход телепортнет в правильный.

[RomanRom2]

в какой "не в тот"? там же один вход...
из одного в другой, последовательно. из мапы (скажем) 0 идем в инст с мапой (скажем) 10, из 10 в 11, из 11 в 12. вот труп остался лежать в 12. мы сдохли и нас телепортнуло в 0. при подходе ко входу в инст (0->10) что будет происходить?

воскрешение и телепорт сразу в 12? или в 10 только?

[Йоха]

что за многоуровневые инсты ? пример ? в любом инсте при входе духом появляешся в самом начале, независимо от того где умер. Под "началом" подразумевается точка в которую попадаешь после портала если войти в инст в нормальном состоянии.
А, или есть инсты с двумя входами, например 5 ппл в гундраке, ну так там куда заходишь там и появляешся сразу за порталом.

[rsa]

эх, молодежь... вход в инст 409 "огненные недра" находится в инсте 230 "глубины черной горы".

[TOM_RUS]

Разве в огненные недра не NPC портует? (ну и в BWL через орб)...

[rsa]

вот я и говорю - эх, молодежь... этого НПЦ не так давно поставили, раньше тот кто просто дошел до 409 уже мог себе орден прикручивать (а туда еще чтобы войти квестцепочка на 120 позиций была... помню прошел, и наш рейд первые 2 моба в недрах положили... 40 рыл экипированных по самое то...)

[Йоха]

Нормальные люди в Molten Core попадают путем разговора с эльфом в самом начале, там где вылазишь из лавы. Что бы этот нпц портовал в МС, у этого же нпц берется квест.
Та же фигня была в BWL, где вход в инст в виде портала был из UBRS, но если сделать определенную цепочку, то сразу в BWL портовало от Hand of command в конце тонеля направо от входа в LBRS.

В любом случае если умереть в БВЛ, а зайти в LBRS, то реснешся у входа в LBRS, там в свое время были глобальные ПВП битвы между рейдами орды и альянса 40 на 40 тел, которые траили BWL.

Так что умерев в МС и зайдя BRD, реснешся у входа, так же как если зайти в любой другой инст где нет твоего трупика

[TOM_RUS]

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

вот я и говорю - эх, молодежь... этого НПЦ не так давно поставили, раньше тот кто просто дошел до 409 уже мог себе орден прикручивать (а туда еще чтобы войти квестцепочка на 120 позиций была... помню прошел, и наш рейд первые 2 моба в недрах положили... 40 рыл экипированных по самое то...)

NPC добавили в патче 1.11.1, что достаточно давно было...

[Йоха]

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

вот я и говорю - эх, молодежь... этого НПЦ не так давно поставили

лол, этот нпц стоит там как минимум с февраля 2006 года, я тогда стал рейдить.
Никакой дурак в МС не ходил через BRD, всем делали преквест.

[RomanRom2]

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

вот я и говорю - эх, молодежь...

точно

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

этого НПЦ не так давно поставили

ох жесть, серьезно? как все понерфили...

ну хорошо, вопрос может не по теме, но второй от меня: а как оно было в "оригинале" то?

[rsa]

Сообщение от Йоха

лол, этот нпц стоит там как минимум с февраля 2006 года, я тогда стал рейдить.

ну а я где-то тогда как раз перестал 1й раз

[Йоха]

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

ну хорошо, вопрос может не по теме, но второй от меня: а как оно было в "оригинале" то?

так же как и сейчас. нет никаких вложенных инстов.

[LordJZ]

Сообщение от Йоха

так же как и сейчас. нет никаких вложенных инстов.

Blackwing Lair?

Добавлено: Аха, невнимательно читал...

[RomanRom2]

да есть такие, есть. были по меньшей мере два, не помню названий.
деды говорят, что после смерти тебя телепортит на материк, на кладбище.

после попытки войти в инст происходит следующее, два варианта:
1. ты мертвый:
- если группа уже расформирована и/или трупа нет, то воскрешаешься на месте и никуда не телепортируешься, остаешься перед входом.
- если все ок с группой и трупом, то телепортишься в самый вложенный инст в место, где оказываешься сразу после входа в инст. по сути туда, где труп.
2. ты живой:
- если группа уже расформирована и/или трупа нет, то воскрешаешься на месте и никуда не телепортируешься, остаешься перед входом.
- если все ок с группой и трупом, то опять два варианта:
а) если у тебя есть квестовый итем и/или выполнен квест (где как), то подходишь к нпц и он тебя может телепортировать сразу во внутренний инст.
б) если у тебя ничего нет, то нпц тебя пошлет в пеший эротический поход, то есть ты входишь в первый инст как обычно. и вэлкам по новой через него бежать.

[Den]

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

Вы зачем часть // the player cannot have a corpse already,
убили? Она должна выполняться при любых раскладах или баг какойто в другом месте.

Если с этой частью, то иногда в лог вылетает:

BuildPlayerRepop: player Тест(1) already has a corpse
..\..\src\game\Player.cpp:4460: Error: Assertion in Player::BuildPlayerRepop failed: false
Stack Trace:
WorldSession::ExecuteOpcode() m:\11426\mangos\src\game\worldsession.cpp:930 + 0x0
WorldSession::Update() m:\11426\mangos\src\game\worldsession.cpp: 230 + 0x0
World::UpdateSessions() m:\11426\mangos\src\game\world.cpp: 1906 + 0x10
World::Update() m:\11426\mangos\src\game\world.cpp: 1454 + 0x0
WorldRunnable::run() m:\11426\mangos\src\mangosd\worldrunnable.cpp: 61 + 0x0
ACE_Based::Thread::ThreadTask() m:\11426\mangos\src\shared\threading.cpp:190 + 0x0
ACE_OS_Thread_Adapter::invoke()+0x74 [0x0]
endthreadex()+0x44 [0x0]
endthreadex()+0xd8 [0x0]
GetModuleFileNameA()+0x1ba [0x0]

[Vladimir]

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

- если все ок с группой и трупом, то телепортишься в самый вложенный инст в место, где оказываешься сразу после входа в инст

Понятно. Тогда у нас телепортации вроде тоже не хватает.

Если с этой частью, то иногда в лог вылетает:

Из приведенного стека непонятно на каком опкоде вылетает.

[Vladimir]

Я сейчас тестирую переписанную версию:
https://gist.github.com/961603

[added]
В [11446].