v4.3.13: cross-version compat wrappers, updated README, all TOCs bumped

Author: paradosi

CHANGELOG.md

@@ -2,6 +2,23 @@
 
 All notable changes to Postal (TBC Anniversary) will be documented in this file.
 
+## [4.3.13] - 2026-02-17
+
+### Fixed
+- **Cross-version compatibility**: Added safe wrappers for all container and item APIs that differ between WoW versions
+  - `C_Container.GetContainerNumFreeSlots` / `GetContainerNumFreeSlots`
+  - `C_Container.GetContainerNumSlots` / `GetContainerNumSlots`
+  - `C_Container.GetContainerItemID` / `GetContainerItemID`
+  - `C_Container.PickupContainerItem` / `PickupContainerItem`
+  - `C_Container.GetContainerFreeSlots` / `GetContainerFreeSlots`
+  - `C_Item.GetItemInfo` / `GetItemInfo`
+  - `C_Item.GetItemCount` / `GetItemCount`
+- **Select.lua**: Replaced all hardcoded `C_Container` and `C_Item` calls with safe compat wrappers
+- **OpenAll.lua**: Replaced direct `C_Container.GetContainerItemInfo` and `C_Item` calls with compat wrappers; added nil guard for `OpenAllMail` (may not exist on all clients)
+- **Express.lua**: Replaced `C_Item.GetItemInfo` calls with safe wrapper
+- **Forward.lua**: Replaced all `C_Container` and `C_Item` calls with safe wrappers; removed version-branching `if/else` blocks
+- **QuickAttach.lua**: Replaced `C_Item.GetItemInfo` call with safe wrapper
+
 ## [4.3.12] - 2026-02-15
 
 ### Fixed

Modules/Express.lua

@@ -179,7 +179,7 @@ function Postal_Express:ContainerFrameItemButtonOnModifiedClick(bag, slot, butto
 		if not itemid then return end
 		local itemInfo = Postal:GetContainerItemInfoCompat(bag, slot)
 		local itemlocked = itemInfo and itemInfo.isLocked or false
-		local itemq, _,_, itemc, itemsc, _, itemes = select(3,C_Item.GetItemInfo(itemid))
+		local itemq, _,_, itemc, itemsc, _, itemes = select(3,Postal.GetItemInfoSafe(itemid))
 		itemes = itemes and #itemes > 0
 		if Postal.db.profile.Express.BulkSend and itemq and itemc then
 			local itemsinmail = 0
@@ -203,7 +203,7 @@ function Postal_Express:ContainerFrameItemButtonOnModifiedClick(bag, slot, butto
 						if not tid or itemlocked2 or Postal_Express_IsSoulbound(b, s) then
 							-- item locked, already attached, soulbound
 						else
-							local tq, _,_, tc, tsc, _, tes = select(3,C_Item.GetItemInfo(tid))
+							local tq, _,_, tc, tsc, _, tes = select(3,Postal.GetItemInfoSafe(tid))
 							-- tc = (tq or "").."."..(tc or "")
 							tsc = (tc or "").."."..(tsc or "")
 							tes = tes and #tes > 0

Modules/Forward.lua

@@ -51,7 +51,7 @@ local function ContainsStackableItem(messageindex)
 	for itemIndex = 1, ATTACHMENTS_MAX_SEND do
 		itemID = select(2, GetInboxItem(messageindex, itemIndex))
 		if itemID ~= nil then
-			itemStackCount = select(8,C_Item.GetItemInfo(itemID))
+			itemStackCount = select(8,Postal.GetItemInfoSafe(itemID))
 			if itemStackCount > 1 then return true end
 		end
 	end
@@ -64,11 +64,7 @@ local function FreeBagSpace()
 	local FreeSpace = 0
 	for bagID = 0, 4, 1 do
 		local numberOfFreeSlots, bagType
---		if Postal.WOWBCClassic then
---			numberOfFreeSlots, bagType = GetContainerNumFreeSlots(bagID)
---		else
-			numberOfFreeSlots, bagType = C_Container.GetContainerNumFreeSlots(bagID)
---		end
+			numberOfFreeSlots, bagType = Postal.GetContainerNumFreeSlotsSafe(bagID)
 		FreeSpace = FreeSpace + numberOfFreeSlots
 	end
 	return FreeSpace
@@ -82,9 +78,9 @@ local function Postal_Inventory_Change(action)
 		wipe(PostalForwardTable)
 		for bagID = 0, 4, 1 do
 			if Postal.WOWBCClassic then
-				table.insert(PostalForwardTable, GetContainerFreeSlots(bagID))
+				table.insert(PostalForwardTable, Postal.GetContainerFreeSlotsSafe(bagID))
 			else
-				table.insert(PostalForwardTable, C_Container.GetContainerFreeSlots(bagID))
+				table.insert(PostalForwardTable, Postal.GetContainerFreeSlotsSafe(bagID))
 			end
 		end
 		return 0, 0
@@ -93,9 +89,9 @@ local function Postal_Inventory_Change(action)
 		wipe(TempTable)
 		for bagID = 0, 4, 1 do
 			if Postal.WOWBCClassic then
-				TempTable = GetContainerFreeSlots(bagID)
+				TempTable = Postal.GetContainerFreeSlotsSafe(bagID)
 			else
-				TempTable = C_Container.GetContainerFreeSlots(bagID)
+				TempTable = Postal.GetContainerFreeSlotsSafe(bagID)
 			end
 			for Key = 1, #PostalForwardTable[bagID + 1], 1 do
 				if not contains(TempTable, PostalForwardTable[bagID + 1][Key]) then
@@ -159,11 +155,7 @@ function Postal_Forward_ForwardMailItems(action)
 	if action == 2 then
 		Postal_Forward:UnregisterEvent("BAG_UPDATE_DELAYED","Postal_Forward_ForwardMailItemsEvent")
 		bagID, itemIndex = Postal_Inventory_Change(2)
-		if Postal.WOWBCClassic then
-			PickupContainerItem(bagID, itemIndex)
-		else
-			C_Container.PickupContainerItem(bagID, itemIndex)
-		end
+		Postal.PickupContainerItemSafe(bagID, itemIndex)
 		ClickSendMailItemButton()
 		Postal_Forward_ForwardMailItems(1)
 		return

Modules/OpenAll.lua

@@ -13,7 +13,7 @@ Postal_OpenAll.description2 = L[ [[|cFFFFCC00*|r Simple filters are available fo
 local MAX_MAIL_SHOWN = 100
 local GetContainerNumFreeSlotsSafe = (C_Container and C_Container.GetContainerNumFreeSlots) or GetContainerNumFreeSlots
 local GetContainerNumSlotsSafe = (C_Container and C_Container.GetContainerNumSlots) or GetContainerNumSlots
-local GetContainerItemInfoSafe = C_Container and C_Container.GetContainerItemInfo or nil
+-- Uses Postal:GetContainerItemInfoCompat() from Postal.lua
 local mailIndex, attachIndex
 local numUnshownItems
 local lastItem, lastNumAttach, lastNumGold
@@ -133,7 +133,7 @@ function Postal_OpenAll:OnEnable()
 		Postal_OpenAll:RegisterEvent("PLAYER_INTERACTION_MANAGER_FRAME_SHOW")
 	end
 	-- For enabling after a disable
-	OpenAllMail:Hide() -- hide Blizzard's Open All button
+	if OpenAllMail then OpenAllMail:Hide() end -- hide Blizzard's Open All button
 	button:Show()
 	Postal_OpenAllMenuButton:SetScript("OnHide", Postal_DropDownMenu.HideMenu)
 	Postal_OpenAllMenuButton:Show()
@@ -142,7 +142,7 @@ end
 function Postal_OpenAll:OnDisable()
 	self:Reset()
 	button:Hide()
-	OpenAllMail:Show() -- show Blizzard's Open All button
+	if OpenAllMail then OpenAllMail:Show() end -- show Blizzard's Open All button
 	Postal_OpenAllMenuButton:SetScript("OnHide", nil)
 	Postal_OpenAllMenuButton:Hide()
 end
@@ -306,24 +306,19 @@ function Postal_OpenAll:ProcessNext()
 			local name, itemID, itemTexture, count, quality, canUse = GetInboxItem(mailIndex, attachIndex)
 			local link = GetInboxItemLink(mailIndex, attachIndex)
 			local itemID = strmatch(link, "item:(%d+)")
-			local stackSize = select(8, C_Item.GetItemInfo(link))
-			if itemID and stackSize and C_Item.GetItemCount(itemID) > 0 then
+			local stackSize = select(8, Postal.GetItemInfoSafe(link))
+			if itemID and stackSize and Postal.GetItemCountSafe(itemID) > 0 then
 				for bag = 0, NUM_BAG_SLOTS do
 					local containerNumSlots = GetContainerNumSlotsSafe(bag)
 					for slot = 1, containerNumSlots do
 						local count2, link2
-						if Postal.WOWBCClassic then
-							count2 = select(2, GetContainerItemInfo(bag, slot))
-							link2 = select(7, GetContainerItemInfo(bag, slot))
+						local itemInfo = Postal:GetContainerItemInfoCompat(bag, slot)
+						if itemInfo then
+							count2 = itemInfo.stackCount
+							link2 = itemInfo.hyperlink
 						else
-							if GetContainerItemInfoSafe and GetContainerItemInfoSafe(bag, slot) then
-								local itemInfo = GetContainerItemInfoSafe(bag, slot)
-								count2 = itemInfo.stackCount
-								link2 = itemInfo.hyperlink
-							else
-								count2 = 0
-								link2 = nil
-							end
+							count2 = 0
+							link2 = nil
 						end
 						if link2 then
 							local itemID2 = strmatch(link2, "item:(%d+)")

Modules/QuickAttach.lua

@@ -223,7 +223,7 @@ function Postal_QuickAttachLeftButtonClick(classID, subclassID)
 					itemID = itemInfo and itemInfo.itemID or nil
 					if itemID then
 						if not itemInfoCache[itemID] then
-							itemInfoCache[itemID] = {C_Item.GetItemInfo(itemID)}
+							itemInfoCache[itemID] = {Postal.GetItemInfoSafe(itemID)}
 						end
 						local info = itemInfoCache[itemID]
 						bindType = info[14]

Modules/Select.lua

@@ -350,11 +350,7 @@ function Postal_Select:ProcessNext()
 				local free = 0
 				for bag = 0, NUM_BAG_SLOTS do
 					local bagFree, bagFam
---					if Postal.WOWBCClassic then
---						bagFree, bagFam = GetContainerNumFreeSlots(bag)
---					else
-						bagFree, bagFam = C_Container.GetContainerNumFreeSlots(bag)
---					end
+						bagFree, bagFam = Postal.GetContainerNumFreeSlotsSafe(bag)
 					if bagFam == 0 then
 						free = free + bagFree
 					end
@@ -374,29 +370,19 @@ function Postal_Select:ProcessNext()
 				local name, itemID, itemTexture, count, quality, canUse = GetInboxItem(mailIndex, attachIndex)
 				local link = GetInboxItemLink(mailIndex, attachIndex)
 				itemID = strmatch(link, "item:(%d+)")
-				local stackSize = select(8, C_Item.GetItemInfo(link))
-				if itemID and stackSize and C_Item.GetItemCount(itemID) > 0 then
+				local stackSize = select(8, Postal.GetItemInfoSafe(link))
+				if itemID and stackSize and Postal.GetItemCountSafe(itemID) > 0 then
 					for bag = 0, NUM_BAG_SLOTS do
-						local ContainerNumSlots
-						if Postal.WOWBCClassic then
-							ContainerNumSlots = GetContainerNumSlots(bag)
-						else
-							ContainerNumSlots = C_Container.GetContainerNumSlots(bag)
-						end
+						local ContainerNumSlots = Postal.GetContainerNumSlotsSafe(bag)
 						for slot = 1, ContainerNumSlots do
 							local count2, link2
-							if Postal.WOWBCClassic then
-								count2 = select(2, GetContainerItemInfo(bag, slot))
-								link2 = select(7, GetContainerItemInfo(bag, slot))
+							local itemInfo = Postal:GetContainerItemInfoCompat(bag, slot)
+							if itemInfo then
+								count2 = itemInfo.stackCount
+								link2 = itemInfo.hyperlink
 							else
-								if C_Container and C_Container.GetContainerItemInfo(bag, slot) then
-									local itemInfo = C_Container.GetContainerItemInfo(bag, slot)
-									count2 = itemInfo.stackCount
-									link2 = itemInfo.hyperlink
-								else
-									count2 = 0
-									link2 = nil
-								end
+								count2 = 0
+								link2 = nil
 							end
 							if link2 then
 								local itemID2 = strmatch(link2, "item:(%d+)")

Postal.lua

@@ -117,6 +117,27 @@ end
 -- Shared Compat Utilities  --
 -------------------------------
 
+-- Container API wrappers: prefer C_Container namespace (Retail/modern), fall back to legacy globals (Classic/TBC)
+local GetContainerNumFreeSlotsSafe = (C_Container and C_Container.GetContainerNumFreeSlots) or GetContainerNumFreeSlots
+local GetContainerNumSlotsSafe = (C_Container and C_Container.GetContainerNumSlots) or GetContainerNumSlots
+local GetContainerItemIDSafe = (C_Container and C_Container.GetContainerItemID) or GetContainerItemID
+local PickupContainerItemSafe = (C_Container and C_Container.PickupContainerItem) or PickupContainerItem
+local GetContainerFreeSlotsSafe = (C_Container and C_Container.GetContainerFreeSlots) or GetContainerFreeSlots
+
+-- Item API wrappers: prefer C_Item namespace (Retail/modern), fall back to legacy globals (Classic/TBC)
+local GetItemInfoSafe = (C_Item and C_Item.GetItemInfo) or GetItemInfo
+local GetItemCountSafe = (C_Item and C_Item.GetItemCount) or GetItemCount
+
+-- Export safe wrappers so modules can use them via Postal
+Postal.GetContainerNumFreeSlotsSafe = GetContainerNumFreeSlotsSafe
+Postal.GetContainerNumSlotsSafe = GetContainerNumSlotsSafe
+Postal.GetContainerItemIDSafe = GetContainerItemIDSafe
+Postal.PickupContainerItemSafe = PickupContainerItemSafe
+Postal.GetContainerFreeSlotsSafe = GetContainerFreeSlotsSafe
+Postal.GetItemInfoSafe = GetItemInfoSafe
+Postal.GetItemCountSafe = GetItemCountSafe
+
+-- GetContainerItemInfo compat: returns a table with .iconFileID, .stackCount, .isLocked, .hyperlink, .itemID
 local GetContainerItemInfoSafe = C_Container and C_Container.GetContainerItemInfo or nil
 local GetContainerItemInfoLegacy = GetContainerItemInfo
 

Postal.toc

@@ -1,8 +1,8 @@
 ## Interface: 110205
 ## Title: Postal
-## Version: 4.3.12
-## X-Build: 663
-## X-ReleaseDate: 2025-10-29T20:49:10Z
+## Version: 4.3.13
+## X-Build: 673
+## X-ReleaseDate: 2026-02-17T00:00:00Z
 ## X-Revision: 663
 ## Author: paradosi@Dreamscythe
 ## X-Credits: Ammo, Rabbit, Grennon, Mikk, oscarucb, Jonny aka The_Original_Manbot

Postal_Cata.toc

@@ -1,8 +1,8 @@
 ## Interface: 40402
 ## Title: Postal
-## Version: v4.3.4
-## X-Build: 663
-## X-ReleaseDate: 2025-10-29T20:49:10Z
+## Version: 4.3.13
+## X-Build: 673
+## X-ReleaseDate: 2026-02-17T00:00:00Z
 ## X-Revision: 663
 ## Author: paradosi@Dreamscythe
 ## X-Credits: Ammo, Rabbit, Grennon, Mikk, oscarucb, Jonny aka The_Original_Manbot

Postal_Mists.toc

@@ -1,8 +1,8 @@
 ## Interface: 50502
 ## Title: Postal
-## Version: v4.3.4
-## X-Build: 663
-## X-ReleaseDate: 2025-10-29T20:49:10Z
+## Version: 4.3.13
+## X-Build: 673
+## X-ReleaseDate: 2026-02-17T00:00:00Z
 ## X-Revision: 663
 ## Author: paradosi@Dreamscythe
 ## X-Credits: Ammo, Rabbit, Grennon, Mikk, oscarucb, Jonny aka The_Original_Manbot