From 60808a268d8f0429365bc676291386f457c9a387 Mon Sep 17 00:00:00 2001 From: hondacrx Date: Sat, 1 Jan 2022 15:37:01 -0500 Subject: [PATCH] Core/Player: don't send errors to client from Satisfy* methods when Player::CanRewardQuest() is called with msg = false. Port From (https://github.com/TrinityCore/TrinityCore/commit/cc0a9d1fe96de7d59659c678814e6371be57774d) --- Source/Game/Entities/Player/Player.Quest.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Source/Game/Entities/Player/Player.Quest.cs b/Source/Game/Entities/Player/Player.Quest.cs index a10f6243c..a8b8cf1b2 100644 --- a/Source/Game/Entities/Player/Player.Quest.cs +++ b/Source/Game/Entities/Player/Player.Quest.cs @@ -536,11 +536,11 @@ namespace Game.Entities 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)