Bagnon

Bagnon

122M Downloads

Sorting Error

marulol opened this issue · 1 comments

commented

Which software were you running?

  • Addon version name:
  • Client used: [e.g. Vanilla 1.14.4, Retail PTR x.x.x, etc]

Have you read the changelog?
Mark if read.

Please describe the bug.
A clear and concise description of what the bug is.

Clicking sort brings up LUA error

Please describe how to reproduce it.

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

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

Error Logs are Important!
Type /console scriptErrors 1 in the chat and reload the game. If an error window appears when reproducing the bug, please write here the first message (labelled 1/Many) that appeared. ⚠️ By default, the last error message is shown, not the first! You need to navigate back to the first error.

13x BagBrother/core/api/sorting.lua:122: attempt to compare nil with number
[string "@BagBrother/core/api/sorting.lua"]:122: in function GetSpaces' [string "@BagBrother/core/api/sorting.lua"]:43: in function Iterate'
[string "@BagBrother/core/api/sorting.lua"]:36: in function Run' [string "@BagBrother/core/api/sorting.lua"]:30: in function Start'
[string "@BagBrother/core/classes/frameBase.lua"]:116: in function <BagBrother/core/classes/frameBase.lua:115>
[string "=(tail call)"]: ?
[string "@BagBrother/frames/containers/inventory.lua"]:105: in function `SortItems'
[string "@BagBrother/core/classes/sortButton.lua"]:39: in function <BagBrother/core/classes/sortButton.lua:30>

Locals:
self =

{
SendSignal = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:82
CanRun = defined @BagBrother/core/api/sorting.lua:171
RegisterMessage = defined @Ace3/CallbackHandler-1.0/CallbackHandler-1.0.lua:80
NewModule = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:68
GetOrder = defined @BagBrother/core/api/sorting.lua:150
Proprieties =
{
}
GetFamilies = defined @BagBrother/core/api/sorting.lua:135
target = BagnonInventory1 {
}
SendMessage = defined @Ace3/CallbackHandler-1.0/CallbackHandler-1.0.lua:49
Delaying = defined @BagBrother/libs/MutexDelay-1.0/MutexDelay-1.0.lua:46
Rule = defined @BagBrother/core/api/sorting.lua:183
UnregisterSignal = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:78
FitsIn = defined @BagBrother/core/api/sorting.lua:175
Move = defined @BagBrother/core/api/sorting.lua:196
GetSpaces = defined @BagBrother/core/api/sorting.lua:110
RegisterSignal = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:74
UnregisterAllMessages = defined @Ace3/CallbackHandler-1.0/CallbackHandler-1.0.lua:164
Run = defined @BagBrother/core/api/sorting.lua:33
UnregisterAllEvents = defined @Ace3/CallbackHandler-1.0/CallbackHandler-1.0.lua:164
Iterate = defined @BagBrother/core/api/sorting.lua:42
UnregisterMessage = defined @Ace3/CallbackHandler-1.0/CallbackHandler-1.0.lua:143
Stop = defined @BagBrother/core/api/sorting.lua:103
RegisterEvent = defined @Ace3/CallbackHandler-1.0/CallbackHandler-1.0.lua:80
Start = defined @BagBrother/core/api/sorting.lua:23
Delay = defined @BagBrother/libs/MutexDelay-1.0/MutexDelay-1.0.lua:27
UnregisterEvent = defined @Ace3/CallbackHandler-1.0/CallbackHandler-1.0.lua:143
Tag = "BAGNON_"
}
spaces =
{
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 =
{
}
72 =
{
}
73 =
{
}
74 =
{
}
75 =
{
}
76 =
{
}
77 =
{
}
78 =
{
}
79 =
{
}
80 =
{
}
81 =
{
}
82 =
{
}
83 =
{
}
84 =
{
}
85 =
{
}
86 =
{
}
87 =
{
}
88 =
{
}
89 =
{
}
90

commented

Cool, GetItemInfo can fail to return info. Good to know (yeah, the client fibbed there).