Mercurial > wow > itemauditor
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,