nchebrov/azerothcore-wotlk
1
0
Fork 0
mirror of https://github.com/mod-playerbots/azerothcore-wotlk.git synced 2026-02-15 16:26:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(Core/Groups): fix group enchanting level not resetting properly (#5501)

Browse Source
This commit is contained in:
Axel Cocat
2021-05-07 13:51:39 +02:00
committed by GitHub
parent acb7c6291c
commit 0e8e21b812
3 changed files with 15 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/server/game/Groups/Group.cpp
View File

@@ -557,10 +557,6 @@ bool Group::RemoveMember(ObjectGuid guid, const RemoveMethod& method /*= GROUP_R
CharacterDatabase.Execute(stmt);
}
// Reevaluate group enchanter if the leaving player had enchanting skill or the player is offline
if (!player || player->GetSkillValue(SKILL_ENCHANTING))
ResetMaxEnchantingLevel();
// Remove player from loot rolls
for (Rolls::iterator it = RollId.begin(); it != RollId.end();)
{
@@ -602,6 +598,12 @@ bool Group::RemoveMember(ObjectGuid guid, const RemoveMethod& method /*= GROUP_R
sWorld->UpdateGlobalPlayerGroup(guid.GetCounter(), 0);
}
// Reevaluate group enchanter if the leaving player had enchanting skill or the player is offline
if (!player || player->GetSkillValue(SKILL_ENCHANTING))
{
ResetMaxEnchantingLevel();
}
// Pick new leader if necessary
bool validLeader = true;
if (m_leaderGuid == guid)
@@ -2057,8 +2059,11 @@ void Group::ResetMaxEnchantingLevel()
for (member_citerator citr = m_memberSlots.begin(); citr != m_memberSlots.end(); ++citr)
{
pMember = ObjectAccessor::FindPlayer(citr->guid);
if (pMember && m_maxEnchantingLevel < pMember->GetSkillValue(SKILL_ENCHANTING))
if (pMember && pMember->GetSession() && !pMember->GetSession()->IsSocketClosed()
&& m_maxEnchantingLevel < pMember->GetSkillValue(SKILL_ENCHANTING))
{
m_maxEnchantingLevel = pMember->GetSkillValue(SKILL_ENCHANTING);
}
}
}