2021-06-11 14:02:28 +02:00
|
|
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
|
|
From: Aikar <aikar@aikar.co>
|
|
|
|
Date: Fri, 18 Mar 2016 13:17:38 -0400
|
|
|
|
Subject: [PATCH] Default loading permissions.yml before plugins
|
|
|
|
|
|
|
|
Under previous behavior, plugins were not able to check if a player had a permission
|
|
|
|
if it was defined in permissions.yml. there is no clean way for a plugin to fix that either.
|
|
|
|
|
|
|
|
This will change the order so that by default, permissions.yml loads BEFORE plugins instead of after.
|
|
|
|
|
|
|
|
This gives plugins expected permission checks.
|
|
|
|
|
|
|
|
It also helps improve the expected logic, as servers should set the initial defaults, and then let plugins
|
|
|
|
modify that. Under the previous logic, plugins were unable (cleanly) override permissions.yml.
|
|
|
|
|
|
|
|
A config option has been added for those who depend on the previous behavior, but I don't expect that.
|
|
|
|
|
|
|
|
diff --git a/src/main/java/org/bukkit/craftbukkit/CraftServer.java b/src/main/java/org/bukkit/craftbukkit/CraftServer.java
|
2022-07-27 20:52:03 +02:00
|
|
|
index 3dc04ad966c9f44f62af918c50ff1e8ed461fe3b..9f9d2c32d184a9ef7f2fbdef85129654b43cf3ab 100644
|
2021-06-11 14:02:28 +02:00
|
|
|
--- a/src/main/java/org/bukkit/craftbukkit/CraftServer.java
|
|
|
|
+++ b/src/main/java/org/bukkit/craftbukkit/CraftServer.java
|
2022-07-04 16:38:06 +02:00
|
|
|
@@ -460,6 +460,7 @@ public final class CraftServer implements Server {
|
2021-06-21 10:41:00 +02:00
|
|
|
if (type == PluginLoadOrder.STARTUP) {
|
|
|
|
this.helpMap.clear();
|
|
|
|
this.helpMap.initializeGeneralTopics();
|
2022-06-09 10:51:45 +02:00
|
|
|
+ if (io.papermc.paper.configuration.GlobalConfiguration.get().misc.loadPermissionsYmlBeforePlugins) loadCustomPermissions(); // Paper
|
2021-06-11 14:02:28 +02:00
|
|
|
}
|
|
|
|
|
2021-06-21 10:41:00 +02:00
|
|
|
Plugin[] plugins = this.pluginManager.getPlugins();
|
2022-07-04 16:38:06 +02:00
|
|
|
@@ -479,7 +480,7 @@ public final class CraftServer implements Server {
|
2021-06-12 02:57:04 +02:00
|
|
|
this.commandMap.registerServerAliases();
|
2021-06-11 14:02:28 +02:00
|
|
|
DefaultPermissions.registerCorePermissions();
|
|
|
|
CraftDefaultPermissions.registerCorePermissions();
|
2021-06-12 02:57:04 +02:00
|
|
|
- this.loadCustomPermissions();
|
2022-06-09 10:51:45 +02:00
|
|
|
+ if (!io.papermc.paper.configuration.GlobalConfiguration.get().misc.loadPermissionsYmlBeforePlugins) this.loadCustomPermissions(); // Paper
|
2021-06-12 02:57:04 +02:00
|
|
|
this.helpMap.initializeCommands();
|
|
|
|
this.syncCommands();
|
2021-06-11 14:02:28 +02:00
|
|
|
}
|