Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility with Inventory-Profiles #65

Open
QZLin opened this issue Sep 10, 2022 · 4 comments
Open

Compatibility with Inventory-Profiles #65

QZLin opened this issue Sep 10, 2022 · 4 comments

Comments

@QZLin
Copy link

QZLin commented Sep 10, 2022

Cannot sort oversized stack with Inventory-Profiles when join multiplayer game
blackd/Inventory-Profiles#169 (comment)

Found something funny, I use Stacker (Andrew6rant/Stacker) and Inventory-Profiles on Minecraft fabric 1.19.1.
Sort function will not work for oversized stacks if I directly join a multiplayer game. But if I leave the multiplayer game then open a local world and close it, sorting will work on both the local world and multiplayer world (for oversized stacks).

Looks like the problem is caused by Stacker because because ItemStack::getMaxCount returns 64 when connected to a server. IPN uses this value to check if the stack is oversized.
image
I suggest you report this to Stacker.

@Wubzzz1337
Copy link

Any news on this?

@Wubzzz1337
Copy link

@Wubzzz1337
Copy link

Bumping because this is a quick fix and hope dev will see it :(
@Andrew6rant

@Andrew6rant
Copy link
Owner

Stacker has a fundamental problem with the way I coded it - namely the fact that I didn't account for dedicated servers compared to integrated servers.

In earlier mod versions it wasn't too bad. I used Cloth Config as the config API (which was only built and meant for client use), so I had no protection against client-server desync if the configs did not match up. But in the switch from 1.18.1 to 1.18.2, or maybe it was 1.18.2 to 1.19, Minecraft changed the way blocks and items were registered - to be initialized on world load versus right at the start of the game. This I believe, is where the issue first popped up.

It isn't a quick fix to implement, but it is a necessary one. This was my second ever mod, and at the beginning I built it without really knowing what I was doing.

I don't have the time to dedicate to rewriting the mod at the moment, but I am planning on overhauling the whole system in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants