changeset 153:613fd4bc6999

Ticket 45 - Fixed a bug that caused the crafting window to craft if you turn off all of the deciders. Fixed an issue where the crafting window would crash if GetTradeSkillReagentItemLink() failed to return a link. This happened with my Wild Turkey
author Asa Ayers <Asa.Ayers@Gmail.com>
date Sat, 04 Dec 2010 08:41:44 -0800
parents cbb427a8d5a5
children 090e466fd956
files CHANGELOG.txt Modules/Crafting.lua Modules/QuickAuctions.lua
diffstat 3 files changed, 11 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/CHANGELOG.txt	Mon Nov 29 11:25:55 2010 -0800
+++ b/CHANGELOG.txt	Sat Dec 04 08:41:44 2010 -0800
@@ -1,3 +1,8 @@
+2010-12-04  Asa Ayers  <Asa.Ayers@Gmail.com>
+
+- Ticket 45 - Fixed a bug that caused the crafting window to craft if you turn off all of the deciders.
+- Fixed an issue where the crafting window would crash if GetTradeSkillReagentItemLink() failed to return a link. This happened with my Wild Turkey
+
 2010-11-29  Asa Ayers  <Asa.Ayers@Gmail.com>
 
 - Fixed a bug where your equipped items were not counted as items in your inventory. If you replaced an equipped item and one of them had a cost associated, it would transfer the value between the two.
--- a/Modules/Crafting.lua	Mon Nov 29 11:25:55 2010 -0800
+++ b/Modules/Crafting.lua	Sat Dec 04 08:41:44 2010 -0800
@@ -198,7 +198,7 @@
 	if Crafting.filter_have_mats and row[6] == 0 then
 		return false
 	end
-	if strfind(row[4], 'VETO: .*') or row[5] == 0 then
+	if (row[4] and strfind(row[4], 'VETO: .*')) or row[5] == 0 then
 		return false
 	end
 	return true
--- a/Modules/QuickAuctions.lua	Mon Nov 29 11:25:55 2010 -0800
+++ b/Modules/QuickAuctions.lua	Sat Dec 04 08:41:44 2010 -0800
@@ -181,7 +181,11 @@
 
 function ItemAuditor:GetReagentCost(link, total)
 	local totalCost = 0
-	
+
+	if not link then
+		return 99990000
+	end
+
 	if PT:ItemInSet(link,"Tradeskill.Mat.BySource.Vendor") then
 		local _, _, _, _, _, _, _, _, _, _, itemVendorPrice = GetItemInfo (link);
 		totalCost = itemVendorPrice * total