search

elBukkit / PreciousStones

Self-service protection for Minecraft servers
http://dev.bukkit.org/server-mods/preciousstones/
10 stars 10 forks source link

Allowed players can delete a field #52

Closed xtomyserrax closed 3 years ago

xtomyserrax commented 3 years ago

I have a cuboid (my plot) and i add my friend to my plot (ps allow nickname) and my friend can delete my field

xtomyserrax commented 3 years ago

They can use /ps take even though it has allowed-can-break flag disabled

ikilltheundead commented 3 years ago

Possible fix: here However I am unable to test or verify that it works. Not a good fix as the owner of a field is a string, instead of a player object, though this might be done for a good reason.

xtomyserrax commented 3 years ago

The thing here is that we are not allowing a player as a guest, so the system gives co-owner permissions. I think the best alternative is to revamp the allow command by adding a new string, so you can define which type of permission group you would like to give to the new member.

Also in that command, instead of looking if player is owner, might be useful to check this flag: "allowed-can-break"

This report relates with #28