Jaliborc / Bagnon

Single window displays for you items
200 stars 112 forks source link

wotlk classic: Pressing clean on first session in "player bank, burst out error" #1552

Open wazerstar opened 1 year ago

wazerstar commented 1 year ago

Running Software (issues missing this information will be deleted):

Describe the bug Gives error

To Reproduce Steps to reproduce the behaviour:

  1. Go to 'Player personal bank'
  2. Click on 'Click to clean items.'
  3. See error

Expected behaviour Sort items without errors popping up

Screenshots If applicable, add screenshots to help explain your problem.

Error Logs

1x ...rface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:178: attempt to compare nil with number
[string "@Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua"]:178: in function <...rface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:175>
[string "=[C]"]: in function `sort'
[string "@Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua"]:156: in function `GetOrder'
[string "@Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua"]:69: in function `Iterate'
[string "@Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua"]:35: in function `Run'
[string "@Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua"]:29: in function `Start'
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\frame.lua"]:92: in function <...ace\AddOns\Bagnon\common\Wildpants\classes\frame.lua:91>
[string "=(tail call)"]: ?
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\bank-Bank.lua"]:29: in function `SortItems'
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\sortButton.lua"]:39: in function <...ddOns\Bagnon\common\Wildpants\classes\sortButton.lua:30>

Locals:
a = <table> {
 filtered = false
 class = 4
 worthless = false
 iconFileID = 133291
 hyperlink = "|cff0070dd|Hitem:37683::::::::80:::::::::|h[Necromancer's Amulet]|h|r"
 level = 200
 icon = 133291
 quality = 3
 set = 1
 hasLoot = false
 count = 1
 isFiltered = false
 lootable = false
 bind = 1
 crafting = false
 equip = "INVTYPE_NECK"
 space = <table> {
 }
 subclass = 0
 stack = 1
 price = 53434
 link = "|cff0070dd|Hitem:37683::::::::80:::::::::|h[Necromancer's Amulet]|h|r"
 id = 37683
 name = "Necromancer's Amulet"
 stackCount = 1
 isReadable = false
 hasNoValue = false
 readable = false
 itemID = 37683
 minLevel = 80
 family = 0
 expac = 254
 isLocked = false
 locked = false
}
b = <table> {
 space = <table> {
 }
 subclass = 0
 set = 1
 filtered = false
 class = 4
 worthless = false
 iconFileID = 133339
 lootable = false
 stackCount = 1
 icon = 133339
 link = "|cffffffff|Hitem:43285::::::::80:::::::::|h[]|h|r"
 hasLoot = false
 equip = "INVTYPE_NECK"
 locked = false
 isReadable = false
 hasNoValue = false
 isLocked = false
 count = 1
 id = 43285
 family = 0
 isFiltered = false
 readable = false
 hyperlink = "|cffffffff|Hitem:43285::::::::80:::::::::|h[]|h|r"
}
(for generator) = <function> defined =[C]:-1
(for state) = <table> {
 1 = "set"
 2 = "class"
 3 = "subclass"
 4 = "equip"
 5 = "quality"
 6 = "icon"
 7 = "level"
 8 = "name"
 9 = "id"
 10 = "count"
}
(for control) = 5
_ = 5
prop = "quality"
(*temporary) = 3
(*temporary) = nil
(*temporary) = "attempt to compare nil with number"
Sort = <table> {
 SendSignal = <function> defined @Interface\AddOns\BagBrother\Libs\WildAddon-1.0\WildAddon-1.0.lua:82
 CanRun = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:163
 RegisterMessage = <function> defined @Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:90
 NewModule = <function> defined @Interface\AddOns\BagBrother\Libs\WildAddon-1.0\WildAddon-1.0.lua:68
 GetOrder = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:142
 Proprieties = <table> {
 }
 owner = "Vazka"
 bags = <table> {
 }
 GetFamilies = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:127
 SendMessage = <function> defined @Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:59
 Delaying = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\libs\MutexDelay-1.0\MutexDelay-1.0.lua:46
 Rule = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:175
 UnregisterSignal = <function> defined @Interface\AddOns\BagBrother\Libs\WildAddon-1.0\WildAddon-1.0.lua:78
 FitsIn = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:167
 Move = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:188
 GetSpaces = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:109
 RegisterSignal = <function> defined @Interface\AddOns\BagBrother\Libs\WildAddon-1.0\WildAddon-1.0.lua:74
 UnregisterAllMessages = <function> defined @Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:174
 Run = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:32
 UnregisterAllEvents = <function> defined @Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:174
 Iterate = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:41
 UnregisterMessage = <function> defined @Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:153
 Stop = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:102
 RegisterEvent = <function> defined @Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:90
 Start = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\api\sorting.lua:22
 Delay = <function> defined @Interface\AddOns\Bagnon\common\Wildpants\libs\MutexDelay-1.0\MutexDelay-1.0.lua:27
 UnregisterEvent = <function> defined @Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:153
 Tag = "BAGNON_"
}

Additional context Clicking first time gives error, clicking second time starts the sorting.

wazerstar commented 1 year ago

More errors when clicking in bank on other chars bank search field

25x bad argument #1 to '?' (Usage: local itemLink = C_Item.GetItemLink(itemLocation))
[string "=[C]"]: in function `GetItemLink'
[string "@Interface\AddOns\BagBrother\libs\ItemSearch-1.3-1\API.lua"]:40: in function `Matches'
[string "@Interface\AddOns\BagBrother\addons\core\classes\item.lua"]:308: in function `?'
[string "@Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:119: in function <...las\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:119>
[string "=[C]"]: ?
[string "@Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:29: in function <...las\Libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:25>
[string "@Interface\AddOns\Atlas\Libs\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:64: in function `SendMessage'
[string "@Interface\AddOns\BagBrother\libs\WildAddon-1.0\WildAddon-1.0-1.lua"]:83: in function `SendSignal'
[string "@Interface\AddOns\Bagnon\src\title.lua"]:56: in function <Interface\AddOns\Bagnon\src\title.lua:54>