ALL THE THINGS

ALL THE THINGS

31M Downloads

AllTheThings in conjunction with TSM Crafting addon crashes the game upon closing a trade skill window

dangoth opened this issue ยท 4 comments

commented

My game freezes whenever I try to close the trade skill window with both of the addons enabled (in my case, enchanting). The freeze also happens if the window closes due to another event, such as using a mage teleport skill.

commented

I can only assume this has been fixed or is in a workable state by now. If not, please refer to the (retail) errors channel on our Discord to place a more detailed report. :)

commented

I can confirm that this problem still exists in version BFA 2.0.6 together with TSM 4.9.38. Switching to the WowUI before closing the windows omits the crash.

It would be great if there was a fix for this, as I'm currently crashing my game quite often.

commented

I have dug a bit into the code and the problem seems to lie with setting the ATT frame invisible when closing the TSM windows. There is an event handler for TRADE_SKILL_CLOSE but at the same time a few lines below the show, hide and setVisible methods of the TSM frame get replaces with versions that also set the ATT visibility. This combination seems to trigger something twice. If I alter the code at line 12481 as below, the error disappears (but the ATT frame stays visible).

elseif e == "TRADE_SKILL_CLOSE" then
	if not((TSM_API and TSMAPI_FOUR) or TSMCraftingTradeSkillFrame) then
		self:SetVisible(false);
	end
end

As this prevents the problem, I will keep this modification, but there is still the issue with the ATT frame not being closed.

commented

I cannot trigger the crash now, is this still relevant?