Skip to content
New issue

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

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

Already on GitHub? # to your account

Added option to right click in chests #24

Open
wants to merge 1 commit into
base: 1.16
Choose a base branch
from

Conversation

HughBone
Copy link

@HughBone HughBone commented Apr 5, 2021

No description provided.

@kyrptonaught
Copy link
Owner

Have you verified that this does not lead to any instances of dupes? I always avoided implementing this at risk of the changes contents of the shulker not being saved properly with the chest. Can you run some tests, on a server as well? Changing the contents of the shulker, make the chunk save and load and see if it persists

@HughBone
Copy link
Author

HughBone commented Apr 6, 2021

Only tested briefly in singleplayer, I figured there was a reason you limited that functionality haha. I'll get back to you once I've ran some tests.

@kyrptonaught
Copy link
Owner

I think it's also important to know what happens when the opened shulker gets moved. What if another player opens the chest and takes out the shulker you have opened? Or if a hopper takes it? Or another player breaks the chest?

Thank you for testing this, if we can get it working properly without the potential for dupes, i'm not opposed to merging it

@HughBone
Copy link
Author

HughBone commented Apr 7, 2021

RIP, dupes do happen if a player takes the shulker box out of the chest while it's in use.

@kyrptonaught
Copy link
Owner

kyrptonaught commented Apr 7, 2021

That's unfortunate. If you can figure out a way to prevent this, feel free to. Perhaps auto closing the opened shulker when it's detected that it's moved, in an effective way. This fix could be beneficial for normal use as well. It would have to be server side of course. Maybe an event that could get fired when the item was moved? Or tick the opened handled screen and verify if the item is still in its location?

@MeeniMc
Copy link
Contributor

MeeniMc commented Feb 9, 2022

I think a limited version of this works when used only for ender-chests.

If I understand correctly the dupe process, player A opens the shulker from within the chest, player B concurrently removes the shulker from the chest, player A closes the shulker and repopulates the chest slot with the (now duplicate) shulker.

When the player is accessing the opened shulker from within the ender-chest, he cannot access the source ender-chest anymore in the interface, and other players cannot assist in duping by accessing the chest concurrently, so there should be no dupe (?)

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

Successfully merging this pull request may close these issues.

3 participants