Bug report

Question

Bug report

Rezy942 opened this issue 2 years ago · 15 comments

Rezy942 commented 2 years ago

Hover over elemental storm in obsidian hold - spams constantly:

33x HandyNotes_Dragonflight/common.lua:1027: attempt to index field '?' (a nil value)
[string "@HandyNotes_Dragonflight/common.lua"]:1027: in function <HandyNotes_Dragonflight/common.lua:1018>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function `TryShowTooltip'
[string "@Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua"]:120: in function <...zzard_SharedMapDataProviders/AreaPOIDataProvider.lua:113>

Locals:
self = Frame {
0 =
GetGlobalPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:310
PanTo = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:317
owningMap = FlightMapFrame {
}
zoomedInNudge = 0.250000
UseFrameLevelType = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:422
GetNudgeSourceZoomedInMagnitude = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:232
SetScaleStyle = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:358
GetNudgeSourcePinZoomedInNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:268
UpdateTooltip = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:118
GetMap = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:256
OnCanvasSizeChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:336
ApplyCurrentPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:393
areaPoiID = 7221
IsIgnoringGlobalPinScale = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:344
GetHighlightType = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:98
endScale = 1.200000
HighlightTexture = Texture {
}
OnMouseEnter = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:113
pinTemplate = "FlightMap_AreaPOIPinTemplate"
OnMouseUp = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:177
OnAcquired = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:72
PanAndZoomTo = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:322
ApplyCurrentAlpha = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:414
OnLoad = defined @Blizzard_FlightMap/FM_AreaPOIDataProvider.lua:10
TryShowTooltip = defined =[C]:-1
pinFrameLevel = "PIN_FRAME_LEVEL_AREA_POI"
OnReleased = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:147
startAlpha = 1
description = "A Primalist thunderstorm buffets Nokhudon Hold, imbuing creatures with new powers and strengths."
ClearNudgeSettings = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:198
OnCanvasPanChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:332
OnMouseLeave = defined =[C]:-1
SetNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:285
GetNudgeSourcePinZoomedOutNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:264
startScale = 1
OnClick = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:151
Texture = Texture {
}
GetNudgeSourceRadius = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:219
shouldGlow = false
GetPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:305
OnCanvasScaleChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:327
normalizedY = 0.527564
normalizedX = 0.357383
GetNudgeTargetFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:211
GetFrameLevelType = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:427
SetAlphaStyle = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:383
SetScalingLimits = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:348
pinFrameLevelType = "PIN_FRAME_LEVEL_AREA_POI"
Cr

Krovikan · Answer 1 · 2023-01-12T14:57:43.000Z

Hello.

Version of the addon?

Regards

Rezy942 · Answer 2 · 2023-01-12T15:16:59.000Z

HandyNotes_Dragonflight-v29

This error has been happening for a week or two at least, maybe more.
Unsure the cause.

Ioney · Answer 3 · 2023-01-12T19:46:48.000Z

is it still happening if you deactivate all other addons except HandyNotes and HandyNotes: Dragonflight?

Rezy942 · Answer 4 · 2023-01-14T13:38:02.000Z

Tested with ONLY handynotes + handynotes dragonflight + bugsack + buggrabber
I can reproduce the bug easily by talking to flight path NPC in valdraken and hovering over the elemental storm icon in any zone
Hope you can find a fix for this. THnaks

64x HandyNotes_Dragonflight/common.lua:1027: attempt to index field '?' (a nil value)
[string "@HandyNotes_Dragonflight/common.lua"]:1027: in function <HandyNotes_Dragonflight/common.lua:1018>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function `TryShowTooltip'
[string "@Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua"]:120: in function <...zzard_SharedMapDataProviders/AreaPOIDataProvider.lua:113>

Locals:
self = Frame {
0 =
GetGlobalPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:310
PanTo = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:317
owningMap = FlightMapFrame {
}
zoomedInNudge = 0.250000
UseFrameLevelType = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:422
GetNudgeSourceZoomedInMagnitude = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:232
SetScaleStyle = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:358
GetNudgeSourcePinZoomedInNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:268
UpdateTooltip = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:118
GetMap = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:256
OnCanvasSizeChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:336
ApplyCurrentPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:393
areaPoiID = 7221
IsIgnoringGlobalPinScale = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:344
GetHighlightType = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:98
endScale = 1.200000
HighlightTexture = Texture {
}
OnMouseEnter = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:113
pinTemplate = "FlightMap_AreaPOIPinTemplate"
OnMouseUp = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:177
OnAcquired = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:72
PanAndZoomTo = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:322
ApplyCurrentAlpha = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:414
OnLoad = defined @Blizzard_FlightMap/FM_AreaPOIDataProvider.lua:10
TryShowTooltip = defined =[C]:-1
pinFrameLevel = "PIN_FRAME_LEVEL_AREA_POI"
OnReleased = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:147
startAlpha = 1
description = "A Primalist thunderstorm buffets Nokhudon Hold, imbuing creatures with new powers and strengths."
ClearNudgeSettings = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:198
OnCanvasPanChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:332
OnMouseLeave = defined =[C]:-1
SetNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:285
GetNudgeSourcePinZoomedOutNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:264
startScale = 1
OnClick = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:151
Texture = Texture {
}
GetNudgeSourceRadius = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:219
shouldGlow = false
GetPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:305
OnCanvasScaleChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:327
normalizedY = 0.527564
normalizedX = 0.357383
GetNudgeTargetFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:211
GetFrameLevelType = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:427
SetAlphaStyle = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:383
SetScalingLimits = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:348
pinFrameLevelType = "PIN_FRAME_LEVEL_AREA_POI"
Cr

Dairyman · Answer 5 · 2023-01-14T16:31:08.000Z

Problem is the flight map shows the Storms and uses the mapID 2057 which is not listed in ELEMENTAL_STORM_MOB_ACHIVEMENTS. Plus to support the flight map correctly would need to associate the areaPoiID to the zones they belong too.

wyldclaw · Answer 6 · 2023-01-14T19:10:43.000Z

So the easiest fix would be if stormType and not mapID == 2057 and group:GetDisplay(mapID) then that would just disable the Elemental Storm integration on the Dragon Isles flight/continent map.

The alternate fix would be to add mapIDs to ELEMENTAL_STORM_AREA_POIS and do some more checking.

Ioney · Answer 7 · 2023-01-16T21:15:22.000Z

if stormType and not FlightMapFrame:IsShown() and group:GetDisplay(mapID) then

is probably a more general solution

Corefanatik · Answer 8 · 2023-01-18T15:06:34.000Z

nice nice ;)
thx for the quick response.good day all.

Corefanatik · Answer 9 · 2023-01-18T14:03:33.000Z

Hi .is there an fix for this in work? its the only lua error that i had for some weeks now. or I disable it before I get too used to it now. ;)

wyldclaw · Answer 10 · 2023-01-18T14:47:45.000Z

@Corefanatik yes, this will be fixed in the next release!

Rezy942 · Answer 11 · 2023-01-18T16:59:49.000Z

Thank you for getting on top of this so quick!

Corefanatik · Answer 12 · 2023-01-24T04:29:44.000Z

Closed?? im on lua error 1596 ;)

1596x HandyNotes_Dragonflight/common.lua:1027: attempt to index field '?' (a nil value)
[string "@HandyNotes_Dragonflight/common.lua"]:1027: in function <HandyNotes_Dragonflight/common.lua:1018>
[string "=[C]"]: in function `TryShowTooltip'
[string "@Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua"]:120: in function <...zzard_SharedMapDataProviders/AreaPOIDataProvider.lua:113>

Locals:
self = Frame {
0 =
GetGlobalPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:310
PanTo = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:317
owningMap = FlightMapFrame {
}
zoomedInNudge = 0.250000
UseFrameLevelType = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:422
GetNudgeSourceZoomedInMagnitude = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:232
SetScaleStyle = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:358
GetNudgeSourcePinZoomedInNudgeFactor = defined @Blizzard_MapCanvas/
MapCanvas_DataProviderBase.lua:232
SetScaleStyle = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:358
GetNudgeSourcePinZoomedInNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:268
UpdateTooltip = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:118
GetMap = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:256
OnCanvasSizeChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:336
ApplyCurrentPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:393
areaPoiID = 7254
IsIgnoringGlobalPinScale = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:344
GetHighlightType = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:98
endScale = 1.200000
HighlightTexture = Texture {
}
OnMouseEnter = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:113
pinTemplate = "FlightMap_AreaPOIPinTemplate"
OnMouseUp = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:177
OnAcquired = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:72
PanAndZoomTo = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:322
ApplyCurrentAlpha = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:414
OnLoad = defined @Blizzard_FlightMap/FM_AreaPOIDataProvider.lua:10
TryShowTooltip = defined =[C]:-1
pinFrameLevel = "PIN_FRAME_LEVEL_AREA_POI"
OnReleased = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:147
startAlpha = 1
description = "Ein Sandsturm der Primalisten hüllt den Schlackensumpf ein und erfüllt Kreaturen mit neuer Macht und Kräften."
ClearNudgeSettings = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:198
OnCanvasPanChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:332
OnMouseLeave = defined @Blizzard_SharedMapDataProviders/AreaPOIDataProvider.lua:174
SetNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:285
GetNudgeSourcePinZoomedOutNudgeFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:264
startScale = 1
OnClick = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:151
Texture = Texture {
}
GetNudgeSourceRadius = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:219
shouldGlow = false
GetPosition = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:305
OnCanvasScaleChanged = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:327
normalizedY = 0.349469
normalizedX = 0.434018
GetNudgeTargetFactor = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:211
GetFrameLevelType = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:427
SetAlphaStyle = defined @Blizzard_MapCanvas/MapCanvas_DataProviderBase.lua:383
SetScalingLimits = defined @Blizzard_MapCanvas/

Krovikan · Answer 13 · 2023-01-24T10:43:30.000Z

Hello:

The Collaborator close the Issue because the fix is working well in our test. Only remains release the v30 asap.

Regards

Ioney · Answer 14 · 2023-01-24T11:23:52.000Z

reopened to inform everybody that it is fixed in v30 which will be released asap.

Ioney · Answer 15 · 2023-01-25T10:05:35.000Z

v30 is now released.

Share to