diff options
Diffstat (limited to 'src/server/game/Entities/Player')
| -rw-r--r-- | src/server/game/Entities/Player/Player.cpp | 45 |
1 files changed, 37 insertions, 8 deletions
diff --git a/src/server/game/Entities/Player/Player.cpp b/src/server/game/Entities/Player/Player.cpp index c53bb8a693c..23f919da9c0 100644 --- a/src/server/game/Entities/Player/Player.cpp +++ b/src/server/game/Entities/Player/Player.cpp @@ -16535,21 +16535,50 @@ void Player::RemoveRewardedQuest(uint32 questId, bool update /*= true*/) void Player::SendQuestUpdate(uint32 questId) { - uint32 zone = 0, area = 0; - GetZoneAndAreaId(zone, area); - - SpellAreaForQuestAreaMapBounds saBounds = sSpellMgr->GetSpellAreaForQuestAreaMapBounds(area, questId); + SpellAreaForQuestMapBounds saBounds = sSpellMgr->GetSpellAreaForQuestMapBounds(questId); if (saBounds.first != saBounds.second) { - for (SpellAreaForQuestAreaMap::const_iterator itr = saBounds.first; itr != saBounds.second; ++itr) + std::set<uint32> aurasToRemove, aurasToCast; + uint32 zone = 0, area = 0; + GetZoneAndAreaId(zone, area); + + for (SpellAreaForQuestMap::const_iterator itr = saBounds.first; itr != saBounds.second; ++itr) { if (itr->second->flags & SPELL_AREA_FLAG_AUTOREMOVE && !itr->second->IsFitToRequirements(this, zone, area)) - RemoveAurasDueToSpell(itr->second->spellId); + aurasToRemove.insert(itr->second->spellId); else if (itr->second->flags & SPELL_AREA_FLAG_AUTOCAST && !(itr->second->flags & SPELL_AREA_FLAG_IGNORE_AUTOCAST_ON_QUEST_STATUS_CHANGE)) - if (!HasAura(itr->second->spellId)) - CastSpell(this, itr->second->spellId, true); + aurasToCast.insert(itr->second->spellId); + } + + // Auras matching the requirements will be inside the aurasToCast container. + // Auras not matching the requirements may prevent using auras matching the requirements. + // aurasToCast will erase conflicting auras in aurasToRemove container to handle spells used by multiple quests. + + for (auto itr = aurasToRemove.begin(); itr != aurasToRemove.end();) + { + bool auraRemoved = false; + + for (const auto i : aurasToCast) + { + if (*itr == i) + { + itr = aurasToRemove.erase(itr); + auraRemoved = true; + break; + } + } + + if (!auraRemoved) + ++itr; } + + for (auto spellId : aurasToCast) + if (!HasAura(spellId)) + CastSpell(this, spellId, true); + + for (auto spellId : aurasToRemove) + RemoveAurasDueToSpell(spellId); } UpdateVisibleGameobjectsOrSpellClicks(); |
