mirror of
https://github.com/PlaceholderAPI/PlaceholderAPI
synced 2024-11-18 00:46:55 +01:00
Update Adventure-platform-bukkit to 4.3.2
This commit is contained in:
parent
98082398fc
commit
19fca16653
@ -110,7 +110,12 @@ public abstract class PlaceholderExpansion extends PlaceholderHook {
|
||||
* command is used
|
||||
*
|
||||
* @return if this expansion should persist through placeholder reloads
|
||||
*
|
||||
* @deprecated PlaceholderExpansions registered through their {@link #register()} and not through
|
||||
* {@link me.clip.placeholderapi.expansion.manager.LocalExpansionManager#register(Class)}
|
||||
* will be considered internal now and not be unregistered during Plugin reloads.
|
||||
*/
|
||||
@Deprecated
|
||||
public boolean persist() {
|
||||
return false;
|
||||
}
|
||||
|
@ -46,6 +46,7 @@ import me.clip.placeholderapi.expansion.Cacheable;
|
||||
import me.clip.placeholderapi.expansion.Cleanable;
|
||||
import me.clip.placeholderapi.expansion.Configurable;
|
||||
import me.clip.placeholderapi.expansion.PlaceholderExpansion;
|
||||
import me.clip.placeholderapi.expansion.PlaceholderExpansion.Type;
|
||||
import me.clip.placeholderapi.expansion.Taskable;
|
||||
import me.clip.placeholderapi.expansion.VersionSpecific;
|
||||
import me.clip.placeholderapi.expansion.cloud.CloudExpansion;
|
||||
@ -264,6 +265,9 @@ public final class LocalExpansionManager implements Listener {
|
||||
|
||||
if (expansion instanceof VersionSpecific) {
|
||||
VersionSpecific nms = (VersionSpecific) expansion;
|
||||
Msg.warn("Nag Author(s) %s of expansion %s about their usage of the deprecated "
|
||||
+ "VersionSpecific interface!", expansion.getAuthor(), expansion.getIdentifier());
|
||||
Msg.warn("They should switch to a new method of determining the Server version.");
|
||||
if (!nms.isCompatibleWith(PlaceholderAPIPlugin.getServerVersion())) {
|
||||
Msg.warn("Your server version is incompatible with expansion %s %s",
|
||||
expansion.getIdentifier(), expansion.getVersion());
|
||||
@ -324,6 +328,19 @@ public final class LocalExpansionManager implements Listener {
|
||||
return false;
|
||||
}
|
||||
|
||||
// Don't unregister expansions that are marked internal or set to be persistent.
|
||||
if (expansion.getExpansionType() == Type.INTERNAL || expansion.persist()) {
|
||||
// Print warning if an external expansion is set to be persistent.
|
||||
if (expansion.getExpansionType() == Type.EXTERNAL && expansion.persist()) {
|
||||
Msg.warn("Nag author(s) %s about their expansion %s being marked as \"external\" "
|
||||
+ "but having persist() set to true!", expansion.getAuthor(), expansion.getIdentifier());
|
||||
Msg.warn("External Expansions should not be set to be persistent! PlaceholderAPI "
|
||||
+ "will respect this setting and skip the unregister of this Expansion...");
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
Bukkit.getPluginManager().callEvent(new ExpansionUnregisterEvent(expansion));
|
||||
|
||||
if (expansion instanceof Listener) {
|
||||
@ -393,7 +410,7 @@ public final class LocalExpansionManager implements Listener {
|
||||
|
||||
private void unregisterAll() {
|
||||
for (final PlaceholderExpansion expansion : Sets.newHashSet(expansions.values())) {
|
||||
if (expansion.persist()) {
|
||||
if (expansion.persist() || expansion.getExpansionType() == Type.INTERNAL) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user