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

Crafts appear to never end client side on terminal when relogging #6857

Closed
Swedz opened this issue Dec 25, 2022 · 3 comments
Closed

Crafts appear to never end client side on terminal when relogging #6857

Swedz opened this issue Dec 25, 2022 · 3 comments
Labels
bug Self explanatory?
Milestone

Comments

@Swedz
Copy link

Swedz commented Dec 25, 2022

Describe the bug

Sometimes crafts will remain at the pinned row of a terminal for some players on a multiplayer server even though it has completed.

How to reproduce the bug

  • Be on multiplayer
  • Request a craft from a network (ideally something that takes about half a minute, but long enough that you can do the following steps without any issue)
  • Log out
  • Wait for the craft to complete
  • Join back

Expected behavior

The crafting process to complete and not stay pinned to the top of the terminal.

Additional details

I have only tested this on fabric

Which minecraft version are you using?

1.19

On which mod loaders does it happen?

Fabric

Crash log

N/A

@Swedz Swedz added the bug Self explanatory? label Dec 25, 2022
@shartte
Copy link
Member

shartte commented Dec 25, 2022

Yes, there's sadly no way to "re-validate" a crafting job once the server has been left by the client.

We need to clear pending jobs from the client when they re-join a server. We may potentially check once if the craft is still active and keep it.

The reason this is so tricky: the crafting jobs only exist in the Crafting CPU multiblock. If the chunk containing that gets unloaded, the crafting job no longer exists for the grid. But it does come back into existence when the chunk reloads.

@CinnamonShaey
Copy link

also happens on forge sadly, a fix or a way/keybind to manually "unpin" crafts would be nice

@Technici4n Technici4n added this to the 1.19.4 milestone Feb 26, 2023
@Technici4n
Copy link
Member

An idea might be to store crafts completed while a player was offline, and send the completion packets once it logs back on.

On the other hand, this is useless if the player has closed Minecraft, so that would waste quite some storage space... Checking available jobs would probably work too, but doesn't feel great.

We should probably just clear the list when the player joins a server.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Self explanatory?
Projects
None yet
Development

No branches or pull requests

4 participants