changeset 138:7d258c041b11

Fixed an issue with /ia invested that caused all items to display twice. This was introduced by the fix in Ticket 42. At this point ItemAuditor watches mail for auctions sold or purchased, watches for buy/sell (money and 1 item type change) and conversions/tradeskills. Milling isn't working yet because there is too much time between the first event and the last event.
author Asa Ayers <Asa.Ayers@Gmail.com>
date Sat, 02 Oct 2010 20:34:07 -0700
parents 526036e4358f
children 7e1496b25311
files CHANGELOG.txt Modules/DisplayInvested.lua
diffstat 2 files changed, 9 insertions(+), 8 deletions(-) [+]
line wrap: on
line diff
--- a/CHANGELOG.txt	Sat Oct 02 19:21:56 2010 -0700
+++ b/CHANGELOG.txt	Sat Oct 02 20:34:07 2010 -0700
@@ -3,6 +3,7 @@
 - I removed the persistent queue (Ticket 33) It caused to many bugs and was blocking me from getting other things done. I think its something that will get implemented again once I figure out the best way to do it.
 - Ticket 43 - Updated the way ItemAuditor counts how many of each item you own to only count your current faction.
 - Ticket 42 - In order to fix an issue with having multiple items that are the same base item with different enchants, all costs are now tracked against the base item instead of the exact item.
+- Fixed an issue with /ia invested that caused all items to display twice. This was introduced by the fix in Ticket 42.
 
 2010-09-15  Asa Ayers  <Asa.Ayers@Gmail.com>
 
@@ -240,4 +241,4 @@
 - Removing libs from my repo so I can use externals instead
 - comment/debugging code cleanup
 - First public revision.
-- At this point ItemAuditor watches mail for auctions sold or purchased, watches for buy/sell (money and 1 item type change) and conversions/tradeskills. Milling isn't working yet because there is too much time between the first event and the last event.
\ No newline at end of file
+- At this point ItemAuditor watches mail for auctions sold or purchased, watches for buy/sell (money and 1 item type change) and conversions/tradeskills. Milling isn't working yet because there is too much time between the first event and the last event.
--- a/Modules/DisplayInvested.lua	Sat Oct 02 19:21:56 2010 -0700
+++ b/Modules/DisplayInvested.lua	Sat Oct 02 20:34:07 2010 -0700
@@ -185,20 +185,20 @@
 					investedPerItem,
 					count,
 				}
-				
+
 				totalInvested = totalInvested + investedTotal
-				
+
 				i = i + 1
-				includedItems[safeLink] = true
+				includedItems[ItemAuditor:GetIDFromLink(safeLink)] = true
 			end
 		end
 		
 		local inventory = ItemAuditor:GetCurrentInventory()
 		
-		for link, count in pairs(inventory.items) do
-			if includedItems[link] == nil then
-				local count = ItemAuditor:GetItemCount(ItemAuditor:GetIDFromLink(link))
-				local itemName, link = GetItemInfo(link)
+		for itemID, count in pairs(inventory.items) do
+			if includedItems[itemID] == nil then
+				local itemName, link = GetItemInfo(itemID)
+				local count = ItemAuditor:GetItemCount(itemID)
 				tableData[i] = {
 					itemName.."|"..link,
 					0,