syndenbock / Farmer

World of Warcraft addon that makes farming fun
2 stars 0 forks source link

Error When I login or Reload #14

Closed Comicus closed 4 months ago

Comicus commented 1 year ago

2x Farmer error: Interface\AddOns\Farmer\classes\Storage.lua:77: table index is nil [string "@Interface\AddOns\Farmer\core\Events.lua"]:29: in function <Interface\AddOns\Farmer\core\Events.lua:25> [string "=[C]"]: in function pcall' [string "@Interface\AddOns\Farmer\utils\Utils.lua"]:123: in function <Interface\AddOns\Farmer\utils\Utils.lua:122> [string "@Interface\AddOns\Farmer\classes\CallbackHandler.lua"]:22: in function__callCallbacks' [string "@Interface\AddOns\Farmer\classes\CallbackHandler.lua"]:48: in function `call' [string "@Interface\AddOns\Farmer\core\Events.lua"]:10: in function <Interface\AddOns\Farmer\core\Events.lua:9>

syndenbock commented 1 year ago

Are you playing Classic, Wrath or Retail? Is this happening on all your characters or only on one? And would you mind uploading a screenshot of your current equipment and your bags? It seems an item in your equipment or your bags returns empty information causing this error.

WorkPixels commented 1 year ago

WoW Classic WotLK Version 3.4.0.46902

App Version Farmer-3.16.5

27x Farmer error: Interface\AddOns\Farmer\classes\Storage.lua:90: table index is nil [string "@Interface\AddOns\Farmer\core\Events.lua"]:29: in function <Interface\AddOns\Farmer\core\Events.lua:25> [string "=[C]"]: in function pcall' [string "@Interface\AddOns\Farmer\utils\Utils.lua"]:123: in function <Interface\AddOns\Farmer\utils\Utils.lua:122> [string "@Interface\AddOns\Farmer\classes\CallbackHandler.lua"]:22: in function__callCallbacks' [string "@Interface\AddOns\Farmer\classes\CallbackHandler.lua"]:48: in function `call' [string "@Interface\AddOns\Farmer\core\Events.lua"]:10: in function <Interface\AddOns\Farmer\core\Events.lua:9>

syndenbock commented 1 year ago

I have just released a new version 3.16.6 where this error should no longer occur but print a debug message with an item id instead. Please let me know said item id so I can look into a possible issue with some items.

WorkPixels commented 1 year ago

So far no errors with the update.

syndenbock commented 1 year ago

And is there any message in the chat after logging in? Because unless the cause of the error magically vanished, a message should be printed stating something about an item link could not be found for an item id.

WorkPixels commented 1 year ago

This is the only thing I see.

Farmer: No link found for item id 41164.

WorkPixels commented 1 year ago

Mammoth Cutters = 41164

syndenbock commented 1 year ago

I assume you have Mammoth Cutters equipped as ammo? If so, can you unequip any ammo, /reload and check if a message is printed still?

WorkPixels commented 1 year ago

I wasn't able to unequip the ammo. I logged into another character. It seems that it's not able to count equipped ammo.

Farmer: No link found for item id 32883. Felbane Slugs = 32883

WorkPixels commented 1 year ago

When I log in to any character without ammo equipped I don't see any errors.

syndenbock commented 1 year ago

Thanks a ton for the input! I don't have a classic character with ammo so I couldn't test this myself but now I at least know what causes the error and can look into finding a fix.

syndenbock commented 1 year ago

The fix actually is trivial, I had assumed equipped ammo is removed from the bags but it's just a visual slot. So the error is fixed by simply not checking the equipped ammo slot.

syndenbock commented 1 year ago

Just to confirm: The message no longer pops up in version 3.16.7, correct?

WorkPixels commented 1 year ago

I do not see any errors