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

Player#closeInventory does not work in client-side for Bedrock players #5259

Open
bilektugrul opened this issue Jan 4, 2025 · 3 comments
Open
Labels
Confirmed Bug The bug reported is confirmed and able to be replicated.

Comments

@bilektugrul
Copy link

Describe the bug

When Player#closeInventory is executed for a Bedrock player, it closes the open inventory in server-side, but Bedrock client will still have the inventory GUI open. This leads to a desync.

To Reproduce

  1. Create a command that waits 2-3 seconds after its usage then executes Player#closeInventory. Preferably add Player#getCurrentInventory debugs before and after the wait.
  2. Execute the command and open any GUI like player inventory, chest inventory or crafting inventory.
  3. After set wait seconds, GUI will be closed in server-side but client will still see it.

Expected behaviour

GUI should be closed in both sides, server and client.

Screenshots / Videos

No response

Server Version and Plugins

No response

Geyser Dump

No response

Geyser Version

1.2.0-SNAPSHOT

Minecraft: Bedrock Edition Device/Version

Latest

Additional Context

No response

@bilektugrul bilektugrul changed the title Player#closeInventory does not work for Bedrock players Player#closeInventory does not work in client-side for Bedrock players Jan 4, 2025
@onebeastchris onebeastchris added the Confirmed Bug The bug reported is confirmed and able to be replicated. label Jan 14, 2025
@onebeastchris
Copy link
Member

Marking as confirmed - bedrock unfortunately broke this in an odd way where only some containers (e.g. chests can be closed via the method we used before). I'm working on implementing hacky alternatives wherever possible, however, closing the player inventory might not be possible nowadays :(

@bilektugrul
Copy link
Author

Marking as confirmed - bedrock unfortunately broke this in an odd way where only some containers (e.g. chests can be closed via the method we used before). I'm working on implementing hacky alternatives wherever possible, however, closing the player inventory might not be possible nowadays :(

Please let me know about ETAs if you can provide. Thanks for the work.

@onebeastchris
Copy link
Member

as always, none - we work on this in our free time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Confirmed Bug The bug reported is confirmed and able to be replicated.
Projects
None yet
Development

No branches or pull requests

2 participants