From 0819822401c9f7cc6a0fc93153a3643dd23e8320 Mon Sep 17 00:00:00 2001
From: Ackis <Ackis@dummy.example.com>
Date: Tue, 21 Oct 2008 16:41:37 +0000
Subject: [PATCH] Remove global skilltype and replace it with localized
 version

---
 ARLFrame.lua        |    3 ++-
 AckisRecipeList.lua |   39 ++++++++++++++++++++++++++++++++++-----
 2 files changed, 36 insertions(+), 6 deletions(-)

diff --git a/ARLFrame.lua b/ARLFrame.lua
index cac6bc9..d91c7bc 100644
--- a/ARLFrame.lua
+++ b/ARLFrame.lua
@@ -196,13 +196,14 @@ function addon:ShowScanButton()
 		addon.ScanButton:SetPoint("RIGHT", ATSWOptionsButton, "LEFT", 0, 0)
 		addon.ScanButton:SetHeight(ATSWOptionsButton:GetHeight())
 		addon.ScanButton:SetWidth(80)
-	elseif (addon.SkillType == "Trade") then
+	else
 	-- Anchor to trade window
 		addon.ScanButton:SetParent(TradeSkillFrame)
 		addon.ScanButton:ClearAllPoints()
 		addon.ScanButton:SetPoint("RIGHT",TradeSkillFrameCloseButton,"LEFT",10,0)
 		addon.ScanButton:SetWidth(addon.ScanButton:GetTextWidth() + 10)
 	end
+
 	addon.ScanButton:SetFrameStrata("DIALOG")
 	addon.ScanButton:Show()
 
diff --git a/AckisRecipeList.lua b/AckisRecipeList.lua
index 0f0aa12..1cf7101 100644
--- a/AckisRecipeList.lua
+++ b/AckisRecipeList.lua
@@ -34,7 +34,6 @@ local DEBUG = false
 
 -- We should probably clean thes up and use arguments/return values
 -- Global variables which are used between multiple files
-addon.SkillType = nil
 
 addon.playerProfession = ""
 addon.MaxFilterIndex = 129
@@ -300,10 +299,12 @@ end
 
 function addon:TRADE_SKILL_SHOW()
 
-	addon.SkillType = "Trade"
+	addon:OpenTradeWindow()
 
 	if (addon.ScanButton and not Skillet) then
+
 		self:ShowScanButton()
+
 	end
 
 end
@@ -315,14 +316,18 @@ end
 
 function addon:TRADE_SKILL_CLOSE()
 
-	addon.SkillType = nil
+	addon:CloseTradeWindow()
 
 	if (addon.db.profile.closeguionskillclose and addon.Frame) then
+
 		self:CloseWindow()
+
 	end
 
 	if (addon.ScanButton and not Skillet) then
+
 		addon.ScanButton:Hide()
+
 	end
 
 end
@@ -1250,6 +1255,30 @@ do
 
 	local playerdata = nil
 
+	local tradewindowopened = false
+
+	-- Description: Toggles the flag that a trade window is opened
+	-- Expected result: Flag is toggled on
+	-- Input: None
+	-- Output: None
+
+	function addon:OpenTradeWindow()
+
+		tradewindowopened = true
+
+	end
+
+	-- Description: Toggles the flag that a trade window is opened
+	-- Expected result: Flag is toggled on
+	-- Input: None
+	-- Output: None
+
+	function addon:CloseTradeWindow()
+
+		tradewindowopened = false
+
+	end
+
 	-- Description: Updates the reputation table.  This only happens seldomly so I'm not worried about effeciency
 	-- Expected result: Reputation table is updated with appropiate levels
 	-- Input: None
@@ -1273,13 +1302,13 @@ do
 	function addon:AckisRecipeList_Command(textdump)
 
 		-- If we don't have a trade skill window open, lets return out of here
-		if (addon.SkillType == nil) then
+		if (not tradewindowopened) then
 
 			self:Print(L["OpenTradeSkillWindow"])
 			return
 
 		-- Trade type skills
-		elseif (addon.SkillType == "Trade") then
+		else
 
 			-- First time a scan has been run, we need to get the player specifc data, specifically faction information, profession information and other pertinant data.
 			if (playerdata == nil) then
-- 
1.7.9.5