SavedInstances

SavedInstances

13M Downloads

Daily/Weekly Quest Tooltip Bug

Roddd opened this issue ยท 1 comments

commented

local function ShowQuestTooltip has some bugs due to zone/map changes I think. One particular weirdness I've seen is that quest info returns inconsistent information. Check out this dump of the qi table.

[00:40:55] { ["isDaily"] = true,["Title"] = Blingtron 4000,["Link"] = [Blingtron 4000],}

[00:40:55] { ["isDaily"] = true,["Zone"] = { ["mapType"] = 4,["mapID"] = 341,["name"] = Black Temple,["parentMapID"] = 339,} ,["Title"] = Blingtron 5000,["Link"] = [Blingtron 5000],}

[00:40:55] { ["isDaily"] = true,["Zone"] = { ["mapType"] = 3,["mapID"] = 23,["name"] = Eastern Plaguelands,["parentMapID"] = 13,} ,["Title"] = Blingtron 6000,["Link"] = [Blingtron 6000],}

A few things to note:

  1. No one cares about the zone for Blingtron. Doesn't even make sense that it has any. My first attempt at fixing it ended up listing the zones. This should probably be filtered out by exception.
  2. Blingtron 4000 quest is not returning with a Zone element on home PC. Inconsistent.
  3. The list of zones associated were even more different when I tried this at work. It listed some of the BfA zones as the zone for Blingtron. At work PC, it was showing a Zone for Blingtron 4000 quest. What?

My PR should fix these things. One bug I noticed was the weekly [Sealing Fates: Order Resources] lists the zone as Kul Tiras when it should be Dalaran. I don't know enough about how that is being obtained yet.

commented

PR: #129