Stack splitting broken

[kalbert312]

Addon Version

v0.1.30

World of Warcraft Version

Classic Era (And SoD/Hardcore)

Describe the bug

When trying to split a stack and move it to the "free space" slot in the bag, sometimes it will not work and spit out an error in the game's error UI: "Could not split these items." If I try to drag the entire stack to the "free space" slot, it will say "That item does not go in that container". I have no special bags, just 4 Traveler's Backpacks. Happens for any item (herbs, potions, armor, etc) when it's in this bug state. A /reload and game restart both did not resolve the issue.

Steps to reproduce

Try splitting stacks of varying numbers until it happens. I'm not sure what is the exact cause.

[Cidan]

This is for things with native stacks, right? Not "unstackable" items like weapons, etc.

[kalbert312]

This is for things with native stacks, right? Not "unstackable" items like weapons, etc.

Correct, first happened to me with a Greater Arcane Elixir stack. Once it's bugged, it happens to any type of item. I tried splitting potions and herb stacks and same result.

[Cidan]

Interesting -- okay, do you have any leads on how to reproduce this reliably? I don't play Classic but I can try to debug and fix this.

[kalbert312]

Here's a reliable repro. I moved all items to the first available slots per the show bags view (there's a couple empty slots here and there). After I split the first potion, it works, but then the second split doesn't. If I move all the items back to first available slots stack split works again.

bb-stack-issue.mp4

[Cidan]

That's interesting! Let me see if I can reproduce this locally -- BetterBags doesn't actually touch any of the split mechanics or click interactions. Blizzard doesn't allow for modification to clicks on item buttons.

[Cidan]

And just like that, I was able to reproduce it. Looking at it now, will get a fix out soon.

[Cidan]

Okay, fixed in .31, which is going out now. Re-open this bug if you see anything else, and thank you!

[kalbert312]

Still seems to be an issue after the update. I reverted to 0.1.27 and can't repro. I can't seem to reopen the issue though.

[kalbert312]

Earliest repro version looks like 0.1.28. I swapped to that and reproed immediately. 0.1.27 no issue.

[Cidan]

Alright, I'll continue to investigate.

[Cidan]

Okay, I was able to narrow it down -- working on a fix now.

[Cidan]

Alright, pushed another fix in .32. The bug was in keyrings being marked as bag free slots by accident, causing all kinds of issues. Let me know if you hit anything else.