From ec013e28616e46fbd17709cdc962a87397529338 Mon Sep 17 00:00:00 2001 From: Xurkon <36556990+Xurkon@users.noreply.github.com> Date: Sat, 21 Feb 2026 11:29:02 -0600 Subject: [PATCH] Demote cache count mismatch from Error to Debug When isQuestLogGood passes, the cache IS valid. The goodQuestsCount != numQuests mismatch is a false positive on WotLK private servers where GetNumQuestLogEntries returns a different numQuests than what the objectives loop validated. Log at debug level so it doesn't spam player chat. --- Modules/QuestieValidateGameCache.lua | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Modules/QuestieValidateGameCache.lua b/Modules/QuestieValidateGameCache.lua index 8c019e4..5625fb4 100644 --- a/Modules/QuestieValidateGameCache.lua +++ b/Modules/QuestieValidateGameCache.lua @@ -122,10 +122,10 @@ local function OnQuestLogUpdate() end if goodQuestsCount ~= numQuests then - -- This shouldn't be possible - - Questie:Error("Game Cache has still a broken quest log. Good quest: "..goodQuestsCount.."/"..numQuests..". Please report this on Github or Discord!") -- Translations might not be available yet. - -- TODO should we stop whole addon loading progress? + -- This count mismatch can occur on WotLK private servers where GetNumQuestLogEntries + -- returns a different value than objectives-loop counted. The cache is valid because + -- isQuestLogGood already passed above. Log at debug level only. + Questie:Debug(Questie.DEBUG_INFO, "[QuestieValidateGameCache] Quest count mismatch (expected "..tostring(numQuests)..", validated "..tostring(goodQuestsCount).."). Cache is still valid.") end DestroyEventFrame()