BrannFilterBag v2.0.4 — Bugfix

Fixes

Mythic Keystone not showing in the bag

The Mythic Keystone uses a special |Hkeystone:...|h hyperlink format instead of the standard |Hitem:...|h format. All C_Item API calls (GetItemInfo, GetItemInfoInstant, GetItemNameByID) return nil for non-item links, causing the Keystone to be silently skipped during both filter evaluation and the "Other" section. A new fallback now extracts the item name directly from the hyperlink text and uses the data already provided by C_Container.GetContainerItemInfo().

"Script ran too long" error when opening the bag

The equipment set location check called EquipmentManager_GetLocationData() for every equipment slot in every set for every bag item — an O(Items × Sets × Slots) operation. For items currently equipped on the player (not in bags), the Blizzard function entered an expensive internal loop, eventually hitting the script timeout. Equipment set locations are now cached once per refresh (2-second TTL) and resolved via fast bitfield unpacking instead of the costly Blizzard API, reducing the check to O(1) per item.

Shift-clicking items posted the chat link twice

The native ContainerFrameItemButtonTemplate already handles Shift+Click via HandleModifiedItemClick(). The addon's HookScript("OnClick") was calling ChatEdit_InsertLink() on top of that, resulting in duplicate links. The redundant call has been removed from the hook. Clone buttons (items shown in multiple sections) retain their own Shift+Click logic, as they do not use the native template.