AlexFolland/AutoGear

Invalid inventory slot in GetInventorySlotInfo in TBC

jamtur01 opened this issue · 12 comments

jamtur01 commented

I am guessing some InventorySlot got renamed or renumbered or something. I'll poke through the API to see if I can work out what's happening.

Message: Interface\AddOns\AutoGear\AutoGear.lua:2976: Invalid inventory slot in GetInventorySlotInfo
Time: Thu May 20 03:00:54 2021
Count: 4
Stack: Interface\AddOns\AutoGear\AutoGear.lua:2976: Invalid inventory slot in GetInventorySlotInfo
[string "=[C]"]: ?
[string "=[C]"]: in function `GetInventorySlotInfo'
[string "@Interface\AddOns\AutoGear\AutoGear.lua"]:2976: in function <Interface\AddOns\AutoGear\AutoGear.lua:2965>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "@Interface\AddOns\Bartender4\libs\AceHook-3.0\AceHook-3.0.lua"]:95: in function <...e\AddOns\Bartender4\libs\AceHook-3.0\AceHook-3.0.lua:87>
[string "=[C]"]: ?
[string "=[C]"]: in function `SetAction'
[string "@Interface\AddOns\Bartender4\libs\LibActionButton-1.0\LibActionButton-1.0.lua"]:1536: in function `SetTooltip'
[string "@Interface\AddOns\Bartender4\libs\LibActionButton-1.0\LibActionButton-1.0.lua"]:1303: in function <...er4\libs\LibActionButton-1.0\LibActionButton-1.0.lua:1296>
[string "@Interface\AddOns\Bartender4\libs\LibActionButton-1.0\LibActionButton-1.0.lua"]:520: in function <...er4\libs\LibActionButton-1.0\LibActionButton-1.0.lua:518>

Locals:
jamtur01 commented

And indeed https://wowwiki-archive.fandom.com/wiki/InventorySlotName

Looks like AmmoSlot and RangedSlot were removed in TBC. Marvelous.

  • ❤️1
jamtur01 commented

Nope I am wrong - they were removed in Cataclysm. So that's not quite right.

  • ❤️1
AlexFolland commented

What item were you hovering when the error first appeared? That would tell us a lot about the cause of this, since the error was in the code for tooltip score comparison.

AlexFolland commented

Also, was Locals: actually blank? There could be some valuable entries in there.

jamtur01 commented
  1. I thought it was a wand but will need to test some more.
  2. Locals was indeed blank.
jamtur01 commented 
Message: Interface\AddOns\AutoGear\AutoGear.lua:2976: Invalid inventory slot in GetInventorySlotInfo
Time: Fri May 21 01:24:14 2021
Count: 1
Stack: Interface\AddOns\AutoGear\AutoGear.lua:2976: Invalid inventory slot in GetInventorySlotInfo
[string "=[C]"]: ?
[string "=[C]"]: in function `GetInventorySlotInfo'
[string "@Interface\AddOns\AutoGear\AutoGear.lua"]:2976: in function <Interface\AddOns\AutoGear\AutoGear.lua:2965>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "@Interface\AddOns\Bartender4\libs\AceHook-3.0\AceHook-3.0.lua"]:95: in function <...e\AddOns\Bartender4\libs\AceHook-3.0\AceHook-3.0.lua:87>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function `SetBagItem'
[string "@Interface\FrameXML\ContainerFrame.lua"]:1340: in function `ContainerFrameItemButton_OnEnter'
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\item.lua"]:369: in function `ShowTooltip'
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\item.lua"]:340: in function `UpdateTooltip'
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\item.lua"]:180: in function <...face\AddOns\Bagnon\common\Wildpants\classes\item.lua:179>

Locals:
jamtur01 commented

I wonder if it's actually bubbling up from somewhere else?

jamtur01 commented

Okay - I also get this:

Message: Interface\AddOns\AutoGear\AutoGear.lua:2970: attempt to concatenate local 'name' (a nil value)
Time: Sat May 22 12:32:48 2021
Count: 67
Stack: Interface\AddOns\AutoGear\AutoGear.lua:2970: attempt to concatenate local 'name' (a nil value)
[string "=[C]"]: ?
[string "@Interface\AddOns\AutoGear\AutoGear.lua"]:2970: in function <Interface\AddOns\AutoGear\AutoGear.lua:2965>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function `SetCompareItem'
[string "@Interface\FrameXML\GameTooltip.lua"]:716: in function `GameTooltip_ShowCompareItem'
[string "@Interface\FrameXML\GameTooltip.lua"]:545: in function <Interface\FrameXML\GameTooltip.lua:543>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "@Interface\AddOns\Bartender4\libs\AceHook-3.0\AceHook-3.0.lua"]:95: in function <...e\AddOns\Bartender4\libs\AceHook-3.0\AceHook-3.0.lua:87>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function `SetBagItem'
[string "@Interface\FrameXML\ContainerFrame.lua"]:1340: in function `ContainerFrameItemButton_OnEnter'
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\item.lua"]:369: in function `ShowTooltip'
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\item.lua"]:340: in function `UpdateTooltip'
[string "@Interface\AddOns\Bagnon\common\Wildpants\classes\item.lua"]:180: in function <...face\AddOns\Bagnon\common\Wildpants\classes\item.lua:179>

Locals:

When I select Off-hand items and items that could fit in main or offhand (swords, daggers, etc).

jamtur01 commented

Okay - I think one of the issues is Mining Picks. Which don't have a "name" as such. Not clear what I did with a nil there.

jamtur01 commented

The only error I am still seeing is this:

Date: 2021-06-13 14:39:37
ID: 2
Error occured in: Global
Count: 1
Message: ..\AddOns\AutoGear\AutoGear.lua line 3062:
   Invalid inventory slot in GetInventorySlotInfo
Debug:
   [string "=(tail call)"]: ?
   [string "=[C]"]: ?
   [string "=[C]"]: GetInventorySlotInfo()
   [string "@AutoGear\AutoGear.lua"]:3062:
      AutoGear\AutoGear.lua:3047
   [string "=[C]"]: ?
   [string "@Auc-Advanced\Libs\LibExtraTip\LibExtraTip.lua"]:498: OnTooltipSetItem()
   [string "@Enchantrix\EnxMain.lua"]:232:
      Enchantrix\EnxMain.lua:228
   [string "=[C]"]: ?
   [string "=[C]"]: ?
   [string "=[C]"]: ?
   [string "=[C]"]: ?
   [string "=[C]"]: SetAction()
   [string "@Bartender4\libs\LibActionButton-1.0\LibActionButton-1.0.lua"]:1536: SetTooltip()
   [string "@Bartender4\libs\LibActionButton-1.0\LibActionButton-1.0.lua"]:1303:
      ...er4\libs\LibActionButton-1.0\LibActionButton-1.0.lua:1296
   [string "@Bartender4\libs\LibActionButton-1.0\LibActionButton-1.0.lua"]:520:
      ...er4\libs\LibActionButton-1.0\LibActionButton-1.0.lua:518
Locals:
(*temporary) = true
(*temporary) = <function> defined =[C]:-1

AddOns:
  Swatter, v8.2.6511 (SwimmingSeadragon)
  AucAdvanced, v8.2.6471 (SwimmingSeadragon)
  aucadvancedsearcherpawn, v1.5.1
  AucFilterBasic, v8.2.6497 (SwimmingSeadragon)
  AucScanData, v8.2.6498 (SwimmingSeadragon)
  AucStatHistogram, v8.2.6499 (SwimmingSeadragon)
  AucStatiLevel, v8.2.6503 (SwimmingSeadragon)
  AucStatPurchased, v8.2.6500 (SwimmingSeadragon)
  AucStatSimple, v8.2.6501 (SwimmingSeadragon)
  AucStatStdDev, v8.2.6502 (SwimmingSeadragon)
  AucUtilFixAH, v8.2.6504 (SwimmingSeadragon)
  Auctionator, v100.0.11
  AutoGear, v
  BagBrother, v
  Bagnon, v9.0.7
  BagnonConfig, v
  Bartender4, v4.10.9
  BeanCounter, v8.2.6505 (SwimmingSeadragon)
  CharacterStatsTBC, v1.1.1
  ChatBarClassic, v3.17.3a
  ClassicSpellActivations, v
  Details, v
  DetailsStreamer, v
  DetailsTinyThreat, v
  Enchantrix, v8.2.6506 (SwimmingSeadragon)
  EnchantrixBarker, v8.2.6507 (SwimmingSeadragon)
  GSE, v2.6.55
  GSEGUI, v2.6.55
  GSELDB, v2.6.55
  Informant, v8.2.6508 (SwimmingSeadragon)
  KartarRiposteAlert, v1.0.0
  Masque, v9.0.6
  MissingTradeSkillsListTBC, v2.5.1
  NameplateSCT, v1.9
  Pawn, v2.5.17
  Plater, vPlater-v331.1-BCC
  Poisoner, v2.5.1.3-bcc
  Questie, v6.3.15
  SilverDragon, vv20501.0
  SilverDragonBurningCrusade, vv20501.0
  SilverDragonClassic, vv20501.0
  SlideBar, v8.2.6509 (SwimmingSeadragon)
  SpeedyAutoLoot, v2.0.43-Beta
  Stubby, v8.2.6510 (SwimmingSeadragon)
  TFTB, v1.30
  TitanClassic, v1.3.2.20501
  TitanClassicAmmo, v1.3.2.20501
  TitanClassicBag, v1.3.2.20501
  TitanClassicClock, v1.3.2.20501
  TitanClassicGold, v1.3.2.20501
  TitanClassicLocation, v1.3.2.20501
  TitanClassicLootType, v1.3.2.20501
  TitanClassicPerformance, v1.3.2.20501
  TitanClassicRegen, v1.3.2.20501
  TitanClassicRepair, v1.3.2.20501
  TitanClassicVolume, v1.3.2.20501
  TitanClassicXP, v1.3.2.20501
  TomTom, vv3.0.1
  TradeSkillMaster, vv4.10.43
  TradeSkillMasterAppHelper, vv4.0.14
  WoWPro, v2.5.1-A2
  WoWProDailies, v2.5.1-A2
  WoWProLeveling, v2.5.1-A2
  BlizRuntimeLib_enUS v2.5.1.20501 <none>
  (ck=7e1)
AlexFolland commented

I think I fixed this. I reproduced it when mousing over a mount and discovered that was the source of the issue here. Can you confirm you can no longer reproduce it with the latest release I just pushed? Thanks!

jamtur01 commented

Fixed! Thanks @AlexFolland 🖤