doadin / Baggins

zlib License
6 stars 7 forks source link

Opening bag error #79

Closed Weetbix813 closed 1 year ago

Weetbix813 commented 1 year ago

Error on opening bags:

33x Baggins/Baggins.lua:3130: attempt to compare number with nil [string "@Baggins/Baggins.lua"]:3130: in function UpdateItemButton' [string "@Baggins/Baggins.lua"]:1916: in functionReallyLayoutSection' [string "@Baggins/Baggins.lua"]:757: in function `?' [string "@Baggins/Baggins.lua"]:171: in function <Baggins/Baggins.lua:163>

Locals: self =

{ cannotDetachTooltip = true modules =
{ } CancelTimer = defined @DataStore/libs/AceTimer-3.0/AceTimer-3.0.lua:147 CreateItemButton = defined @Baggins/Baggins.lua:2509 GetIncludeRule = defined @Baggins/Baggins-Filtering.lua:462 RemoveSection = defined @Baggins/Baggins.lua:3850 GetCachedItem = defined @Baggins/Baggins-Filtering.lua:534 OptimizeSectionLayout = defined @Baggins/Baggins.lua:1564 UpdateSectionContents = defined @Baggins/Baggins.lua:1195 IsAnyBankOpen = defined @Baggins/Baggins.lua:4126 RemoveRule = defined @Baggins/Baggins.lua:3867 OnTextUpdate = defined @Baggins/Baggins.lua:3590 InitOptions = defined @Baggins/Baggins-Options.lua:1050 ScheduleTimer = defined @DataStore/libs/AceTimer-3.0/AceTimer-3.0.lua:94 UpdateDB = defined @Baggins/Baggins-Options.lua:1031 GetBagTypes = defined @Baggins/Baggins-Filtering.lua:89 UpdateItemButtons = defined @Baggins/Baggins.lua:2974 LayoutSection = defined @Baggins/Baggins.lua:1854 OnBankOpened = defined @Baggins/Baggins.lua:708 SetEnabledState = defined @DataStore/libs/AceAddon-3.0/AceAddon-3.0.lua:420 Hook = defined @Baggins/Libs/AceHook-3.0/AceHook-3.0.lua:273 RegisterSkin = defined @Baggins/Baggins-Skins.lua:32 InitBagCategoryOptions = defined @Baggins/Baggins-Options.lua:3560 UpdateLayout = defined @Baggins/Baggins.lua:2963 CountNormalSlots = defined @Baggins/Baggins.lua:3678 AddCustomRule = defined @Baggins/Baggins-Filtering.lua:101 RuleTypeIterator = defined @Baggins/Baggins-Filtering.lua:123 NewModule = defined @DataStore/libs/AceAddon-3.0/AceAddon-3.0.lua:231 MoveBag = defined @Baggins/Baggins.lua:3764 ForceFullRefresh = defined @Baggins/Baggins-Filtering.lua:430 DoRecheckCompress = defined @Baggins/Baggins-ItemOps.lua:224 GetAce3Opts = defined @Baggins/Baggins-Filtering.lua:152 GetName = defined @DataStore/libs/AceAddon-3.0/AceAddon-3.0.lua:275 MoveRule = defined @Baggins/Baggins.lua:3799 defaultcategories =
{ } ClearSectionCaches = defined @Baggins/Baggins.lua:994 Disable = defined @DataStore/libs/AceAddon-3.0/AceAddon-3.0.lua:310 CategoryMatchRemoved = defined @Baggins/Baggins.lua:866 Unhook = defined @Baggins/Libs/AceHook-3.0/AceHook-3.0.lua:417 CloseSpecialWindows = defined @Baggins/Baggins.lua:4160 CreateBagPlacementFrame = defined @Baggins/Baggins.lua:1973 IncludeItemInCategory = defined @Baggins/Baggins-Filtering.lua:494 Debug = defined @Baggins/Baggins.lua:114 CategoryMatchAdded = defined @Baggins/Baggins.lua:806 FireSignal = defined @Baggins/Baggins.lua:194 IsActive = defined @Baggins/Baggins.lua:387 CountAmmoSlots = defined @Baggins/Baggins.lua:3682 SlotMoved = defined @Baggins/Baggins.lua:905 SetDefaultModuleLibraries = defined @DataStore/libs/AceAddon-3.0/AceAddon-3.0.lua:363 Baggins_CategoriesChanged = <func

doadin commented 1 year ago

Is this retail or wrath or classic?

Weetbix813 commented 1 year ago

This is retail!

doadin commented 1 year ago

Thank you! I have not run into this issue myself, however I have pushed an update out just now v4.6.1. This version should hopefully fix this error. This error has to do with quality colors on items, if you notice any issue with items not having the proper quality color please let me know, and I will look into it further.

Weetbix813 commented 1 year ago

Thank you! seems to have fixed, I'll keep an eye on the coloring :)

doadin commented 1 year ago

Thank you! I am going to close this, but If you need anything else feel free to reopen this issue or make another!