Wesley1808 / ServerCore

A mod that aims to optimize the minecraft server.
75 stars 11 forks source link

feat: add villager lobotomy whitelist tag #76

Closed Jarva closed 12 months ago

Jarva commented 12 months ago

Allows a tag based override on whether a villager should be lobotomized, beneficial for farms that require villagers ticking in 1x1 containers, like a stacking raid farm

Wesley1808 commented 12 months ago

This probably won't be too helpful for players unless you manually add the tags to their villagers as admin. There's also the alternative of building 2x1 cages using (for example) walls so the villager can't move there, which is more useful for players - but still not really a perfect solution either.

I don't see the harm in adding it regardless though, it can be useful in specific scenarios where admins can change the tags. Do you mind if I rename no_lobotomy to exclude_lobotomization to be a bit more consistent with exclude_ear (activation range exclusion tag)?

Jarva commented 12 months ago

I initially had it as a custom name, but I thought admins would want to prevent people from bypassing it, as a tag it works as an admin override, which specifically works for my use-case

The rename is fine with me, this is purely just to contribute a change I already utilize on my server