From fa7974b0d8e0e89c977f50f8fa92692098f41f7e Mon Sep 17 00:00:00 2001 From: Xurkon Date: Mon, 16 Mar 2026 22:14:04 -0500 Subject: [PATCH] fix: mouseover NPC check uses raw npcData instead of GetNPC to prevent CRITICAL spam --- Modules/QuestieLearner.lua | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Modules/QuestieLearner.lua b/Modules/QuestieLearner.lua index 35b28ec..ca37d65 100644 --- a/Modules/QuestieLearner.lua +++ b/Modules/QuestieLearner.lua @@ -524,10 +524,10 @@ function QuestieLearner:OnMouseoverUnit() local isQuestGiver = NpcFlagsHasQuestGiver(npcFlags) if not isQuestGiver then - -- Check if the DB already knows this NPC as a quest starter or finisher - local dbNpc = QuestieDB and QuestieDB.GetNPC and QuestieDB:GetNPC(npcId) - if dbNpc and (dbNpc[7] or dbNpc[8]) then - -- known quest-related NPC: update coordinates only + -- Silently check raw table — do NOT call GetNPC which logs CRITICAL for every miss + local rawNpc = QuestieDB and QuestieDB.npcData and QuestieDB.npcData[npcId] + if rawNpc and (rawNpc[10] or rawNpc[11]) then + -- known quest starter (key 10) or quest ender (key 11) isQuestGiver = true end end