It used to be that the TRADE_SKILL_CLOSE event would only be fired for the character that was logged with i.e. UnitID == "player". It's no longer the case since now ppl can link their trade skills in chat. If we click such a link, we end up with a TRADE_SKILL_CLOSE event for a trade skill windows that is not the one of the currently logged character.
Anyone knows if there is a way to know what character the trade skill window was open for without having to hook ChatFrame_OnHyperlinkShow?
you could listen for TRADE_SKILL_UPDATE (maybe even TRADE_SKILL_SHOW, but i'm not sure if it fires at the exact right time) and set a variable to be the viewed unit (IsTradeSkillLinked()). then when you get your TRADE_SKILL_CLOSE you'd know who was being viewed.
i assume IsTradeSkillLinked fails by the time you get your TRADE_SKILL_CLOSE event, yeah?
From what I've been reading, Armory acts on the TRADE_SKILL_CLOSE events, performing actions (such as opening/closing headers) that causes TRADE_SKILL_UPDATE events to fire within the TRADE_SKILL_CLOSE event, and this causes issues with other addons.