From 1248d7a1f6134181d064331fc2616895eba59994 Mon Sep 17 00:00:00 2001
From: Xurkon <36556990+Xurkon@users.noreply.github.com>
Date: Sat, 21 Feb 2026 12:07:07 -0600
Subject: [PATCH] Fix Arrow pointing to completed objectives instead of
 finisher

QuestieArrow._CollectObjective was only checking objective.Completed
to determine if it should skip an objective. In WotLK, the server often
doesn't send the completed flag until turn-in time.

This caused the arrow to still pick up the kill/interaction spawns (like
the 5/5 Goblin Prisoners) because their Completed flag hadn't been
set, overpowering the finisher spawn.

Added a fallback check: if objective.Collected >= objective.Needed,
skip the objective.
---
 Modules/Arrow/QuestieArrow.lua | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Modules/Arrow/QuestieArrow.lua b/Modules/Arrow/QuestieArrow.lua
index 004dace..f1c0f9f 100644
--- a/Modules/Arrow/QuestieArrow.lua
+++ b/Modules/Arrow/QuestieArrow.lua
@@ -600,6 +600,13 @@ function QuestieArrow:UpdateNearestTargets()
                 return
             end
 
+            -- If the objective is numerically fulfilled but the server hasn't sent the Completed flag yet
+            if objective.Needed and objective.Collected and type(objective.Needed) == "number" and type(objective.Collected) == "number" then
+                if objective.Collected >= objective.Needed then
+                    return
+                end
+            end
+
             for _, spawnData in pairs(objective.spawnList) do
                 if spawnData and spawnData.Spawns then
                     for zone, spawns in pairs(spawnData.Spawns) do