Describe the bug

Wrapping a battle pet item into a gift doesn't update the icon and is not moved to the correct category.

To Reproduce

1. Purchase a bunch of [Red Ribboned Wrapping Paper] from Brog Hamfist in Goldshire Inn.
2. Put any battle pet to cage.
3. Click "Recent items" label to move the battle pet item under "Battle pets".
4. Wrap the battle pet item into a [Red Ribboned Wrapping Paper].

Expected behavior

• The item icon is changed to the [Red Ribboned Gift].
• The gift item is moved to the "Consumable" section.
• The tooltip says "Red Ribboned Gift".

Actual behavior

• The battle pet icon is unchanged.
• The gift item is still under the "Battle pets" section.
• The tooltip says "Red Ribboned Gift".

Screenshots

World of Warcraft Release and Version (Retail 9.2.7, etc)

• AdiBags 1.9.26
• WoW Retail 9.2.7.45114

Have you tried disabling all your other addons to ensure only this addon causes the issue?

• AdiBags and AdiBags Configuration are the 2 only add-ons loaded.

Additional context

• Game language set to English.
• Fresh AdiBags installation (delete the AdiBags.lua file from SavedVariables).

Closing then opening the bag again fixes the icon. However, the item is still showing under the "Battle pets" section instead of "Consumables". The only way to have it under the right category is to do a /reload.

For other item types (ie a piece of equipment), the icon seems to be properly updated but the resulting wrapped gift is not moved to its new category until the bag is closed and open again.

Thanks for the report! We'll take a look at this soon.

So I just tested it, and figuring out if it was previously in recent items would be excessive work for a somewhat weird UX. Making it materialize in recent items "feels" better as a UX, because you'll likely want to trade it right then and there.

I'm going to go ahead with the recent items for all new wraps, if you don't mind.

I'm going to go ahead with the recent items for all new wraps

It might actually even be better because the wraped gift would be easier to find in "Recent Items" than in another category. And from there it would be still possible to sort it to its category by clicking "recent items".

Agreed -- this also applies to the reverse. An unwrap will show up in recent items.

Fixed in master, will go out in next release, or clone master to use now!

I just tested the fix. The texture is properly updated as expected but the item still remains in its current category ("Battle pets") until the bags are closed and opened again wher it appears under the right category ("Consumables"). Is it something that would be complicated to fix ? Should I raise another ticket for this case ?

Hm, would the item appearing in "recent items" be acceptable? I feel like that should be the use case here, but I'm curious as to your feedback.

If the item was previously in "recent items", I think it would make sense to leave it in "Recent Items", but if the item was already put in a category using the filter/rules, I would run the filters again on the item to move it to a new category if applicable.