Wuild / GatherLite

World of Warcraft: Classic Gathering addon keeping track on where you have found stuff
MIT License
31 stars 15 forks source link

LUA error on checking old nodes #51

Closed Daribon closed 5 years ago

Daribon commented 5 years ago

Describe the bug When hovering a node collected from older version of the addon, LUA error is produced and it doesn't display what player collected it. The error seems to appear on every single node collected from around version 1.1.5. Do note that it doesn't appear on herbs collected on version 1.1.7.

The following error will appear:

Message: Interface\AddOns\GatherLite\methods.lua:497: attempt to index field 'player' (a nil value) Time: Fri Sep 6 13:39:32 2019 Count: 143 Stack: Interface\AddOns\GatherLite\methods.lua:497: attempt to index field 'player' (a nil value) Interface\AddOns\GatherLite\methods.lua:497: in function <Interface\AddOns\GatherLite\methods.lua:484>

Locals: (temporary) = nil (temporary) =

{ [Fadeleaf] =
{ } } (temporary) = "[Fadeleaf]" (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = GatherLiteTooltip { TopOverlay = { } updateTooltip = 0.200000 needsReset = true BottomOverlay = { } comparing = false 0 = } (temporary) = "[Fadeleaf]" (temporary) = "x2" (temporary) = (temporary) = "22" (temporary) = ":" (temporary) = "36" (temporary) = ":" (temporary) = "12" (temporary) = "" (temporary) = 12 (temporary) = "attempt to index field 'player' (a nil value)" _GatherLite =
{ spellIDs =
{ } frames =
{ } classColours =
{ } UpdateInterval = 1 defaultConfigs =
{ } showingTooltip = true version = "1.2.0" minimapFrame = { } minimap = GatherLite_MinimapButton { } tooltip = GatherLiteTooltip { } updateMiniMapPosition = defined @Interface\AddOns\GatherLite\GatherLite.lua:46 mainFrame = { } tracker =
{ } showTooltip = defined @Interface\AddOns\GatherLite\GatherLite.lua:26 gatherSpellRanges =
{ } name = "GatherLite" nodes =
{ } isClassic = true hideTooltip = defined @Interface\AddOns\GatherLite\GatherLite.lua:40 TimeSinceLastUpdate = 0.542000 } f = herbalismworldmap248 { 0 = texture = { } } node =
{ GUID = "Player-4476-014C6A53" type = "herbalism" date =
{ } name = "Fadeleaf" position =
{ } target = "fadeleaf" loot =
{ } icon = 134193 spellID = 2366 } lootTable = true

To Reproduce Steps to reproduce the behavior:

  1. Go to map
  2. Hover over an old collected node.
  3. See error

Screenshots Wow_bdUwfHFXA3

Wuild commented 5 years ago

please try 1.2.1 and report back

Daribon commented 5 years ago

It's fixed in 1.2.1, thanks.