ItemRack Classic

ItemRack Classic

7M Downloads

switching gear from bank doesnt work on SoD current patch (p5)

jomysan opened this issue ยท 3 comments

commented

Hello, as title says.

Exporting from bugsack

93x ItemRack/ItemRack.lua:34: bad argument #1 to 'IsInventorySlotEngravable' (outside of expected range 0 to 4294967295 - Usage: local result = C_Engraving.IsInventorySlotEngravable(containerIndex, slotIndex))

[string "=[C]"]: in function IsInventorySlotEngravable' [string "@ItemRack/ItemRack.lua"]:34: in function AppendRuneID'
[string "@ItemRack/ItemRack.lua"]:700: in function getid' [string "@ItemRack/ItemRack.lua"]:813: in function FindInBank'
[string "@ItemRack/ItemRack.lua"]:2177: in function MissingItems' [string "@ItemRack/ItemRack.lua"]:1255: in function BuildMenu'
[string "@ItemRack/ItemRack.lua"]:1917: in function `OnClick'

commented

Copying my comment from closed issue #266:

Did further research and found arkayenro/arkinventory#2019. I am not sure this is the correct fix, but I changed line 36 of ItemRack.lua to return C_Engraving and blizzard_id ~= nil and C_Engraving.IsEngravingEnabled() and it seems to work.

See also:
https://us.forums.blizzard.com/en/wow/t/sod-engraved-items-in-the-bank-isinventoryslotengravable-bug/1964618

You can look at the git changes arkayenro did to fix the issue: ArkInventory-3.11.01-alpha-7-with-git-changes-from-6.zip

commented

it works! very fixed atm with no lua error popping!
saved my raid day! tyvm!

(ill posto on curse as well giving u full credits)

commented

That just makes the addon think it's running in Era instead of Season of Discovery. A more appropriate fix would be to add a line in ItemRack.AppendRuneID to check for bad parameters.

Add a new line after line 41, and paste in:
if bag < 0 then return "" end

So it should look like this starting at line 41:

		function ItemRack.AppendRuneID(bag, slot)
			if bag < 0 then return "" end
			if slot then

That seems to get it working with runes.