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

@@ -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