arkayenro / arkinventory

A World of Warcraft Inventory mod for Retail, Burning Crusade, and Classic
108 stars 14 forks source link

Please help provide a work around to a long-standing issue with TSM #1786

Open Gogo1951 opened 1 year ago

Gogo1951 commented 1 year ago

So TSM has this feature that lets you send items based on rules.

However, something gets muddied when Ark is installed. Probably something around cached bag slots for each item. Dunno.

The end result is your mailing rules only half-fire.

TSM has no interest in fixing the issue. I have put in a bunch of requests to them... they just say "Use a different bag manager."

The manual fix is to log out and log back in, thus refreshing where TSM knows your items to be.

Or you can do a little wonky "restack" by moving your bags around. Swap bag 1 and 2, swap bag 3 and 4, and hit Empty on your backpack. Solves the problem.

Or you can manually drag the items that didn't mail to a new inventory spot, and then run the mailing rules again. So tedious!

It's just so annoying.

I'd love it if your "Restack" button just moved everything 1 slot over before re-stacking. I think that's all that would be needed to fix the issue with TSM.

arkayenro commented 1 year ago

AI is a virtual overaly - it changes nothing at the bag level so it cant interfere with any other mod that way.

the only time AI moves an item is during restacks or you manually moving an item to another slot. if it only happens after a restack then that would presume TSM isnt rescanning the bags even though all item moves are done by the game so are visible

worse - you never rely on cached data when looking for items as thats going to cause issues. my restack code deliberately ignores my cache and checks whats actually in each underlying bag/slot because thats the only way to be accurate, especially when moving that many items around.

if TSM is using cached data wouldnt that likely end up with the wrong item being sent/deleted/whatevered?

Gogo1951 commented 1 year ago

Yeah, not at all saying your code is at fault.

I just wrote a little macro that fixes the issue by picking everything up and moving it 1 space over.

TSM staff claimed this was caused by ArkInventory. I didn't claim that. It's fine to close this. It's just a frustrating issue that's plagued Classic since launch.

It's likely with TSM code.