Cidan / BetterBags

A total replacement AddOn for World of Warcraft bag frames, inspired by AdiBags.
MIT License
88 stars 39 forks source link

recent items #68

Closed deathmessinger closed 11 months ago

deathmessinger commented 11 months ago

The recent item are not going to their expected location after mouse over and closing bags I am also getting this error (which I think might have something to do with it)

4x BetterBags/views/gridview.lua:72: attempt to index local 'oldSection' (a nil value) [string "@BetterBags/views/gridview.lua"]:72: in function GridView' [string "@BetterBags/frames/bag.lua"]:194: in functionDraw' [string "@BetterBags/core/init.lua"]:120: in function `cb' [string "@BetterBags/core/events.lua"]:41: in function <BetterBags/core/events.lua:36>

[string "@TomTom/libs/CallbackHandler-1.0-8/CallbackHandler-1.0.lua"]:19: in function <...Tom/libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:15> [string "@TomTom/libs/CallbackHandler-1.0-8/CallbackHandler-1.0.lua"]:54: in function SendMessage' [string "@BetterBags/core/events.lua"]:94: in functionSendMessage' [string "@BetterBags/data/items.lua"]:132: in function <BetterBags/data/items.lua:126> string "=[C]": in function `xpcall' [string "@FrameXML/ObjectAPI/ContinuableContainer.lua"]:83: in function <..FrameXML/ObjectAPI/ContinuableContainer.lua:75>

[string "@BetterBags/data/items.lua"]:126: in function ProcessContainer' [string "@BetterBags/data/items.lua"]:120: in functionRefreshBackpack' [string "@BetterBags/data/items.lua"]:64: in function RefreshAll' [string "@BetterBags/data/items.lua"]:48: in functioncb' [string "@BetterBags/core/events.lua"]:81: in function <BetterBags/core/events.lua:76>

Locals: self =

{ SetDefaultModuleLibraries = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:363 Enable = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:290 NewModule = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:231 EnableModule = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:328 modules =
{ } GetModule = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:206 IterateEmbeds = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:438 IsModule = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:213 defaultModuleLibraries =
{ } baseName = "BetterBags" ListView = defined @BetterBags/views/listview.lua:43 SetEnabledState = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:420 UpdateListSize = defined @BetterBags/views/listview.lua:27 enabledState = false OneBagView = defined @BetterBags/views/oneview.lua:23 SetDefaultModuleState = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:383 moduleName = "Views" GetName = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:275 defaultModuleState = true Disable = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:310 IterateModules = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:433 name = "BetterBags_Views" SetDefaultModulePrototype = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:405 IsEnabled = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:447 orderedModules =
{ } GridView = defined @BetterBags/views/gridview.lua:23 DisableModule = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:346 } bag =
{ menuList =
{ } bottomBar = Frame { } sections =
{ } toRelease =
{ } recentItems =
{ } toReleaseSections =
{ } freeBagSlotsButton =
{ } moneyFrame =
{ } isReagentBank = false drawOnClose = false frame = BetterBagsBagBackpack { } slots =
{ } freeSlots =
{ } currentItemCount = 14 freeReagentBagSlotsButton =
{ } itemsByBagAndSlot =
{ } kind = 0 content =
{ } resizeHandle = Button { } } dirtyItems =
{ 1 =
{ } 2 =
{ } 3 =
{ } 4 =
{ } 5 =
{ } 6 =
{ } 7 =
{ } 8 =
{ } 9 =
{ } 10 =
{ } 11 =
{ } 12 =
{ } 13 =
{ } 14 =
{ } 15 =
{ } 16 =
{ } 17 =
{ } 18 =
{ } 19 =
{ } 20 =
{ } 21 =
{ } 22 =
{ } 23 =
{ } 24 =
{ } 25 =
{ } 26 =
{ } 27 =
{ } 28 =
{ } 29 =
{ } 30 =
{ } 31 =
{ } 32 =
{ } 33 =
{ } 34 =
{ } 35 =
{ } 36 =
{ } 37 =
{ } 38 =
{ } 39 =
{ } 40 =
{ } 41 =
{ } 42 =
{ } 43 =
{ } 44 =
{ } 45 =
{ } 46 =
{ } 47 =
{ } 48 =
{ } 49 =
{ } 50 =
{ } 51 =
{ } 52 =
{ } 53 =
{ } 54 =
{ } 55 =
{ } 56 =
{ } 57 =
{ } 58 =
{ } 59 =
{ } 60 =
{ } 61 =
{ } 62 =
{ } 63 =
{ } 64 =
{ } 65 =
{ } 66 =
{ } 67 =
{ } 68 =
{ } 69 =
{ } 70 =
{ } 71 =
{

Cidan commented 11 months ago

The recent item are not going to their expected location after mouse over and closing bags

Mousing over and closing does not currently redraw, this is known and pending a rework of how recent items works, outlined in the other github issues.

The error is definitely not great -- I may not be able to get to this tonight, but it's top priority tomorrow.

deathmessinger commented 11 months ago

bbissue.txt

I just had it pop again by equipping a piece of gear I just got as a quest reward

Cidan commented 11 months ago

Fixed in .79 I believe -- lmk if you can reproduce.