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

ProtectHorses option in config no longer applies to non-horse mounts. #93

Closed
Rodzynus opened this issue Feb 23, 2017 · 1 comment
Closed
Labels
🐛Bug Issue is a validated bug report.

Comments

@Rodzynus
Copy link

1.11 version split mounts into separate entities, they used to be variations of Horse entity. This causes donkeys, mules, skeleton horses and llamas to require container trust and ignore "ProtectHorses: false" value in the config.
I use ZHorse plugin to keep horses safe and this problem makes it unable for people to mount their own horses if they happened to dismount on someone else's claim.

@RoboMWM
Copy link

RoboMWM commented Feb 23, 2017

Yea, I'm currently overhauling the plugin (and likely will have only basic, perhaps configurable protection of such entities), but if you want to PR a fix feel free to do so.

@RoboMWM RoboMWM added the 🐛Bug Issue is a validated bug report. label Feb 23, 2017
Rodzynus added a commit to Rodzynus/GriefPrevention that referenced this issue Feb 23, 2017
Rodzynus added a commit to Rodzynus/GriefPrevention that referenced this issue Feb 24, 2017
RoboMWM pushed a commit that referenced this issue Feb 24, 2017

* Fixed issue #93, this time using AbstractHorse entity.
@RoboMWM RoboMWM closed this as completed Feb 27, 2017
Minidodo1 pushed a commit to Minidodo1/GriefPrevention that referenced this issue Jun 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Issue is a validated bug report.
Projects
None yet
Development

No branches or pull requests

2 participants