My addon is definitely one of the addons causing it. I told a guildie to disable mine in TS last night and he still encountered the issue - so he had a different plugin causing the same problem (it seems that the vast majority of plugins are not causing this problem).

/reloadui works but you need to do it every time you open the map (or inventory, or any other UI).

I just need to know what I have to fix. The game isn't displaying any errors or anything, so I have no idea where to start with this.

zgrssd wrote: »

I installed your addon jsut to see if it happened.
And it did without being in a group. I need to nail down wich other screens can cause this.

All I did was:
Logging in with my 2nd Char this day/first char since installation of addon.
Go to Stablemaster to train my horse.
Go to Clothing Station to check if I can start reserach, nothing there.
Go to Blacksmithign station and doing the same, started one (2nd Trait on Greataxe).
Try to either exit mapscreen or BS screen using ESC and it not working (Alt still working).

I cannot reproduce it even by following the steps precisely, so it might be realted to some Reserach Start traking addon.

ESC also becomes unuseable whenever:
I start a Trait Reserach, while having your addon on (seems to be unreltead with research addons).
Withdrawing money from the bank while having your addon on (this makes reproduction a lot easer, I guess).

The only tip I have:
Add d() messages to every function and especially events handlers (start and end)
Use BugEater wit henamebeld pre-init debug (needs only be optional dependency) to get d() messages from around Addon Load Event

Double and tripple check that you do not use any Global variables anywhere, unelss you really need them.
I got into debugging hell while developing my UnifiedChatTabs because I accidently used global variables as temp for my functions return values (wich resulted in the value being not what I expected).
Could be you and the API programmers choose a similar name for something global.

Some things about your code:
In getKey you have the same mistake I had with temp values for function return:
local function GetKeys(table)
result = {} -- not marked local, so it is global and will collide with any other use of "result" in any addon or the API that is not marked local either

Same in
function EXT_GROUPLEADER.Extend(value, default)
value = value or {}
Edit: Not there, as the Parameter is automatically local. Overlooked that part.

But fixing those two for my copy did not solve it.
You should perhaps use LibStub in your Lib File instead of the code you use for uniqness. But then again I cannot quite follow your code.

Okay, I tried adding some d-messages and figured out fast that putting ones into extGroupLeaderUpdate or any of it's sub functions was a bad idea. Because they get called every Frame.
Just a tip here: You might be able to cut down on the computational load if you first checked if the angle has even changed compared to last run. The angle will change only on very few frames/updates Plus maybe write a library for this function seperately, so other code can use it without having to register the event/change of mouse angle seperately.

Í also noticed that you tried to mimic classes using metatables (something I only heard of, never done myself). So it could also be some incomplete seperation of data between instances. But like I said: Never used this technique in .lua before.

Nothing from the Main lua or Library file seems to be called when I withdraw from the bank so events don't seem to get fired when they should not. That was pretty much my best guess.
I am thoroughly stumped at the moment at what could cause this, but will continue to investiage if I have time.