Fixed separate items sometimes showing up as one breakable

Items can have the same item ID but different rarities/levels, so now we compare the full item link instead of just the ID for determining duplicates. This allows herbs and ore and such to retain its old merging/stacking behavior within Breakables and fixes multiple items with different iLevels, for example, showing up as a single item.
This commit is contained in:
Parnic
2016-12-10 22:59:13 -06:00
parent e904b2c12f
commit b4416d6daa

View File

@ -1231,10 +1231,8 @@ function Breakables:GetItemIdFromLink(itemLink)
end
function Breakables:MergeBreakables(foundBreakable, breakableList)
local foundItemId = self:GetItemIdFromLink(foundBreakable[IDX_LINK])
for n=1,#breakableList do
local listItemId = self:GetItemIdFromLink(breakableList[n][IDX_LINK])
if foundItemId == listItemId then
if foundBreakable[IDX_LINK] == breakableList[n][IDX_LINK] then
-- always prefer the larger stack
if foundBreakable[IDX_COUNT] > breakableList[n][IDX_COUNT] then
breakableList[n][IDX_BAG] = foundBreakable[IDX_BAG]