wow/itemauditor: Modules/Crafting.lua comparison

comparison Modules/Crafting.lua @ 74:71de6e86a1a4 release 2010-07-29

Fixed a bug where tradeskill that don't produce items, like Inscription Research, cause IA to crash. Added some API enhancements. Added ArkInventory as an optional dependency so it will be loaded before IA.

author	Asa Ayers <Asa.Ayers@Gmail.com>
date	Thu, 29 Jul 2010 22:33:50 -0700
parents	8e9beb8a0330
children	a8fc802b42ba

comparison

equal deleted inserted replaced

-:cd00b87fad31
+:71de6e86a1a4
 		data = ItemAuditor:GetCraftingRow(index)
 	end
 end
+-- ItemAuditor:GetModule('Crafting').filter_queued = false
+Crafting.filter_queued = true
+local function tableFilter(self, row, ...)
+	-- column 5 is how many should be crafted
+	if Crafting.filter_queued and row[5] <= 0 then
+		return false
+	end
+	return true
+end
 local craftingContent = false
 local craftingTable = false
 local btnProcess = false
 local function ShowCrafting(container)
 	if craftingContent == false then
 end
 local lastWinnder = ""
 local function Decide(data)
 	local newDecision = 0
+	local reason = ""
 	for name, decider in pairs(craftingDeciders) do
 		if not ItemAuditor.db.profile.disabled_deciders[name] and name ~= lastWinner then
-			newDecision = decider(data)
+			newDecision, reason = decider(data)
 			if newDecision > data.queue then
 				data.queue = newDecision
-				lastWinner = name
+				lastWinner = (reason or name)
 				return Decide(data)
 			elseif newDecision < 0 then
 				lastWinner = ""
-				return 'VETO: '..name, 0
+				return 'VETO: '..(reason or name), -1
 			end
 		end
 	end
 	winner = lastWinner
 	return -1
 end
 Crafting.RegisterCraftingDecider('Is Profitable', isProfitable)
-local function tableFilter(self, row, ...)
-	-- column 5 is how many should be crafted
-	return row[5] > 0
-end
 local tableData = {}
 function ItemAuditor:UpdateCraftingTable()
 	if LSW == nil then
 		self:Print("This feature requires LilSparky's Workshop.")
 		local recipeLink = GetTradeSkillRecipeLink(i)
 		local stackSize  = 1
 		if recipeLink ~= nil and itemId ~= nil then
 			local skillName, skillType, numAvailable, isExpanded, altVerb = GetTradeSkillInfo(i)
 			local itemName, itemLink= GetItemInfo(itemId)
-			local count = Altoholic:GetItemCount(itemId)
+			-- This check has to be here for things like Inscription Research that don't produce an item.
-			local reagents = {}
+			if itemLink then
-			local totalCost = 0
+				local count = Altoholic:GetItemCount(itemId)
-			for reagentId = 1, GetTradeSkillNumReagents(i) do
+				local reagents = {}
-				local reagentName, _, reagentCount = GetTradeSkillReagentInfo(i, reagentId);
+				local totalCost = 0
-				local reagentLink = GetTradeSkillReagentItemLink(i, reagentId)
+				for reagentId = 1, GetTradeSkillNumReagents(i) do
+					local reagentName, _, reagentCount = GetTradeSkillReagentInfo(i, reagentId);
+					local reagentLink = GetTradeSkillReagentItemLink(i, reagentId)
+					reagents[reagentId] = {
+						name = reagentName,
+						count = reagentCount,
+						price = self:GetReagentCost(reagentLink, reagentCount),
+					}
+					totalCost  = totalCost + self:GetReagentCost(reagentLink, reagentCount)
+				end
+				local data = {
+					recipeLink = recipeLink,
+					link = itemLink,
+					name = itemName,
+					count = count,
+					price = (self:GetAuctionPrice(itemLink) or 0),
+					cost = totalCost,
+					profit = (self:GetAuctionPrice(itemLink) or 0) - totalCost,
+					reagents = reagents,
+					count = count,
+					tradeSkillIndex = i,
+					queue = 0,
+					winner = "",
+				}
-				reagents[reagentId] = {
+				data.winner, data.queue = Decide(data)
-					name = reagentName,
+				data.queue = data.queue - count
-					count = reagentCount,
-					price = self:GetReagentCost(reagentLink, reagentCount),
+				-- If a tradeskill makes 5 at a time and something asks for 9, we should only
-				}
+				-- craft twice to get 10.
-				totalCost  = totalCost + self:GetReagentCost(reagentLink, reagentCount)
+				data.queue = ceil(data.queue / GetTradeSkillNumMade(i))
-			end
+				realData[row] = data
-			local data = {
+				row = row + 1
-				recipeLink = recipeLink,
+			end
-				link = itemLink,
-				name = itemName,
-				count = count,
-				price = (self:GetAuctionPrice(itemLink) or 0),
-				cost = totalCost,
-				profit = (self:GetAuctionPrice(itemLink) or 0) - totalCost,
-				reagents = reagents,
-				count = count,
-				tradeSkillIndex = i,
-				queue = 0,
-				winner = "",
-			}
-			data.winner, data.queue = Decide(data)
-			data.queue = data.queue - count
-			-- If a tradeskill makes 5 at a time and something asks for 9, we should only
-			-- craft twice to get 10.
-			data.queue = ceil(data.queue / GetTradeSkillNumMade(i))
-			realData[row] = data
-			row = row + 1
 		end
 	end
 	table.sort(realData, function(a, b) return a.profit*a.queue > b.profit*b.queue end)
 	if craftingTable then
 		craftingTable:SetFilter(tableFilter)

Mercurial > wow > itemauditor

comparison Modules/Crafting.lua @ 74:71de6e86a1a4 release 2010-07-29