commit 1211dfb057
Author: Ben Woo <30431861+benwoo1110@users.noreply.github.com>
Date: Tue Aug 17 16:29:31 2021 +0800
Implement suggested improvements.
* Use singleton pattern for DefaultContentFilter with getInstance method.
* Have a default SendHandler.
* Don't need streams for small dataset.
* Private WorldListContentParser class to improve readability.
commit 562eed8255
Author: Ben Woo <30431861+benwoo1110@users.noreply.github.com>
Date: Tue Aug 17 00:26:57 2021 +0800
Revamp content display API.
* Implement new content display system.
* Add @FunctionalInterface annotation to displaytools interfaces.
* Use static factory method to create Builder with contents.
* Use T for generic type since it's static method.
* Rename Builder#display to show and require CommandSender.
* Rename package from displaytools to display.
* Move DisplayHandler impls into their own package.
* Overload ContentDisplay#forContent with defaults for list and map.
* Pass CommandSender to send command.
This system is much more versatile when a single ContentDisplay instance
can be used for multiple players.
* Rename ContentDisplay#send to #show.
* Split DisplaySettings into separate classes.
Co-authored-by: Jeremy Wood <farachan@gmail.com>
This may seem like a really negligible change but having this option default to `true` actually causes a lot of pain for support in other plugins that deal with prefixes (LuckPerms and Essentials being the main victims). Users always come in guns blazing, screaming at us because they don't know how to remove the world name from their chat. It's such a simple solution, yet no one thinks that of all plugins, it would be Multiverse that is tainting their beloved chat format.
Please consider updating this default setting. It will save a lot of us trouble in the future 🙏