Cidan / BetterBags

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

Ghost items after selling items with Dejunk Addon #219

Closed TuttiRH closed 8 months ago

TuttiRH commented 8 months ago

Addon Version

v0.1.7-5-ga690f71

World of Warcraft Version

Retail

Describe the bug

Had a full inventory and i'm selling items automatic with the Addon "Dejunk".

Two times these two LUA Errors happens today with the new Alpha. But not everytime i'm selling items.

Steps to reproduce

1x BetterBags-Appearances/main.lua:318: bad argument #1 to 'upper' (string expected, got nil) string "=[C]": in function upper' [string "@BetterBags-Appearances/main.lua"]:318: in functionisUsableByCurrentClass' [string "@BetterBags-Appearances/main.lua"]:353: in function func' [string "@BetterBags/data/categories.lua"]:137: in functionGetCustomCategory' [string "@BetterBags/frames/item.lua"]:379: in function GetCategory' [string "@BetterBags/views/gridview.lua"]:87: in functionRender' [string "@BetterBags/frames/bag.lua"]:213: in function Draw' [string "@BetterBags/core/init.lua"]:141: in functioncb' [string "@BetterBags/core/events.lua"]:45: in function <BetterBags/core/events.lua:40>

[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"]:148: in function <BetterBags/core/events.lua:147>

Locals: (temporary) = nil (temporary) = "string expected, got nil"


35x BetterBags/views/gridview.lua:154: attempt to index field '?' (a nil value) [string "@BetterBags/views/gridview.lua"]:154: in function Render' [string "@BetterBags/frames/bag.lua"]:213: in functionDraw' [string "@BetterBags/core/init.lua"]:141: in function `cb' [string "@BetterBags/core/events.lua"]:45: in function <BetterBags/core/events.lua:40>

[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"]:148: in function <BetterBags/core/events.lua:147>

Locals: view =

{ Render = defined @BetterBags/views/gridview.lua:61 itemCount = 94 itemsByBagAndSlot =
{ } sections =
{ } defer = false freeReagentSlot =
{ } freeSlot =
{ } kind = 2 Wipe = defined @BetterBags/views/gridview.lua:35 content =
{ } } bag =
{ menuList =
{ } searchBox =
{ } sections =
{ } toRelease =
{ } currentView =
{ } slots =
{ } toReleaseSections =
{ } currencyFrame =
{ } moneyFrame =
{ } isReagentBank = false drawOnClose = false views =
{ } currentItemCount = 0 kind = 0 bottomBar = Frame { } frame = BetterBagsBagBackpack { } drawAfterCombat = false resizeHandle = Button { } } dirtyItems =
{ 1 =
{ } 2 =
{ } 3 =
{ } 4 =
{ } 5 =
{ } 6 =
{ } 7 =
{ } 8 =
{ } 9 =
{ } 10 =
{ } 11 =
{ } 12 =
{ } 13 =
{ } 14 =
{ } 15 =
{ } 16 =
{ } 17 =
{ } 18 =
{ } 19 =
{ } } sizeInfo =
{ itemsPerRow = 15 columnCount = 1 height = 638.419067 opacity = 100 scale = 100 width = 739.205383 } categoryChanged = true extraSlotInfo =
{ freeReagentSlotKey = "" emptySlotByBagAndSlot =
{ } emptySlots = 16 emptyReagentSlots = 0 totalItems = 92 freeSlotKey = "4_28" } (for generator) = defined =[C]:-1 (for state) =
{ Armor - Cloth =
{ } Gem - Artifact Relic =
{ } Battle Pet =
{ } Armor - Miscellaneous =
{ } Consumable - Flasks & Phials =
{ } Armor - Leather =
{ } Free Space =
{ } Unknown - Demon Hunter =
{ } Quest - Quest =
{ } Armor - Mail =
{ } Armor - Plate =
{ } Hearthstones =
{ } Junk =
{ } } (for control) = "Armor - Leather" sectionName = "Armor - Leather" section =
{ view = 2 title = FontString { } fillWidth = false frame = Frame { } content =
{ } } (for generator) = defined =[C]:-1 (for state) =
{ 3_2 =
{ } 4_11 =
{ } 3_13 =
{ } } (for control) = "3_2" slotkey = "32" =
{ NormalTexture = BetterBagsItemButton524NormalTexture { } Stock = BetterBagsItemButton524Stock { } LockTexture = BetterBagsItemButton524LockButton { } isFreeSlot = false ilvlText = FontString { } Count = BetterBagsItemButton524Count { } button = BetterBagsItemButton524 { } IconQuestTexture = BetterBagsItemButton524IconQuestTexture { } frame = BetterBagsItemButton524parent { } Cooldown = BetterBagsItemButton524Cooldown { } IconTexture = BetterBagsItemButton524IconTexture { } } (temporary) = nil (temporary) =
{ NormalTexture = BetterBagsItemButton524NormalTexture { } Stock = BetterBagsItemButton524Stock { } LockTexture = BetterBagsItemButton524LockButton { } isFreeSlot = false ilvlText = FontString { } Count = BetterBagsItemButton524Count { } button = BetterBagsItemButton524 { } IconQuestTexture = BetterBagsItemButton524IconQuestTexture { } frame = BetterBagsItemButton524parent { } Cooldown = BetterBagsItemButton524Cooldown { } IconTexture = BetterBagsItemButton524IconTexture { } } (temporary) = 49 (temporary) = 60.999985 (temporary) = FontString { 0 = } (temporary) = 17.999992 (temporary) = (temporary) = 17.999992 (temporary) = 0 (temporary) = "attempt to index field '?' (a nil value)" database =
{ SetDefaultModuleLibraries = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:363 Migrate = defined @BetterBags/core/database.lua:293 ItemCategoryExists = defined @BetterBags/core/database.lua:230 SetItemLevelEnabled = defined @BetterBags/core/database.lua:120 GetItemLock = <fu

Cidan commented 8 months ago

Thanks for the report! I just pushed a new Alpha build that fixes this -- if this comes up again, please reopen this bug.

TuttiRH commented 8 months ago

Installed the new release v0.1.8 and get that Error again.

2x BetterBags-Appearances/main.lua:318: bad argument #1 to 'upper' (string expected, got nil) string "=[C]": in function upper' [string "@BetterBags-Appearances/main.lua"]:318: in functionisUsableByCurrentClass' [string "@BetterBags-Appearances/main.lua"]:353: in function func' [string "@BetterBags/data/categories.lua"]:137: in functionGetCustomCategory' [string "@BetterBags/frames/item.lua"]:379: in function GetCategory' [string "@BetterBags/views/gridview.lua"]:87: in functionRender' [string "@BetterBags/frames/bag.lua"]:213: in function Draw' [string "@BetterBags/core/init.lua"]:141: in functioncb' [string "@BetterBags/core/events.lua"]:45: in function <BetterBags/core/events.lua:40>

[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"]:148: in function <BetterBags/core/events.lua:147>

Locals: (temporary) = nil (temporary) = "string expected, got nil"

Cidan commented 8 months ago

This is related to the BetterBags-Apperance plugin, which is not written by us -- disable that addon and reach out to the author.