aboutsummaryrefslogtreecommitdiff
path: root/src/server/game/Spells/SpellEffects.cpp
diff options
context:
space:
mode:
authorSnapperRy <snapperryen@gmail.com>2016-09-30 13:17:04 +0200
committerjoschiwald <joschiwald.trinity@gmail.com>2017-03-04 19:01:43 +0100
commit4f34c8c5636275ec512622f2e068f02043e02ad9 (patch)
tree46c9e7fc92b2b9e063809237d7596cdc9600c215 /src/server/game/Spells/SpellEffects.cpp
parent034ff93ba869ad5dd69b043a70f7405a1c542c3b (diff)
Core/Spell: set quest to rewarded instead of complete when targeted by SPELL_EFFECT_QUEST_COMPLETE. (#18021)
Prevents people from ending up with strange quests in their quest logs. (cherry picked from commit 6d62f852acf3e1a3ee57cde2442832280aee8149)
Diffstat (limited to 'src/server/game/Spells/SpellEffects.cpp')
-rw-r--r--src/server/game/Spells/SpellEffects.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/server/game/Spells/SpellEffects.cpp b/src/server/game/Spells/SpellEffects.cpp
index 8693cf10747..db59ed14f10 100644
--- a/src/server/game/Spells/SpellEffects.cpp
+++ b/src/server/game/Spells/SpellEffects.cpp
@@ -4246,8 +4246,8 @@ void Spell::EffectQuestComplete(SpellEffIndex /*effIndex*/)
uint16 logSlot = player->FindQuestSlot(questId);
if (logSlot < MAX_QUEST_LOG_SIZE)
player->AreaExploredOrEventHappens(questId);
- else if (player->CanTakeQuest(quest, false)) // never rewarded before
- player->CompleteQuest(questId); // quest not in log - for internal use
+ else if (player->CanTakeQuest(quest, false)) // Check if the quest has already been turned in.
+ player->SetRewardedQuest(questId); // If not, set status to rewarded without broadcasting it to client.
}
}