b-morgan / Skillet-Classic

World of Warcraft Classic addon
GNU General Public License v3.0
6 stars 6 forks source link

bad argument #1 to '?' #181

Closed Alessandro-Barbieri closed 3 months ago

Alessandro-Barbieri commented 3 months ago

I have encountered this error

2x bad argument #1 to '?' (Usage: local itemName, itemLink, itemQuality, itemLevel, itemMinLevel, itemType, itemSubType, itemStackCount, itemEquipLoc, itemTexture, sellPrice, classID, subclassID, bindTy
[string "=[C]"]: ?
[string "=[C]"]: in function `GetItemInfo'
[string "@Skillet-Classic/UI/Sorting.lua"]:117: in function <Skillet-Classic/UI/Sorting.lua:108>
[string "=(tail call)"]: ?
[string "=[C]"]: in function `sort'
[string "@Skillet-Classic/RecipeGroups.lua"]:428: in function `RecipeGroupSort'
[string "@Skillet-Classic/UI/Sorting.lua"]:525: in function `SortAndFilterRecipes'
[string "@Skillet-Classic/SkilletData.lua"]:1238: in function <Skillet-Classic/SkilletData.lua:785>
[string "@Skillet-Classic/SkilletData.lua"]:1306: in function `RescanTrade'
[string "@Skillet-Classic/Skillet.lua"]:1313: in function `SkilletShowWindow'
[string "@Skillet-Classic/Skillet.lua"]:967: in function `TRADE_SKILL_UPDATE'
[string "@Skillet-Classic/Skillet.lua"]:1281: in function `SkilletShow'
[string "@Skillet-Classic/Skillet.lua"]:1038: in function `?'
[string "@Accountant_Classic/Libs/CallbackHandler-1.0-8/CallbackHandler-1.0.lua"]:109: in function <...sic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:109>
[string "=[C]"]: ?
[string "@Accountant_Classic/Libs/CallbackHandler-1.0-8/CallbackHandler-1.0.lua"]:19: in function <...sic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:15>
[string "@Accountant_Classic/Libs/CallbackHandler-1.0-8/CallbackHandler-1.0.lua"]:54: in function `Fire'
[string "@Accountant_Classic/Libs/AceEvent-3.0-4/AceEvent-3.0.lua"]:120: in function <...ccountant_Classic/Libs/AceEvent-3.0/AceEvent-3.0.lua:119>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function `CastSpellByName'
[string "@Blizzard_FrameXML/Classic/SecureTemplates.lua"]:387: in function `handler'
[string "@Blizzard_FrameXML/Classic/SecureTemplates.lua"]:633: in function <Blizzard_FrameXML/Classic/SecureTemplates.lua:577>

Locals:
(*temporary) = "bad argument #1 to '?' (Usage: local itemName, itemLink, itemQuality, itemLevel, itemMinLevel, itemType, itemSubType, itemStackCount, itemEquipLoc, itemTexture, sellPrice, classID, subclassID, bindType, expansionID, setID, isCraftingReagent = C_Item.GetItemInfo(itemInfo))"
b-morgan commented 3 months ago

Replace ...\Addons\Skillet-Classic\UI\Sorting.lua with the contents of the attached zip file and let me know if that fixes it.

Sorting.zip

Note: The above file fixes the lua error but if you opened your profession right after logging in then your client may not have completely downloaded all the item information from the server and the sort results may be wrong. Closing your profession and opening it again should fix this.

Alessandro-Barbieri commented 3 months ago

Looks like it's working now