Fixed a long standing bug where swapped items (equipment swap, crafti…

…ng last mat into an item) would sometimes redraw the bag or not be marked as new. By default, all new incoming items now go to recent items (can be turned off).
Cidan · Sep 2, 2024 · 358504b · 358504b
1 parent 050e418
commit 358504b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/core/constants.lua b/core/constants.lua
@@ -528,11 +528,11 @@ const.DATABASE_DEFAULTS = {
     },
     newItems = {
       [const.BAG_KIND.BACKPACK] = {
-        markRecentItems = false,
+        markRecentItems = true,
         showNewItemFlash = false,
       },
       [const.BAG_KIND.BANK] = {
-        markRecentItems = false,
+        markRecentItems = true,
         showNewItemFlash = false,
       },
     },

diff --git a/data/items.lua b/data/items.lua
@@ -691,6 +691,9 @@ function items:LoadItems(ctx, kind, dataCache, equipmentCache, callback)
       slotInfo:AddToAddedItems(currentItem)
       search:Remove(previousItem)
       search:Add(currentItem)
+      if not ctx:GetBool('wipe') and addon.isRetail and database:GetMarkRecentItems(kind) and currentItem.itemInfo.itemID ~= previousItem.itemInfo.itemID then
+        self:MarkItemAsNew(ctx, currentItem)
+      end
     elseif items:ItemGUIDChanged(currentItem, previousItem) then
       debug:Log("ItemGUIDChanged", currentItem.itemInfo.itemLink)
       slotInfo:AddToRemovedItems(previousItem)