Replies: 7 comments
-
These are specifically treated as a farmable resource. Probably better being a discussion, but I'm just gonna apply the tag for now instead of converting. |
Beta Was this translation helpful? Give feedback.
-
For the farming part in #1564, I agree with you. But for actual breaking of the vines you get from planting glow berries, and placing new vines, I still think this should be protected even if you have containertrust, and still be considered a bug. Why would you be able to break this specific farmable resource with container trust, where you would not be able to with other farmable resources? |
Beta Was this translation helpful? Give feedback.
-
You can break other farmable resources. Wheat, carrots, etc. all are fully breakable because that's just how they work. GP doesn't really have a concept of "farmable but not breakable" although that might be a good enhancement. |
Beta Was this translation helpful? Give feedback.
-
Right, okay. So this new item/block just happen to have a way to be farmed without breaking it, which GriefPrevention have no way to handle(for now at least). I don't know how the code and handles in Minecraft/java works, so I have no idea if it's even possible to handle it the way I think it should be handled. |
Beta Was this translation helpful? Give feedback.
-
It is possible, it's just that GP handles this and similar existing blocks the same way, namely, if you're allowed container access you'd be able to bust other crops, so you should have consistent permission to bust these as they are also farmable crops. I do agree that it would be handy, particularly for these and sweet berries as those damage players. While I'm pretty sure grass floors in a shop are quite uncommon, not leaving players vulnerable to death by berries while afk in a communal area would be nice. |
Beta Was this translation helpful? Give feedback.
-
Thanks for the explanation. This will be more of an enhancement request then I guess. (I've seen a few shops with grass floor on our server over the years, but not too many :p ) |
Beta Was this translation helpful? Give feedback.
-
yes this should be a discussion, as I see no bug to fix here. |
Beta Was this translation helpful? Give feedback.
-
What happened:
Glow berries can be placed and broken inside a claim, with only containertrust.
What was expected:
Placing and breaking glow berries should require full trust, not only containertrust.
Steps to reproduce:
Server and GriefPrevention version:
Paste of
/version
: This server is running Paper version git-Paper-239 (MC: 1.17.1) (Implementing API version 1.17.1-R0.1-SNAPSHOT) (Git: ca708a0)Paste of
/version GriefPrevention
: GriefPrevention version 85b76ccStack trace/error or server log
I have not included server log, as I don't want to expose the entire chat log here. If it's still needed, I'll see what I can do.
GriefPrevention config.yml
Plugin list (if applicable):
AntiXRay, AutoMessage, BuycraftX, Chunky, CommandHook, EntityWatch, Essentials, EssentialsChat, EssentialsDiscord, EssentialsSpawn, F3NPerm, GriefPrevention, HolographicDisplays, InteractiveBooks, InventoryRollback, LogBlock, LuckPerms, Matrix, OpenInv, PlugMan, ProtocolLib, Reports, Vault, VoteRoulette, Votifier, WorldEdit, WorldGuard
Debugging steps attempted
Beta Was this translation helpful? Give feedback.
All reactions