AdiBags

AdiBags

8M Downloads

after leaving arena, emblems update once, then they don't anymore

sod opened this issue ยท 3 comments

commented

AdiBags-v1.9.32 / german retail patch 9.2.7

image

Tried to reproduce it in this vod https://www.twitch.tv/videos/1572612573?t=0h15m10s. The startpoint is after it already froze. In that clip you see how i buy something from a garrison vendor. The blizzard ui updates, but adibags doesn't:

image

I joined an arena skirmish (also happens after arena ranked though) beforehand, then the emblems updated once (the honor I got was reflected) but didn't do any updates afterwards. Also adding/removing emblems in the settings didn't update the UI anymore.

I don't see any errors in my console though.

Maybe related, maybe not: When updating still works, it removes the 1000th divider, like it goes from 1.700 to 1650 (for garrison resources, didn't test the others)

commented

Installed bugsack as recommended and actually there is an error right when the arena match ends:

1x AdiBags\modules\CurrencyFrame.lua:201: attempt to index local 'cell' (a nil value)
[string "@AdiBags\modules\CurrencyFrame.lua"]:201: in function `?'
[string "@AdiBags\libs\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:119: in function <...ags\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:119>
[string "=[C]"]: ?
[string "@AdiBags\libs\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:29: in function <...ags\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:25>
[string "@AdiBags\libs\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:64: in function <...ags\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:59>
[string "=(tail call)"]: ?

seems like cell is nil in

local cell = self.currencyToCell[info.name]
cell.text = cell.icon .. currencyQuantity

commented

Thank you for the report! We completely redid the currency frames in the latest patch, and we definitely missed something. Are you able to trigger this in any other situation other than the Arena?

commented

Fixed and pushed v1.9.33. The fix isn't perfect in terms of compute time, but it should prevent the bug (and fix the missing delimiter too),

Thank you again!