-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug Report] v3.5.24 introduces a conflict with AceHook #118
Comments
and this does not happen with no other addons enabled, or with an older blizzmove version? |
As said, does not occur with v3.5.23. I did not test with older versions before that, but I've never seen the error before, IIRC, and I'm using BM for quite a few years. If I reload with a minimal addon set (or with only BM and BugSack), the error does not occur. But I guess this is due to the absence of AceHook… |
I have to correct myself: If I load only with BM and BugSack and Accountant_Classic (to provide the Ace lib), the error does not occur. So something more must be involved… |
And this is classic yes? (since you selected retail 🙂) |
No, this is Retail. I don't play Classic. |
Accountant_Classic is a Retail addon, despite the name. |
I now reloaded with BM, BugSack, and BattlePetCount as the AceHook lib provider, and I get the error again. Reproducible. |
And the AceHook versions are the same: -- @release $Id: AceHook-3.0.lua 1284 2022-09-25 09:15:30Z nevcairiel $ Weird… |
I installed now Ace3-Release-r1320 separately, and the error does not occur. |
I just verified again with BM v3.5.23 (because I couldn't believe it), but the error does definitely not occur, even with one of my regular addon sets. |
I hope that bug reports by other people will provide a better clue what is going on here, but – for the moment – rolling back to 3.5.23 seems a reasonable thing for me. |
I'll have time to take a look later today probably, but if BM + BattlePetCount is reproduceable on my end, it should be easy enough to debug and fix 👍 |
It seems to be reproducible, yes. It's BattlePetCountNG. |
More precisely: BM 3.5.24 alone: No error |
I've had this error with 3 addons so far: Better Wardrobe and Transmog, BigDebuffs and now BlizzMove, as I disable them more keep showing up... I don't use BattlePetCountNG so something else is contributing to this for me at least. I'm continuing to test on my end, will follow up. Here's the error I got in case it's useful: 1x ...ceBlizzMove/Libs/AceHook-3.0-9/AceHook-3.0.lua:180: Attempting to rehook already active hook GetBagsShown. Locals: |
Disabling BlizzMove fixed the errors in the other two add-ons I mentioned above. Seeing if I can figure out the conflict but I have a lot of add-ons, no promises... |
@raysmith59, for you it's also only with Blizz Move 3.5.24, or with the previous version (3.5.23) too? |
just go with the previous version of blizzmove for now, and it should be fine |
Just 3.5.24, 3.5.23 has no error. |
Unfortunately couldn't figure out the specific conflict, tested BlizzMove with batches of other add-ons and didn't get the error so I figure it has to be a 3+ addon conflict... will keep testing for a bit. |
v3.5.25 should be available on CF soon, and fixes the problem for anyone curious, the issues are caused when an addon using AceEvent, is loaded before BlizzMove, and registers the ADDON_LOADED event thanks for the debugging on your ends |
Glad to hear it, I had at least a 4-way addon conflict causing my issue and I hadn't figured out the specific combo yet. |
Oh, that was fast. Thanks 😃 |
Confirmed that 3.5.25 works for me. |
What game version did the bug occur in?
Retail
Troubleshooting
Describe the bug.
Does not occur with v3.5.23.
Note that the addon mentioned in the error log (Accountant_Classic) is irrelevant, as it is just the provider of the AceHook library (alphabetically first in the loading order).
Steps to reproduce.
Right after login/reload.
I had to tick the "I have tried to reproduce the bug with no other addons loaded" because it is required to post the ticket(!). But actually this is a conflict with another addon (AceHook), so this does not apply.
Expected behavior.
Should not conflict.
Errors
The text was updated successfully, but these errors were encountered: