Introduces a universal bidirectional cross-link engine in QuestieLearner that automatically stitches relationships between all four entity types (NPCs, Quests, Objects, Items) as data is learned — no manual wiring needed. Fixes the tracker's persistent "Unknown Zone" header for custom/unknown quests by replacing unreliable map API calls with the canonical 3.3.5a quest log header walk. Fixes a logic inversion in UntrackQuestId that prevented shift-click untacking from working.
+ +_AddToArray(tbl, key, value, ovrTable, ovrId) adds a value to an array field and mirrors it to the live *DataOverrides table in the same call; _AddToNestedArray does the same for two-level nested arrays (used for quest starters/finishers sub-slots quest[2][1], quest[2][2], etc.); _AddToQuestObjective inserts {entityId, text} pairs into quest[10][slot] with the same dual-write pattern. Every write is idempotent — a no-op if the value is already present. All three primitives write to both learnedData (SavedVariables, persistent) and the live *DataOverrides tables (in-memory, immediate effect on map pins and tooltips).learnedData.npcs. Performs three scans across all learned quests: (a) if quest[2][1] contains this npcId → adds questId to npc[10] (questStarts); (b) if quest[3][1] contains this npcId → adds questId to npc[11] (questEnds); (c) walks quest[10][3] (item objectives) — for each item whose item[2] (npcDrops) already references this NPC, the NPC is injected into quest[10][1] as a creature objective so it receives map pins and tooltip kill-count text.learnedData.quests. Links all referenced entities in one pass: starter NPCs (quest[2][1]) → each known NPC's npc[10]; starter objects (quest[2][2]) → each known object's obj[2]; finisher NPCs (quest[3][1]) → npc[11]; finisher objects (quest[3][2]) → obj[3]; source item (quest[11]) → item[5] (startQuest key per itemDB schema); item drop chain: for each item in quest[10][3] whose item[2] lists known NPCs, those NPCs are injected into quest[10][1] as creature objectives.learnedData.objects. Scans all learned quests: if quest[2][2] references this objectId → adds questId to obj[2] (questStarts); if quest[3][2] references it → adds questId to obj[3] (questEnds).learnedData.items AND whenever a new drop-NPC relationship is added via LearnItemDrop. Two passes: (1) scan all learned quests for quest[11] == itemId → set item[5] = questId (this item starts that quest); (2) scan all learned quests for quest[10][3] entries matching itemId — for each such quest, every NPC in item[2] (drop sources) is injected into quest[10][1] as a creature objective. Practical example: accept a quest needing "Wolf Pelt" → wolf is killed and the pelt drops → LearnItemDrop fires → CrossLinkAfterItem runs → the wolf NPC is immediately added as a creature objective for the pelt quest with active map pins, no reload.npc[10/11] ↔ quest[2/3][1] bidirectionally. For typeSlot=2: stitches obj[2/3] ↔ quest[2/3][2]. For typeSlot=3: adds itemId to quest[2][3] (item starters sub-slot). Called by LearnQuestGiver for all typeSlots, replacing the previous NPC-only check.CrossLinkAfterItem(itemId) every time a new NPC is added to an item's drop list, not only on initial item creation. This retroactively propagates the drop→quest objective chain for previously learned items.CrossLinkAfterX function on first creation of the entity record. Subsequent updates to an existing record skip the cross-link scan (already linked on creation).[9] (which maps to itemLevel in QuestieDB.itemKeys) for quest-source storage. The correct key is [5] (startQuest). Affects CrossLinkAfterQuest and CrossLinkAfterItem.UnitReaction < 4 early-return guard that blocked learning hostile or neutral NPCs via mouseover. The existing QUESTGIVER NPC-flag bitmask check (0x02) remains as the primary filter, ensuring only NPCs with a quest-giver relationship are stored. Hostile quest objective NPCs are now captured on any hover, target, or kill and are correctly cross-linked to their quests.GetQuestLogZoneName(questId). Implements the only fully reliable zone resolution method in 3.3.5a: iterates the quest log from index 1 to GetNumQuestLogEntries() to locate the target quest's log index, then walks backwards from that index until an entry where isHeader=true is found. That header's title string is the zone name (e.g. "Shadowmoon Valley"). GetRealZoneText() and GetCurrentMapAreaID() were previously used but are unreliable because they reflect the currently viewed map, not the quest's zone. The 3.3.5a client places zone header entries directly above their child quests in the flat quest log list — this is guaranteed by the client and is the same method Blizzard's own UI uses to render the zone groups.GetRealZoneText() call with the inline header walk. The loop that already iterates the quest log to find the fallback quest's entry now continues backwards from that same index to find the header. GetAreaIdByZoneName() converts the string to an area ID for zoneOrSort; the raw string is stored as quest.zoneName for display. If no header is found (e.g. "Special" category with no zone header), zoneName remains nil and the tracker falls back to the _GetZoneName(zoneOrSort) path.QuestLogCache fallback objects (identified by _isLogFallback=true and absence of an IsComplete method) now calls GetQuestLogZoneName(capturedId) immediately after patching Objectives, SpecialObjectives, and ExtraObjectives. If a zone name is returned, both quest.zoneName and quest.zoneOrSort are set on the object in-place. Previously this block left both values at their QuestLogCache defaults (zoneOrSort=0, zoneName=nil), which caused every fallback quest to group under "Unknown Zone" in the tracker regardless of where the quest was accepted._fallbackQuests[qid] is now evicted and unconditionally rebuilt if the cached entry has no zoneName. This handles the race condition where BuildFallbackQuest was called during addon initialization before the quest log entries were populated, caching a zoneless entry that would persist for the entire session.UntrackQuestId previously set AutoUntrackedQuests[questId] = nil when the player untracks a quest. The tracker filter expression is not Questie.db.char.AutoUntrackedQuests[questId] — when the value is nil, not nil is true, meaning "show this quest". Clearing the entry on untrack therefore immediately re-showed the quest on the very next tracker update tick, making untacking appear completely broken. Fix: in autoTrackQuests mode, UntrackQuestId now sets AutoUntrackedQuests[questId] = true (evaluated as "hide"). In manual-track mode, it clears TrackedQuests[questId] as before, and AutoUntrackedQuests is cleared instead of being incorrectly toggled. The two tracking modes are now cleanly separated within the function.IsShiftKeyDown() and QuestLogFrame:IsShown() inside AQW_Insert was blocking the re-tracking path (clearing AutoUntrackedQuests[questId]) whenever the quest log frame was not open. Since untacking fires RemoveQuestWatch → UntrackQuestId directly, this AQW_Insert branch was only ever reached for re-tracking — yet the gate required the quest log to be open. The gate has been removed; re-tracking now works from any UI context.Rewrote QuestieLearner from scratch to fix all known data-capture deficiencies. Adds full quest field coverage, grid-based coordinate clustering, quest-giver-only mouseover filtering, async item-info retry, object loot detection, and live inject-on-import so imported data takes effect without a reload.