Mercurial > wow > itemauditor
comparison Modules/Events.lua @ 52:9c95158236d1
Fixed the way I was calculating the value when distributing cost among multiple items.
author | Asa Ayers <Asa.Ayers@Gmail.com> |
---|---|
date | Wed, 21 Jul 2010 20:36:56 -0700 |
parents | 9ff6a3b02332 |
children | c6c98fc1be8e |
comparison
equal
deleted
inserted
replaced
51:84bfb5469189 | 52:9c95158236d1 |
---|---|
233 If something has never been seen on the AH, it must not be very valuable. | 233 If something has never been seen on the AH, it must not be very valuable. |
234 I'm using 1c so it doesn't have much weight and I can't get a devided by zero error. | 234 I'm using 1c so it doesn't have much weight and I can't get a devided by zero error. |
235 The only time I know that this is a problem is when crafting a BOP item, and it | 235 The only time I know that this is a problem is when crafting a BOP item, and it |
236 is always crafted 1 at a time, so a weight of 1 will work. | 236 is always crafted 1 at a time, so a weight of 1 will work. |
237 ]] | 237 ]] |
238 local ap = (addon:GetAuctionPrice(link) or 1) | 238 local ap = (addon:GetAuctionPrice(link) or 1) * change |
239 totalWeight = totalWeight + ap | 239 totalWeight = totalWeight + ap |
240 weights[link] = ap | 240 weights[link] = ap |
241 end | 241 end |
242 | 242 |
243 local valuePerPoint = totalValue / totalWeight | |
244 | |
245 for link, change in pairs(targetItems) do | 243 for link, change in pairs(targetItems) do |
246 self:SaveValue(link, weights[link] * valuePerPoint, change) | 244 local value = totalValue * (weights[link]/totalWeight) |
245 self:SaveValue(link, value, change) | |
247 end | 246 end |
248 end | 247 end |
249 | 248 |
250 function addon:UpdateAudit() | 249 function addon:UpdateAudit() |
251 -- self:Debug("UpdateAudit " .. event) | 250 -- self:Debug("UpdateAudit " .. event) |