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

Not following griefprevention etc. #5

Open
KasperFranz opened this issue Mar 7, 2017 · 6 comments
Open

Not following griefprevention etc. #5

KasperFranz opened this issue Mar 7, 2017 · 6 comments

Comments

@KasperFranz
Copy link

The rotate, fetch from container and equal to container works even when it shouldn't.

the dump to container did not work

This is all done on a vanilla chest using Sponge and GriefPrevention on a Beyond 1.0.1 server

@KasperFranz
Copy link
Author

The right method should ofc. be that you can't take from/deposit to a claimed chest

@McJty
Copy link
Collaborator

McJty commented Mar 7, 2017

How can I even do that? That is not up to this mod to handle as far as I know

@KasperFranz
Copy link
Author

I have no idea - I have never done anything related to forge :p

There is no events fired so no way to catch and cancel it

@KasperFranz
Copy link
Author

Do you have an update on this :)?

@McJty
Copy link
Collaborator

McJty commented Apr 6, 2017

I don't think this is something that has to be fixed on my side. It is up to the grief prevention tools to do that as far as I know. I wouldn't even know how to fix this. Maybe report to whatever project you're using for grief prevention

@gabizou
Copy link

gabizou commented Aug 3, 2017

@McJty Just throwing an idea out here, but I believe it's possible for Sponge to provide protection plugins the ability to "cancel" the interactions (in this case, extracting or opening inventories) by doing MinecraftServer.isBlockProtected(World, BlockPos, EntityPlayer). Since Sponge will throw an event for plugins to interact with the block, I do believe this is the ultimate thing we can do to achieve compatibility with the mod.

It looks like all you have to do really is just perform the check prior to sending or opening inventories for tile entities or IItemHandlers etc.

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