!NoTaint
Reduce the chance that talent switching be forbidden.
These codes are based on very hard work of reviewing Blizzard's source code and testing taint status. The cause of the failure is rather complicated. As far as we know, there are 3 reasons at least.
1. Addon calls ToggleTalentFrame() to open talent panel.
2. Addon calls StaticPopup_Show()
3. Addon use UIFrameFlash()
4. Addon use ChatFrame_AddMessageEventFilter (related to reason 3.)
This addon can fix reason 1, 2 and 4. But for reason 3, there is no workaround fix, you have to modify the code where UIFrameFlash is called (or contact author to fix it).
1111111111111111111111111111111111111111111111111111111111111
staticpopup still taints, but it no longer blocks glyphs/talents. Although an issue could pop up some where else
How is the situation since the 5.2 patch for WoW? I read somewhere that LibChatAnims is still needed, but what about the other points? Is it still suggested to have this addon installed if LibChatAnims is already provided by some other addon?
Not only did they NOT fix the bogus taint errors with 5.2, but now this fix doesn't work either. sigh
@funkydude
I was not advocating removing the LibStub. I stripped it for testing only because LibStub loads after
Code.lua
so for the sake of simplicity I removed it.In any case, the problem was my fault. I should've paid more attention to what you said earlier.
@Fylwind, it would be useful if you answered some of the questions I asked the previous user. The entire point of using LibStub is to prevent re-hooking over and over again. Stripping it is NOT a good thing.
@Fylwind
try r14
So I just updated this addon recently and somehow I feel like I'm getting more taints than I had prior to the update. In particular, the addon will error out and tell me that some add-on has called
UIFrameFlash
, even though Igrep
ed the entire directory and found nothing of the sort. The stack-trace points to Blizzard code as far as I can tell (see below).The error very reliably occurs when I receive a Battle.net whisper — which I have a second window for, so I suspect that's where the flash occurs. The source of the taint, though, appears to be
ChatFrame_MessageEventHandler
.I copied the code from
LibChatAnims.lua
and pasted directly intoCode.lua
and surprisingly it works now (after stripping theLibStub
boilerplate). So it seems that the hack doesn't work for some reason when embedded as a library, but works fine if placed directly in the code for!NoTaint
.Getting errors
Message: Interface\AddOns\!NoTaint\Code.lua:71: AddOn calls UIFrameFlash, you may not be able to switch talent. Time: 02/13/13 19:12:57 Count: 31 Stack: [C]: ? [C]: in function `error' Interface\AddOns\!NoTaint\Code.lua:71: in function <Interface\AddOns\!NoTaint\Code.lua:68> [C]: in function `UIFrameFlash' Interface\AddOns\ChatBar\ChatBar.lua:616: in function <Interface\AddOns\ChatBar\ChatBar.lua:602>
This is a warning info that ChatBar SHOULD NOT use UIFrameFlash or it will block the talent switching sometimes in 5.1
@Winney1907
If you're running with disembedded libraries, try r14. If not, for some reason LibChatAnims hasn't loaded for you. Did you update whilst in game? If so, exit wow first.
Been getting this error with r13.
Awesome mod.
i don't doubt. just odd. I have all my guildies run this so we can raid with no taints and not be held up by reloads every boss fight to change talents and glyphs. Works fine
only DC bug people get is the rez bug that knocks you offline from desync for no real good reason that started in 5.1 :\
@mysticalos
There was a lot of lagged game textures, lag on zoning or crossing map zones (Towlong Steppes to Dread Waste by flying, for example) followed by "disconnected from server". It didn't seem to matter where I was flying from or to, or where I logged into the game. I could log into Stormwind or Shrine of the Seven Stars or Halfhill; lag, lag, disconnect.
that's weird
it works great fo rme. it really does fix 1 2 and 4, i've tested all cases thoroughly.
Obviously 3 is still very bad, i just avoid or fix mods that call UiFrameFlash() directly or pressure someone else to fix it like kemayo did with silverdragon on Jan 7th. :)
when do you get disconnected? on login, when rezzed? changing zones?
I tried this out because like most people, I keep getting taint errors. Unfortunately, with NoTaint installed, I kept getting disconnected from server.
Work nicely with static popup dialog.
Thanks a lot.
Thank you for this. I really hope it will help. This problem doen't happen very often, but when it does its a pain.
i'm one that wrote that test, haha. I wrote it to test 5.2, which actually fixes the taint on blizzards end and compare it to live. Then i tested it with your mod for a bit and found your mod didn't solve my somewhat aggressive tainttest heh.
It's nice you quickly put a fix for it though. that should indeed further reduce taints :) Keep up the good work