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

🪲 Accessing Container Before Sync Erases Items #30

Closed
jonathan-robertson opened this issue Jun 13, 2023 · 1 comment · Fixed by #32
Closed

🪲 Accessing Container Before Sync Erases Items #30

jonathan-robertson opened this issue Jun 13, 2023 · 1 comment · Fixed by #32
Labels
bug Something isn't working

Comments

@jonathan-robertson
Copy link
Owner

jonathan-robertson commented Jun 13, 2023

Immediately accessing container after push and before text change causes container's text to be changed to the notification message but not swapped back to the original text after the given timer expires.

@jonathan-robertson jonathan-robertson added the bug Something isn't working label Jun 13, 2023
@jonathan-robertson jonathan-robertson changed the title 🪲 Notification Text Gets Stuck 🪲 Accessing Container Before Sync Erases Items Jun 13, 2023
@jonathan-robertson
Copy link
Owner Author

Server no longer seems to be aware of TileEntityLootContainer.IsUserAccessing which seems very strange.
How does it prevent one player from accessing a chest that another player is accessing?
Perhaps we're referencing an outdated TileEntity instance?

@jonathan-robertson jonathan-robertson linked a pull request Jun 25, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant