search

HizurosWoWAddOns / Broker_Everything

A collection of module to display informations about own character, friends, guild and more on minimap or panel addons like Bazooka, ChcolateBar, TitanPanel and more.
GNU General Public License v3.0
9 stars 4 forks source link

[BUG]: Ammo module not working in 3.4.1 #32

Closed kazso closed 1 year ago

kazso commented 1 year ago

WoW client

WoW Classic WotLk

What happened?

Logging into a hunter character i get an error, and the ammo tooltip is empty since patch 3.4.1.

I came up with a temporary solution until this is fixed. Changing local _, count = GetContainerItemInfo(item.bag,item.slot); to local count = (C_Container.GetContainerItemInfo(item.bag,item.slot)).stackCount; in ammo_classic.lua works for me (no errors and shows the correct amount of ammo). Only tested this in WotLK Classic.

Lua Error Message?

2x ...ceBroker_Everything/modules/ammo_classic.lua:68: attempt to call global 'GetContainerItemInfo' (a nil value)
[string "@Broker_Everything/modules/ammo_classic.lua"]:68: in function `fnc'
[string "@Broker_Everything/shared.lua"]:826: in function <Broker_Everything/shared.lua:821>
[string "@Broker_Everything/shared.lua"]:855: in function <Broker_Everything/shared.lua:830>
[string "@Broker_Everything/shared.lua"]:1005: in function <Broker_Everything/shared.lua:948>
[string "@Broker_Everything/shared.lua"]:1031: in function <Broker_Everything/shared.lua:1019>

Other addons?

No response

hizuro commented 1 year ago

Thank you for reporting. Update is available.