From 922fa8e0ff76652bbaab395fc5194bf182d22ed1 Mon Sep 17 00:00:00 2001
From: Xurkon <36556990+Xurkon@users.noreply.github.com>
Date: Sat, 21 Feb 2026 12:18:45 -0600
Subject: [PATCH] Fix Lua truthiness bug in QuestieDB.IsComplete

In Lua, 0 evaluates as truthy. The check
'if questLogEntry.isComplete then return questLogEntry.isComplete end'
was completely bypassing our fallback completion logic because
GetQuestLogTitle sets isComplete = 0 when a quest is incomplete.

This caused the arrow to still point to requiredSourceItems (like
the Cold Iron Key drops) because the quest was never being flagged
as 'isComplete=true' internally.

Fixed by explicitly checking for isComplete == 1 or isComplete == -1
before returning.
---
 Database/QuestieDB.lua | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Database/QuestieDB.lua b/Database/QuestieDB.lua
index a530217..70ee71f 100644
--- a/Database/QuestieDB.lua
+++ b/Database/QuestieDB.lua
@@ -1083,7 +1083,9 @@ function QuestieDB.IsComplete(questId)
     --]]
 
     if not questLogEntry then return 0 end
-    if questLogEntry.isComplete then return questLogEntry.isComplete end
+    if questLogEntry.isComplete == 1 or questLogEntry.isComplete == -1 then
+        return questLogEntry.isComplete
+    end
 
     local objectives = questLogEntry.objectives
     if not objectives or #objectives == 0 then