Can't use consumables/usable items from bag on Cataclysm

Question

Can't use consumables/usable items from bag on Cataclysm

capSAR273 opened this issue 2 months ago · 18 comments

capSAR273 commented 2 months ago

Addon Version

0.3.18

World of Warcraft Version

Cataclysm

Describe the bug

Right clicking potions/quest items/etc doesn't do anything

Steps to reproduce

Open bag
Right click a potion/flask/usable item

jelmerj · Answer 1 · 2025-02-20T18:12:14.000Z

Same, getting this error:

4x [ADDON_ACTION_FORBIDDEN] AddOn 'BetterBags' tried to call the protected function 'UNKNOWN()'.
[string "@!BugGrabber/BugGrabber.lua"]:485: in function <!BugGrabber/BugGrabber.lua:485>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function UseContainerItem' [string "@Blizzard_UIPanels_Game/Classic/ContainerFrame_Shared.lua"]:1352: in function ContainerFrameItemButton_OnClick'
[string "*ContainerFrame.xml:163_OnClick"]:13: in function <[string "*ContainerFrame.xml:163_OnClick"]:1>

Locals:
_ = Frame {
}
event = "ADDON_ACTION_FORBIDDEN"
events =

{
}

dex97er · Answer 2 · 2025-02-20T20:38:15.000Z

Same Here
3x [ADDON_ACTION_FORBIDDEN] AddOn 'BetterBags' hat versucht die geschützte Funktion 'UNKNOWN()' aufzurufen.
[string "@!BugGrabber/BugGrabber.lua"]:485: in function <!BugGrabber/BugGrabber.lua:485>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function UseContainerItem' [string "@Blizzard_UIPanels_Game/Classic/ContainerFrame_Shared.lua"]:1352: in function <...zard_UIPanels_Game/Classic/ContainerFrame_Shared.lua:1299> [string "=[C]"]: in function ContainerFrameItemButton_OnClick'
[string "*ContainerFrame.xml:163_OnClick"]:13: in function <[string "*ContainerFrame.xml:163_OnClick"]:1>

Locals:
_ = Frame {
}
event = "ADDON_ACTION_FORBIDDEN"
events =

{
}

Trikas77 · Answer 3 · 2025-02-21T10:40:06.000Z

@Cidan this bug makes the addon actually unusable, I cant use any HS, Enchants, Potions or anything that is "activatable"

I just did some testing, enabling only Betterbag doesnt bug.

But as soon as TradeSkillMaster is enabled the AddOn bugs out instantly.
Without TSM it does bug out at some point later and needs a reload to work again.

7x [ADDON_ACTION_FORBIDDEN] AddOn 'BetterBags' tried to call the protected function 'UNKNOWN()'.
[string "@!BugGrabber/BugGrabber.lua"]:485: in function <!BugGrabber/BugGrabber.lua:485>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function `UseContainerItem'
[string "@Blizzard_UIPanels_Game/Classic/ContainerFrame_Shared.lua"]:1352: in function `ContainerFrameItemButton_OnClick'
[string "*ContainerFrame.xml:163_OnClick"]:13: in function <[string "*ContainerFrame.xml:163_OnClick"]:1>

Locals:
_ = Frame {
}
event = "ADDON_ACTION_FORBIDDEN"
events = <table> {
}

capSAR273 · Answer 4 · 2025-02-21T13:51:06.000Z

taint.log

I enabled the taint log and this is the output if it helps @Cidan

capSAR273 · Answer 5 · 2025-02-21T13:09:59.000Z

I'll notify the TSM developers in case they have any suggestions.

fryguy1013 · Answer 6 · 2025-02-22T02:02:40.000Z

It seems to be something related to this change: https://github.com/Gethe/wow-ui-source/blame/classic/Interface/AddOns/Blizzard_UIPanels_Game/Classic/ContainerFrame_Shared.lua#L1338

AuctionHouseFrame is becoming tainted.

The stack trace for that variable becoming tainted is inside the C_AddOns.LoadAddOn("Blizzard_AuctionHouseUI") call in TSM (TradeSkillMaster\Core\UI\AuctionUI\Core.lua:59). The taint log attributes this to the BetterBags addon.

I hope this helps.

navara · Answer 7 · 2025-02-23T16:40:32.000Z

A TSM user and can confirm this bug in the recent version.

A workaround (without disabling either) is dragging the item on the bar and clicking it from there - that still works.

Chronia82 · Answer 8 · 2025-02-28T14:22:28.000Z

Is this issue still being looked at? I asked Gumdrops (Someone connected to TSM, i think he's a dev, but not 100% sure) about this, and he said the following after testing:

gumdropsEU

This doesn't appear to be related to TSM, the issue is reproducible with only BetterBags. We tried some other bag addons and only BetterBags is causing the issue.

Atm in Classic WoW Cata its very hard to use Betterbags due to this issue.

dex97er · Answer 9 · 2025-02-28T16:28:31.000Z

Cidan said in discord that he is currently very busy but he will try to create a fix for this bug this weekend

navara · Answer 10 · 2025-02-28T16:45:30.000Z

I was getting the same behavior with TSM disabled, so must be something else. Reload helped for a while.

Chronia82 · Answer 11 · 2025-02-28T22:36:54.000Z

Cidan said in discord that he is currently very busy but he will try to create a fix for this bug this weekend

Thank you for the update!

@navara yeah, i can replicate that also. With TSM disabled the error also occurs, but it takes longer.

Jonnnyki · Answer 12 · 2025-03-10T16:36:00.000Z

Cidan said in discord that he is currently very busy but he will try to create a fix for this bug this weekend

any new information on that?

dex97er · Answer 13 · 2025-03-11T02:33:56.000Z

Cidan said in discord that he is currently very busy but he will try to create a fix for this bug this weekend

any new information on that?

I only know what was written about it in discord and unfortunately nothing further happened after that

fryguy1013 · Answer 14 · 2025-03-11T04:13:40.000Z

A simple reproducible scenario. Disable all addons except BetterBags, talk to auctioneer, try to use hearthstone in bags.

fryguy1013 · Answer 15 · 2025-03-11T05:59:13.000Z

I figured it out. In core/constants.lua I added and not addon.isCata to this:

if not addon.isRetail and not addon.isCata then
  Enum.ItemQuality.Poor = 0
  Enum.ItemQuality.Common = 1
  Enum.ItemQuality.Uncommon = 2
  Enum.ItemQuality.Rare = 3
  Enum.ItemQuality.Epic = 4
  Enum.ItemQuality.Legendary = 5
  Enum.ItemQuality.Artifact = 6
  Enum.ItemQuality.Heirloom = 7
  Enum.ItemQuality.WoWToken = 8
end

I added this section above:

local IQ = IQ or {
  Poor = 0,
  Common = 1,
  Uncommon = 2,
  Rare = 3,
  Epic = 4,
  Legendary = 5,
  Artifact = 6,
  Heirloom = 7,
  WoWToken = 8,
}

Then replaced all references to Enum.ItemQuality with IQ in that file.

This seemed to fix it for Cataclysm. I have no idea why Enum.ItemQuality isn't loaded into the global scope when this script is ran but seems to be loaded later on. This fixes my issue, but there's probably a much better way to do it so I won't open a PR.

capSAR273 · Answer 16 · 2025-03-15T02:46:47.000Z

Here's the finished constants.lua file: https://pastebin.com/LXWQLjxH

Thank you for the fix @fryguy1013 , seems like it works now.

fryguy1013 · Answer 17 · 2025-03-15T02:55:31.000Z

My fix breaks the bank btw. I changed it to just make a copy into the const table like so:

if not addon.isRetail then
  const.ItemQuality = {
    Poor = 0,
    Common = 1,
    Uncommon = 2,
    Rare = 3,
    Epic = 4,
    Legendary = 5,
    Artifact = 6,
    Heirloom = 7,
    WoWToken = 8,
  }
else
  const.ItemQuality = Enum.ItemQuality
end

and changed all the references to use const.ItemQuality instead of Enum.ItemQuality and make sure every file included it.

navara · Answer 18 · 2025-03-15T09:50:07.000Z

How does it break the bank? I haven't observed any issues.

Anyway, imho it's time for the pull request, even as a workaround. It doesn't seem Cidan will be able to fix it anytime soon.

Share to

Addon Version

World of Warcraft Version

Describe the bug

Steps to reproduce