Fixed issues with bank search

CaerdonWardrobe.lua

@@ -939,7 +939,9 @@ function CaerdonWardrobeMixin:OnContainerFrameUpdateSearchResults(frame)
 
 		if C_Container and C_Container.GetContainerItemInfo then
 			local itemInfo = C_Container.GetContainerItemInfo(id, itemButton:GetID())
-			isFiltered = itemInfo.isFiltered
+			if itemInfo then
+				isFiltered = itemInfo.isFiltered
+			end
 		else
 			_, _, _, _, _, _, _, isFiltered = GetContainerItemInfo(id, itemButton:GetID())
 		end

CaerdonWardrobe.toc

@@ -5,7 +5,7 @@
 ## Title: Caerdon Wardrobe
 ## Notes: Shows BOE / BOA, openable, and unlearned pets, toys, mounts and wardrobe indicator on bank / bag / auction / merchant / guild bank / loot frame / loot roll / encounter journal slots
 ## Author: Caerdon
-## Version: v3.3.2
+## Version: v3.3.3
 ## SavedVariables: CaerdonWardrobeConfig
 ## OptionalDeps: WagoAnalytics, WoWUnit, AdiBags, AngrierWorldQuests, ArkInventory, Bagnon, BaudBag, cargBags_Nivaya, Combuctor, ElvUI, Inventorian, KkthnxUI, Pawn, Sorted, VenturePlan, WorldQuestsList, WorldQuestTab, WorldQuestTracker, ZygorGuidesViewer
 

features/Bags.lua

@@ -22,13 +22,21 @@ function BagsMixin:Init()
 	hooksecurefunc(ContainerFrameCombinedBags, "UpdateItems", function(...) self:OnUpdateItems(...) end)
 	hooksecurefunc(ContainerFrameCombinedBags, "UpdateSearchResults", function(...) self:OnUpdateSearchResults(...) end)
 
+	EventRegistry:RegisterCallback("ContainerFrame.OpenBag", self.BagOpened, self)
+
 	if isDragonflight then
 		return { "UNIT_SPELLCAST_SUCCEEDED", "TOOLTIP_DATA_UPDATE" }
 	else
 		return { "UNIT_SPELLCAST_SUCCEEDED" }
 	end
 end
 
+function BagsMixin:BagOpened(frame, too)
+	for i, button in frame:EnumerateValidItems() do
+		CaerdonWardrobe:SetItemButtonMogStatusFilter(button, false)
+	end
+end
+
 function BagsMixin:UNIT_SPELLCAST_SUCCEEDED(unitTarget, castGUID, spellID)
 	if unitTarget == "player" then
 		-- Tracking unlock spells to know to refresh
@@ -74,26 +82,14 @@ function BagsMixin:OnUpdateSearchResults(frame)
 
 		if C_Container and C_Container.GetContainerItemInfo then
 			local itemInfo = C_Container.GetContainerItemInfo(button:GetBagID(), button:GetID())
-			isFiltered = itemInfo and itemInfo.isFiltered
+			if itemInfo then
+				isFiltered = itemInfo.isFiltered
+			end
 		else
 			_, _, _, _, _, _, _, isFiltered = GetContainerItemInfo(button:GetBagID(), button:GetID())
 		end
 
-		-- local slot, bag = button:GetSlotAndBagID()
-		-- local item = CaerdonItem:CreateFromBagAndSlot(bag, slot)
-		if button.caerdonButton then
-			if isFiltered then
-				button.caerdonButton.mogStatus:Hide()
-				if button.caerdonButton.bindsOnText then
-					button.caerdonButton.bindsOnText:Hide()
-				end
-			else
-				button.caerdonButton.mogStatus:Show()
-				if button.caerdonButton.bindsOnText then
-					button.caerdonButton.bindsOnText:Show()
-				end
-			end
-		end
+		CaerdonWardrobe:SetItemButtonMogStatusFilter(button, isFiltered)
 	end
 end
 

features/Bank.lua

@@ -1,5 +1,4 @@
 local BankMixin = {}
-local isDragonflight = select(4, GetBuildInfo()) > 100000
 
 function BankMixin:GetName()
 	return "Bank"
@@ -23,11 +22,7 @@ function BankMixin:Init()
 	hooksecurefunc(ContainerFrame13, "UpdateItems", function(...) self:OnUpdateItems(...) end)
 	hooksecurefunc(ContainerFrame13, "UpdateSearchResults", function(...) self:OnUpdateSearchResults(...) end)
 
-	if isDragonflight then
-		return { "TOOLTIP_DATA_UPDATE" }
-	else
-		return {}
-	end
+	return { "TOOLTIP_DATA_UPDATE" }
 end
 
 function BankMixin:TOOLTIP_DATA_UPDATE()
@@ -64,6 +59,7 @@ end
 function BankMixin:OnUpdateSearchResults(frame)
 	for i, button in frame:EnumerateValidItems() do
 		local isFiltered
+
 		if C_Container and C_Container.GetContainerItemInfo then
 			local itemInfo = C_Container.GetContainerItemInfo(button:GetBagID(), button:GetID())
 			if itemInfo then
@@ -73,21 +69,7 @@ function BankMixin:OnUpdateSearchResults(frame)
 			_, _, _, _, _, _, _, isFiltered = GetContainerItemInfo(button:GetBagID(), button:GetID())
 		end
 
-		-- local slot, bag = button:GetSlotAndBagID()
-		-- local item = CaerdonItem:CreateFromBagAndSlot(bag, slot)
-		if button.caerdonButton then
-			if isFiltered then
-				button.caerdonButton.mogStatus:Hide()
-				if button.caerdonButton.bindsOnText then
-					button.caerdonButton.bindsOnText:Hide()
-				end
-			else
-				button.caerdonButton.mogStatus:Show()
-				if button.caerdonButton.bindsOnText then
-					button.caerdonButton.bindsOnText:Show()
-				end
-			end
-		end
+		CaerdonWardrobe:SetItemButtonMogStatusFilter(button, isFiltered)
 	end
 end