Kiatra / BlizzMove

World of Warcraft add-on to make the default windows movable via drag & drop.
23 stars 9 forks source link

[Bug Report] AddOn 'BlizzMove' tried to call the protected function 'UseContainerItem()'. #97

Closed drnoonien closed 1 year ago

drnoonien commented 1 year ago

What game version did the bug occur in?

Retail

Troubleshooting

Describe the bug.

I cannot use items in my bags.

Steps to reproduce.

Expected behavior.

Clicking items in my bag should work.

Errors

1x [ADDON_ACTION_FORBIDDEN] AddOn 'BlizzMove' tried to call the protected function 'UseContainerItem()'.
[string "@Interface/AddOns/!BugGrabber/BugGrabber.lua"]:480: in function <Interface/AddOns/!BugGrabber/BugGrabber.lua:480>
[string "=[C]"]: in function `UseContainerItem'
[string "@Interface/FrameXML/ContainerFrame.lua"]:1384: in function `ContainerFrameItemButton_OnClick'
[string "@Interface/FrameXML/ContainerFrame.lua"]:1435: in function <Interface/FrameXML/ContainerFrame.lua:1423>
drnoonien commented 1 year ago

The culprit seems to be this block of code. Commenting it out resolves the issue. I'm sure it breaks something else instead however.

https://github.com/Kiatra/BlizzMove/blob/master/BlizzMove.lua#L1211

if self.gameVersion >= 100000 then
    -- fix anchor family connection issues with the combined bag
    self:RawHook("UpdateContainerFrameAnchors", function()
        for _, frame in ipairs(ContainerFrameSettingsManager:GetBagsShown()) do
            frame:ClearAllPoints();
        end
        self.hooks.UpdateContainerFrameAnchors();
    end, true);
end
Numynum commented 1 year ago

hi, you're correct and the main thing it'll break is bank bags, and the reagent bag (when dragonflight itself releases)

we're looking into alternative fixes atm

Numynum commented 1 year ago

we're having trouble properly reproducing the problem, but it should hopefully be resolved in the next update if you could confirm if it works, that'd be great :)

drnoonien commented 1 year ago

Seems to be fixed as of the latest update.

Numynum commented 1 year ago

we've had some other users saying the same, glad to have confirmation :)