Merge pull request #149 from Elesario/Fishing-automation-update

Fish tracking fix and restore tracking for "spell" types

Outfitter.lua

@@ -7315,21 +7315,30 @@ function Outfitter:PlayerIsOnQuestID(pQuestID)
 	return false
 end
 
-function Outfitter:GetTrackingEnabled(pTexture)
-	local vNumTypes = GetNumTrackingTypes()
+function Outfitter:GetCurrentSpellTrackingEnabled()
+	local vNumTypes = C_Minimap.GetNumTrackingTypes();
+	for vIndex = 1, vNumTypes do
+		local vName, vTexture, vActive, vType = C_Minimap.GetTrackingInfo(vIndex);
+		if vActive and vType == "spell" then
+			return vTexture;
+		end
+	end
+end
 
+function Outfitter:GetTrackingEnabled(pTexture)
+	local vNumTypes = C_Minimap.GetNumTrackingTypes();	
 	for vIndex = 1, vNumTypes do
-		local vName, vTexture, vActive = GetTrackingInfo(vIndex)
+		local vName, vTexture, vActive = C_Minimap.GetTrackingInfo(vIndex);
 		if vTexture == pTexture then
-			return vActive, vIndex
+			return vActive, vIndex;
 		end
 	end
 end
 
 function Outfitter:SetTrackingEnabled(pTexture, pEnabled)
 	local vActive, vIndex = self:GetTrackingEnabled(pTexture)
 	if vActive ~= pEnabled then
-		SetTracking(vIndex, pEnabled == true or pEnabled == 1)
+		C_Minimap.SetTracking(vIndex, pEnabled == true or pEnabled == 1)
 	end
 end
 

OutfitterScripting.lua

@@ -821,9 +821,12 @@ if event == "OUTFIT_EQUIPPED" then
     end
     
     if setting.EnableFishTracking then
-        setting.savedTracking = Outfitter:GetTrackingEnabled(133888)
-        Outfitter:SetTrackingEnabled(133888, 1)
-        setting.didSetTracking = true
+		local _, vIndex = Outfitter:GetTrackingEnabled(133888)
+		if vIndex then
+			setting.savedTracking = Outfitter:GetCurrentSpellTrackingEnabled()
+			Outfitter:SetTrackingEnabled(133888, 1)
+			setting.didSetTracking = true
+		end
     end
     
    if setting.DisableClicktoMove then
@@ -848,7 +851,11 @@ elseif event == "OUTFIT_UNEQUIPPED" then
    end
  
    if setting.EnableFishTracking and setting.didSetTracking then
-       Outfitter:SetTrackingEnabled(133888, setting.savedTracking)
+       if setting.savedTracking then
+	      Outfitter:SetTrackingEnabled(setting.savedTracking, true)
+	   else
+	      Outfitter:SetTrackingEnabled(133888, false)  -- no spell tracking was enabled
+	   end
        setting.didSetTracking = nil
        setting.savedTracking = nil
    end