Nnoggie / MythicDungeonTools

World of Warcraft AddOn for planning and optimizing Mythic+ dungeon runs
GNU General Public License v2.0
331 stars 231 forks source link

Error when reordering pulls #506

Closed mabnp closed 11 months ago

mabnp commented 12 months ago

I get this error when I drag a pull to another position in the list, on mouse up (when dropping the pull in the new position). Happens every time. Doesn't seem to break anything, the addon works.

7x ...idgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:102: attempt to index local 'button' (a nil value)
[string "@MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:102: in function <...idgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:13>
[string "@MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:1035: in function `Drop'
[string "@MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:253: in function <...idgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:252>
[string "=[C]"]: in function `Hide'
[string "@MythicDungeonTools/libs/AceGUI-3.0-41/AceGUI-3.0.lua"]:176: in function `Release'
[string "@MythicDungeonTools/libs/AceGUI-3.0-41/AceGUI-3.0.lua"]:468: in function `ReleaseChildren'
[string "@MythicDungeonTools/MythicDungeonTools.lua"]:3671: in function <...aceMythicDungeonTools/MythicDungeonTools.lua:3664>
[string "=[C]"]: in function `xpcall'
[string "@MythicDungeonTools/Modules/ErrorHandling.lua"]:247: in function `ReloadPullButtons'
[string "@MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:1118: in function `Drop'
[string "@MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:253: in function <...idgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:252>

Locals:
scrollFrame = <table> {
 OnRelease = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:48
 OnWidthSet = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:152
 parent = <table> {
 }
 OnAcquire = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:43
 userdata = <table> {
 }
 scrollBarShown = true
 scrollframe = ScrollFrame {
 }
 OnHeightSet = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:158
 type = "ScrollFrame"
 AceGUIWidgetVersion = 26
 LayoutFunc = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/AceGUI-3.0.lua:675
 base = <table> {
 }
 LayoutFinished = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:134
 events = <table> {
 }
 FixScroll = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:91
 children = <table> {
 }
 SetStatusTable = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:144
 MoveScroll = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:77
 frame = Frame {
 }
 SetScroll = <function> defined @MythicDungeonTools/libs/AceGUI-3.0/widgets/AceGUIContainer-ScrollFrame.lua:59
 localstatus = <table> {
 }
 content = Frame {
 }
 scrollbar = AceConfigDialogScrollFrame6ScrollBar {
 }
}
buttonList = <table> {
 1 = <table> {
 }
 2 = <table> {
 }
 3 = <table> {
 }
 4 = <table> {
 }
 5 = <table> {
 }
 6 = <table> {
 }
 7 = <table> {
 }
 8 = <table> {
 }
 9 = <table> {
 }
 10 = <table> {
 }
 11 = <table> {
 }
 12 = <table> {
 }
 13 = <table> {
 }
 14 = <table> {
 }
 15 = <table> {
 }
}
id = nil
button = nil
pos = nil
offset = nil
scroll_value_min = 25
scroll_value_max = 975
scroll_value = 0
scroll_frame_height = 456.086731
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = <table> {
 index = 2
 color = <table> {
 }
 entered = false
 Drop = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1034
 background = Texture {
 }
 base = <table> {
 }
 percentageFontString = FontString {
 }
 AceGUIWidgetVersion = 1
 BackupColor = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1284
 ShowShroudedIcon = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1254
 RevertColor = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1292
 updateFunction = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:907
 Initialize = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:124
 ShowCountPerHealth = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1264
 CreateUpdateFunction = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:905
 UpdateColor = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1273
 Disable = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1131
 shroudedIcon = Texture {
 }
 countPerHealthFontString = FontString {
 }
 events = <table> {
 }
 SetMaxPulls = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1159
 ClearPick = <function> defined @MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:1149
 OnAcquire = <function> defi
mabnp commented 12 months ago

When I do it frequently then bugsack doesn't catch the error and MDT error popup appears:

ReloadPullButtons: ...ns/MythicDungeonTools/libs/AceGUI-3.0/AceGUI-3.0.lua:173: attempt to index local 'widget' (a nil value)

17/07/23 11:04:13
MDT: 4.0.4.16
Client: 100105 enUS
Character: Hukd-BurningLegion (Europe)

Route:
!Db1YsnmmmWFPgslpUcCPxmDWCwdjPR6yW1MjrUuUW3osjPpgUOX7QDxzjQIEJ8Bdmh6kr5hYvTG8Fd850RdnXcmjDL(EKKnLyKC1NX(sBehGYvr(berNeYj1GIvzYzHpxs7qoTE7rYTs1ka7dPDul57dS8cZdqggnUqRIz9lDytuxvC3rUBi39w8UvKBj5UvfRCtTRghAoM7jpZ1G5tDpXACpaZ5fhAyxaZWRtPgJ8I9XpQ1xZfb)(UDIgvTPjcrW6eNnNnfj)KXBR4J)tJDcwArLB)qVxdZRl93

Stacktraces

ReloadPullButtons:
[string "@Interface/AddOns/MythicDungeonTools/Modules/ErrorHandling.lua"]:252: in function `ReloadPullButtons'
[string "@Interface/AddOns/MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:1118: in function `Drop'
[string "@Interface/AddOns/MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:253: in function <...idgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:252>
[string "=[C]"]: in function `Hide'
[string "@Interface/AddOns/MythicDungeonTools/libs/AceGUI-3.0/AceGUI-3.0.lua"]:176: in function `Release'
[string "@Interface/AddOns/MythicDungeonTools/libs/AceGUI-3.0/AceGUI-3.0.lua"]:468: in function `ReleaseChildren'
[string "@Interface/AddOns/MythicDungeonTools/MythicDungeonTools.lua"]:3671: in function <...ace/AddOns/MythicDungeonTools/MythicDungeonTools.lua:3664>
[string "=[C]"]: in function `xpcall'
[string "@Interface/AddOns/MythicDungeonTools/Modules/ErrorHandling.lua"]:247: in function `ReloadPullButtons'
[string "@Interface/AddOns/MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:1118: in function `Drop'
[string "@Interface/AddOns/MythicDungeonTools/AceGUIWidgets/AceGUIWidget-MythicDungeonToolsPullButton.lua"]:253: in function <...idgets/AceGUIWidget-MythicDungeonToolsPullButton.lua:252>
Nnoggie commented 11 months ago

fixed in v4.0.4.29