Opening Bank causes error SOD (P5)

[Believe82]

Message: Interface/AddOns/ItemRack/ItemRack.lua:43: bad argument #1 to 'IsInventorySlotEngravable' (outside of expected range 0 to 4294967295 - Usage: local result = C_Engraving.IsInventorySlotEngravable(containerIndex, slotIndex))
Time: Thu Oct 10 11:35:04 2024
Count: 5
Stack: Interface/AddOns/ItemRack/ItemRack.lua:43: bad argument #1 to 'IsInventorySlotEngravable' (outside of expected range 0 to 4294967295 - Usage: local result = C_Engraving.IsInventorySlotEngravable(containerIndex, slotIndex))
[string "@Interface/AddOns/ItemRack/ItemRack.lua"]:43: in function AppendRuneID' [string "@Interface/AddOns/ItemRack/ItemRack.lua"]:709: in function GetID'
[string "@Interface/AddOns/ItemRack/ItemRack.lua"]:1143: in function BuildMenu' [string "@Interface/AddOns/ItemRack/ItemRack.lua"]:1874: in function DockMenuToCharacterSheet'
[string "@Interface/AddOns/ItemRack/ItemRack.lua"]:1853: in function `PaperDollItemSlotButton_OnEnter'
[string "*PaperDollFrame.xml:33_OnEnter"]:1: in function <[string "*PaperDollFrame.xml:33_OnEnter"]:1>

Locals: bag = -1
slot = 1

[Anonomit]

Duplicate of #251 - some solutions posted there.

[gurka]

Duplicate of #251 - some solutions posted there.

Not a duplicate, different error and #251 should have been fixed in 4.2. This issue still happens in 4.23

[gurka]

Modifying this line:

ItemRack/ItemRack/ItemRack.lua

Line 43 in a36c7fc

if C_Engraving.IsInventorySlotEngravable(bag, slot) then

to if bag >= 0 and C_Engraving.IsInventorySlotEngravable(bag, slot) then

seems to fix it. However I only tested it very quickly, but moving gear from and to the bank via ItemRack's "extension" to the Character Info frame works OK.

I can make a PR later today.

Edit: might be interesting to figure out why we get bag=-1 in the first place. I have no idea what the value represents
Edit2: I guess this is related: https://wowwiki-archive.fandom.com/wiki/BagId

[tstirrat]

This is still happening, is this fix something we can merge into master?