diff options
| author | Wyrserth <wyrserth@protonmail.com> | 2019-07-17 16:05:20 +0200 |
|---|---|---|
| committer | Shauren <shauren.trinity@gmail.com> | 2021-12-16 21:20:38 +0100 |
| commit | cc0a9d1fe96de7d59659c678814e6371be57774d (patch) | |
| tree | 60064bad1f0fef4f02ba3ab36f2f87c7688c725c | |
| parent | 093223d7b43b4890054477622343e0874a1998e4 (diff) | |
Core/Player: don't send errors to client from Satisfy* methods when Player::CanRewardQuest() is called with msg = false.
Fixes cases where people would get strange errors like "You are not high enough level for that quest" when entering WotLK instances.
(cherry picked from commit a4ddb30489d3c4cd96817232b28393c2f4e50595)
| -rw-r--r-- | src/server/game/Entities/Player/Player.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/server/game/Entities/Player/Player.cpp b/src/server/game/Entities/Player/Player.cpp index e8ae4bbdd49..e9ccab6cba9 100644 --- a/src/server/game/Entities/Player/Player.cpp +++ b/src/server/game/Entities/Player/Player.cpp @@ -15188,11 +15188,11 @@ bool Player::CanRewardQuest(Quest const* quest, bool msg) const return false; // daily quest can't be rewarded (25 daily quest already completed) - if (!SatisfyQuestDay(quest, true) || !SatisfyQuestWeek(quest, true) || !SatisfyQuestMonth(quest, true) || !SatisfyQuestSeasonal(quest, true)) + if (!SatisfyQuestDay(quest, msg) || !SatisfyQuestWeek(quest, msg) || !SatisfyQuestMonth(quest, msg) || !SatisfyQuestSeasonal(quest, msg)) return false; // player no longer satisfies the quest's requirements (skill level etc.) - if (!SatisfyQuestLevel(quest, true) || !SatisfyQuestSkill(quest, true) || !SatisfyQuestReputation(quest, true)) + if (!SatisfyQuestLevel(quest, msg) || !SatisfyQuestSkill(quest, msg) || !SatisfyQuestReputation(quest, msg)) return false; // rewarded and not repeatable quest (only cheating case, then ignore without message) |