From b7c8c09920387f08bc6e74b6c5ca13d9541b514b Mon Sep 17 00:00:00 2001 From: UltraNix <80540499+UltraNix@users.noreply.github.com> Date: Fri, 15 Apr 2022 16:47:13 +0200 Subject: [PATCH] fix(Core/Quests): Fixed displaying correct quest marks for autocomplete/repeatable/daily completed quests. (#11349) Fixes #11237 --- .../game/Entities/Player/PlayerQuest.cpp | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/server/game/Entities/Player/PlayerQuest.cpp b/src/server/game/Entities/Player/PlayerQuest.cpp index 037d4733d..061b4fcdd 100644 --- a/src/server/game/Entities/Player/PlayerQuest.cpp +++ b/src/server/game/Entities/Player/PlayerQuest.cpp @@ -1612,24 +1612,24 @@ QuestGiverStatus Player::GetQuestDialogStatus(Object* questgiver) continue; QuestStatus status = GetQuestStatus(questId); - if ((status == QUEST_STATUS_COMPLETE && !GetQuestRewardStatus(questId)) || (quest->IsAutoComplete() && CanTakeQuest(quest, false))) + if (status == QUEST_STATUS_COMPLETE && !GetQuestRewardStatus(questId)) { - if (quest->IsRepeatable() || quest->IsDailyOrWeekly()) - { - result2 = DIALOG_STATUS_REWARD_REP; - } - else - { - result2 = DIALOG_STATUS_REWARD; - } + result2 = DIALOG_STATUS_REWARD; } else if (status == QUEST_STATUS_INCOMPLETE) { result2 = DIALOG_STATUS_INCOMPLETE; } + if (quest->IsAutoComplete() && CanTakeQuest(quest, false) && quest->IsRepeatable() && !quest->IsDailyOrWeekly()) + { + result2 = DIALOG_STATUS_REWARD_REP; + } + if (result2 > result) + { result = result2; + } } for (QuestRelations::const_iterator i = qr.first; i != qr.second; ++i)