wow/reagentmaker: data.lua comparison

Avoid scanning when tradeskill is not fully loaded (no recipe or the first one is not a header)

comparison

equal deleted inserted replaced

-:578b9c9479c9
+:886be292feec
 	local GetTradeSkillReagentInfo = GetTradeSkillReagentInfo
 	local GetTradeSkillNumMade = GetTradeSkillNumMade
 	-- the function who scans the tradeskill
 	function A:ScanSimpleRecipes()
-		for i = 1,GetNumTradeSkills() do
+		-- Check if the tradeskill is loaded
+		-- Has to have recipes and begin with a header
+		local NRecipes = GetNumTradeSkills()
+		if NRecipes==0 or select(2,GetTradeSkillInfo(1))~="header" then
+			return
+		end
+		for i = 2,NRecipes do -- The first one is a header
 			local skillName, skillType, numAvailable, isExpanded, serviceType, numSkillUps = GetTradeSkillInfo(i)
 			if skillType ~= "header" then
 				if GetTradeSkillNumReagents(i) == 1 then
 					-- item ID
 					local itemID = A.link2ID(GetTradeSkillItemLink(i))
 						tinsert(A.data[itemID],{reagentID,reagentCount,minMade,maxMade,spellName=skillName})
 					end -- if
 				end -- if
 			end -- if
 		end -- for
+		-- the scanning is complete
+		return true
 	end -- function
 end -- do

Mercurial > wow > reagentmaker