diff options
| author | Duarte Duarte <dnpd.dd@gmail.com> | 2014-08-29 14:09:13 +0100 |
|---|---|---|
| committer | Duarte Duarte <dnpd.dd@gmail.com> | 2014-08-29 14:09:13 +0100 |
| commit | 743088d683bc0b141a6ea9b1713ff7ad0bd7ea32 (patch) | |
| tree | fc8a33433728accedbe8a83f585ebb1f5d6f9302 | |
| parent | fb4a8afa1250454ccb566b3d1841c927d22b9a8f (diff) | |
| parent | c0697c77260f729742eea2fb777ba3117741f902 (diff) | |
Merge pull request #12959 from Nyeriah/patch-qf
Core/Player: Already complete quests shouldn't turn failed
| -rw-r--r-- | src/server/game/Entities/Player/Player.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/server/game/Entities/Player/Player.cpp b/src/server/game/Entities/Player/Player.cpp index baafa14d213..71a7f4b160e 100644 --- a/src/server/game/Entities/Player/Player.cpp +++ b/src/server/game/Entities/Player/Player.cpp @@ -15448,6 +15448,10 @@ void Player::RewardQuest(Quest const* quest, uint32 reward, Object* questGiver, void Player::FailQuest(uint32 questId) { + // Already complete quests shouldn't turn failed. + if (GetQuestStatus(questId) == QUEST_STATUS_COMPLETE) + return; + if (Quest const* quest = sObjectMgr->GetQuestTemplate(questId)) { SetQuestStatus(questId, QUEST_STATUS_FAILED); |