Skip to content

Commit fc15efc

Browse files
committed
Refactor world manager logging and metrics handling
Simplified logging for unsupported world managers and improved custom chart creation for metrics.
1 parent ceea9d0 commit fc15efc

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

src/main/java/net/thenextlvl/perworlds/PerWorldsPlugin.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -84,12 +84,12 @@ private void registerListeners() {
8484

8585
private void warnWorldManager() {
8686
if (knownWorldManagers.stream()
87+
.filter(name -> !name.equals("Worlds"))
8788
.map(getServer().getPluginManager()::getPlugin)
8889
.noneMatch(Objects::nonNull)) return;
8990
getComponentLogger().warn("It appears you are using a third party world management plugin");
90-
getComponentLogger().warn("Please consider switching to 'Worlds' for first hand support");
91+
getComponentLogger().warn("Consider switching to 'Worlds' for first hand support");
9192
getComponentLogger().warn("Download at: https://modrinth.com/project/gBIw3Gvy");
92-
getComponentLogger().warn("Since Worlds already ships with PerWorlds, you have to uninstall this plugin when switching");
9393
}
9494

9595
private void loadGroups() {
@@ -136,9 +136,10 @@ private void registerCommands() {
136136
);
137137

138138
private void addCustomCharts() {
139-
metrics.addCustomChart(new SimplePie("world_management_plugin", () -> knownWorldManagers.stream()
139+
var worldManager = knownWorldManagers.stream()
140140
.filter(name -> getServer().getPluginManager().getPlugin(name) != null)
141-
.findAny().orElse("None")));
141+
.findAny().orElse("None");
142+
metrics.addCustomChart(new SimplePie("world_management_plugin", () -> worldManager));
142143
}
143144

144145
public GroupProvider groupProvider() {

0 commit comments

Comments
 (0)