From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 From: Riley Park Date: Fri, 29 Jan 2021 17:21:55 +0100 Subject: [PATCH] Adventure Co-authored-by: zml Co-authored-by: Jake Potrebic diff --git a/pom.xml b/pom.xml index 9aafdcc49ed83b403abb96891008103e6d6a69fa..1ced7a212684cee8faf07fa9a083adcd47be7fcb 100644 --- a/pom.xml +++ b/pom.xml @@ -22,6 +22,7 @@ 1.8 1.8 UTF-8 + 4.7.0 @@ -31,7 +32,39 @@ + + + + + net.kyori + adventure-bom + ${adventure.version} + pom + import + + + + + + + + net.kyori + adventure-api + + + net.kyori + adventure-text-serializer-gson + + + net.kyori + adventure-text-serializer-legacy + + + net.kyori + adventure-text-serializer-plain + + it.unimi.dsi fastutil @@ -216,6 +249,12 @@ https://javadoc.io/doc/org.yaml/snakeyaml/1.27/ https://javadoc.io/doc/org.jetbrains/annotations-java5/20.1.0/ https://javadoc.io/doc/net.md-5/bungeecord-chat/1.16-R0.4/ + + https://jd.adventure.kyori.net/api/${adventure.version}/ + https://jd.adventure.kyori.net/text-serializer-gson/${adventure.version}/ + https://jd.adventure.kyori.net/text-serializer-legacy/${adventure.version}/ + https://jd.adventure.kyori.net/text-serializer-plain/${adventure.version}/ + diff --git a/src/main/java/co/aikar/timings/TimingsReportListener.java b/src/main/java/co/aikar/timings/TimingsReportListener.java index ef58a6c00f444bd498a2d8fc4e457236f393954f..ecd149157d4fb80444f34bf5633d74bcdb63dec5 100644 --- a/src/main/java/co/aikar/timings/TimingsReportListener.java +++ b/src/main/java/co/aikar/timings/TimingsReportListener.java @@ -15,7 +15,7 @@ import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; @SuppressWarnings("WeakerAccess") -public class TimingsReportListener implements MessageCommandSender { +public class TimingsReportListener implements net.kyori.adventure.audience.ForwardingAudience, MessageCommandSender { // Paper private final List senders; private final Runnable onDone; private String timingsURL; @@ -74,4 +74,17 @@ public class TimingsReportListener implements MessageCommandSender { this.senders.add(Bukkit.getConsoleSender()); } } + + // Paper start + @Override + public void sendMessage(final @NotNull net.kyori.adventure.identity.Identity source, final @NotNull net.kyori.adventure.text.Component message, final @NotNull net.kyori.adventure.audience.MessageType type) { + net.kyori.adventure.audience.ForwardingAudience.super.sendMessage(source, message, type); + } + + @NotNull + @Override + public Iterable audiences() { + return this.senders; + } + // Paper end } diff --git a/src/main/java/io/papermc/paper/chat/ChatComposer.java b/src/main/java/io/papermc/paper/chat/ChatComposer.java new file mode 100644 index 0000000000000000000000000000000000000000..1f03ce9ff40ed12a1825c8e24dabddbbef44d6af --- /dev/null +++ b/src/main/java/io/papermc/paper/chat/ChatComposer.java @@ -0,0 +1,24 @@ +package io.papermc.paper.chat; + +import net.kyori.adventure.text.Component; +import org.bukkit.entity.Player; +import org.jetbrains.annotations.NotNull; + +/** + * A chat composer is responsible for composing chat messages sent by {@link Player}s to the server. + */ +@FunctionalInterface +public interface ChatComposer { + ChatComposer DEFAULT = (player, displayName, message) -> Component.translatable("chat.type.text", displayName, message); + + /** + * Composes a chat message. + * + * @param source the message source + * @param displayName the display name of the {@link Player} sending the message + * @param message the chat message + * @return a composed chat message + */ + @NotNull + Component composeChat(final @NotNull Player source, final @NotNull Component displayName, final @NotNull Component message); +} diff --git a/src/main/java/io/papermc/paper/chat/ChatFormatter.java b/src/main/java/io/papermc/paper/chat/ChatFormatter.java new file mode 100644 index 0000000000000000000000000000000000000000..ae811175089009be6b1db6941e9c5a24b2b1f027 --- /dev/null +++ b/src/main/java/io/papermc/paper/chat/ChatFormatter.java @@ -0,0 +1,28 @@ +package io.papermc.paper.chat; + +import net.kyori.adventure.text.Component; +import org.bukkit.entity.Player; +import org.jetbrains.annotations.NotNull; + +/** + * A chat formatter is responsible for the formatting of chat messages sent by {@link Player}s to the server. + * + * @deprecated in favour of {@link ChatComposer} + */ +@Deprecated +@FunctionalInterface +public interface ChatFormatter { + @Deprecated + ChatFormatter DEFAULT = (displayName, message) -> Component.translatable("chat.type.text", displayName, message); + + /** + * Formats a chat message. + * + * @param displayName the display name of the {@link Player} sending the message + * @param message the chat message + * @return a formatted chat message + */ + @Deprecated + @NotNull + Component chat(final @NotNull Component displayName, final @NotNull Component message); +} diff --git a/src/main/java/io/papermc/paper/event/player/AbstractChatEvent.java b/src/main/java/io/papermc/paper/event/player/AbstractChatEvent.java new file mode 100644 index 0000000000000000000000000000000000000000..f77597786f201b57ac18e14099f7b84f1e4e4cf3 --- /dev/null +++ b/src/main/java/io/papermc/paper/event/player/AbstractChatEvent.java @@ -0,0 +1,140 @@ +package io.papermc.paper.event.player; + +import io.papermc.paper.chat.ChatComposer; +import io.papermc.paper.chat.ChatFormatter; +import java.util.Set; +import net.kyori.adventure.text.Component; +import org.bukkit.entity.Player; +import org.bukkit.event.Cancellable; +import org.bukkit.event.player.PlayerEvent; +import org.checkerframework.checker.nullness.qual.Nullable; +import org.jetbrains.annotations.NotNull; + +import static java.util.Objects.requireNonNull; + +/** + * An abstract implementation of a chat event, handling shared logic. + */ +public abstract class AbstractChatEvent extends PlayerEvent implements Cancellable { + private final Set recipients; + private boolean cancelled = false; + private ChatComposer composer; + @Deprecated private @Nullable ChatFormatter formatter; + private Component message; + + AbstractChatEvent(final boolean async, final @NotNull Player player, final @NotNull Set recipients, final @NotNull ChatComposer composer, final @NotNull Component message) { + super(player, async); + this.recipients = recipients; + this.composer = composer; + this.message = message; + } + + @Deprecated + AbstractChatEvent(final boolean async, final @NotNull Player player, final @NotNull Set recipients, final @NotNull ChatFormatter formatter, final @NotNull Component message) { + super(player, async); + this.recipients = recipients; + this.formatter = formatter; + this.message = message; + } + + /** + * Gets a set of recipients that this chat message will be displayed to. + * + *

The set returned is not guaranteed to be mutable and may auto-populate + * on access. Any listener accessing the returned set should be aware that + * it may reduce performance for a lazy set implementation.

+ * + *

Listeners should be aware that modifying the list may throw {@link + * UnsupportedOperationException} if the event caller provides an + * unmodifiable set.

+ * + * @return a set of players who will receive the chat message + */ + @NotNull + public final Set recipients() { + return this.recipients; + } + + /** + * Gets the chat composer. + * + * @return the chat composer + */ + @NotNull + public final ChatComposer composer() { + if(this.composer == null) { + requireNonNull(this.formatter, "composer and formatter"); + this.composer = (source, displayName, message) -> this.formatter.chat(displayName, message); + } + return this.composer; + } + + /** + * Sets the chat composer. + * + * @param composer the chat composer + * @throws NullPointerException if {@code composer} is {@code null} + */ + public final void composer(final @NotNull ChatComposer composer) { + this.composer = requireNonNull(composer, "composer"); + this.formatter = null; + } + + /** + * Gets the chat formatter. + * + * @return the chat formatter + * @deprecated in favour of {@link #composer()} + */ + @Deprecated + @NotNull + public final ChatFormatter formatter() { + if(this.formatter == null) { + this.formatter = (displayName, message) -> this.composer.composeChat(this.player, displayName, message); + } + return this.formatter; + } + + /** + * Sets the chat formatter. + * + * @param formatter the chat formatter + * @throws NullPointerException if {@code formatter} is {@code null} + * @deprecated in favour of {@link #composer(ChatComposer)} + */ + @Deprecated + public final void formatter(final @NotNull ChatFormatter formatter) { + this.formatter = requireNonNull(formatter, "formatter"); + this.composer = (source, displayName, message) -> formatter.chat(displayName, message); + } + + /** + * Gets the user-supplied message. + * + * @return the user-supplied message + */ + @NotNull + public final Component message() { + return this.message; + } + + /** + * Sets the user-supplied message. + * + * @param message the user-supplied message + * @throws NullPointerException if {@code message} is {@code null} + */ + public final void message(final @NotNull Component message) { + this.message = requireNonNull(message, "message"); + } + + @Override + public final boolean isCancelled() { + return this.cancelled; + } + + @Override + public final void setCancelled(final boolean cancelled) { + this.cancelled = cancelled; + } +} diff --git a/src/main/java/io/papermc/paper/event/player/AsyncChatEvent.java b/src/main/java/io/papermc/paper/event/player/AsyncChatEvent.java new file mode 100644 index 0000000000000000000000000000000000000000..a0f748957f4472103dd27fc95a711a42de7fae89 --- /dev/null +++ b/src/main/java/io/papermc/paper/event/player/AsyncChatEvent.java @@ -0,0 +1,39 @@ +package io.papermc.paper.event.player; + +import io.papermc.paper.chat.ChatComposer; +import io.papermc.paper.chat.ChatFormatter; +import java.util.Set; +import net.kyori.adventure.text.Component; +import org.bukkit.entity.Player; +import org.bukkit.event.HandlerList; +import org.jetbrains.annotations.NotNull; + +/** + * An event fired when a {@link Player} sends a chat message to the server. + */ +public final class AsyncChatEvent extends AbstractChatEvent { + private static final HandlerList HANDLERS = new HandlerList(); + + public AsyncChatEvent(final boolean async, final @NotNull Player player, final @NotNull Set recipients, final @NotNull ChatComposer composer, final @NotNull Component message) { + super(async, player, recipients, composer, message); + } + + /** + * @deprecated use {@link #AsyncChatEvent(boolean, Player, Set, ChatComposer, Component)} + */ + @Deprecated + public AsyncChatEvent(final boolean async, final @NotNull Player player, final @NotNull Set recipients, final @NotNull ChatFormatter formatter, final @NotNull Component message) { + super(async, player, recipients, formatter, message); + } + + @NotNull + @Override + public HandlerList getHandlers() { + return HANDLERS; + } + + @NotNull + public static HandlerList getHandlerList() { + return HANDLERS; + } +} diff --git a/src/main/java/io/papermc/paper/event/player/ChatEvent.java b/src/main/java/io/papermc/paper/event/player/ChatEvent.java new file mode 100644 index 0000000000000000000000000000000000000000..13c5df5fb8ce1d0203d99e88dd691019146a8f52 --- /dev/null +++ b/src/main/java/io/papermc/paper/event/player/ChatEvent.java @@ -0,0 +1,44 @@ +package io.papermc.paper.event.player; + +import io.papermc.paper.chat.ChatComposer; +import io.papermc.paper.chat.ChatFormatter; +import java.util.Set; +import net.kyori.adventure.text.Component; +import org.bukkit.Warning; +import org.bukkit.entity.Player; +import org.bukkit.event.HandlerList; +import org.jetbrains.annotations.NotNull; + +/** + * An event fired when a {@link Player} sends a chat message to the server. + * + * @deprecated Listening to this event forces chat to wait for the main thread, delaying chat messages. It is recommended to use {@link AsyncChatEvent} instead, wherever possible. + */ +@Deprecated +@Warning(reason = "Listening to this event forces chat to wait for the main thread, delaying chat messages.") +public final class ChatEvent extends AbstractChatEvent { + private static final HandlerList HANDLERS = new HandlerList(); + + public ChatEvent(final @NotNull Player player, final @NotNull Set recipients, final @NotNull ChatComposer composer, final @NotNull Component message) { + super(false, player, recipients, composer, message); + } + + /** + * @deprecated use {@link #ChatEvent(Player, Set, ChatComposer, Component)} + */ + @Deprecated + public ChatEvent(final @NotNull Player player, final @NotNull Set recipients, final @NotNull ChatFormatter formatter, final @NotNull Component message) { + super(false, player, recipients, formatter, message); + } + + @NotNull + @Override + public HandlerList getHandlers() { + return HANDLERS; + } + + @NotNull + public static HandlerList getHandlerList() { + return HANDLERS; + } +} diff --git a/src/main/java/io/papermc/paper/text/PaperComponents.java b/src/main/java/io/papermc/paper/text/PaperComponents.java new file mode 100644 index 0000000000000000000000000000000000000000..77db592d05b754f879f8d1790642e9d9bbd30a4e --- /dev/null +++ b/src/main/java/io/papermc/paper/text/PaperComponents.java @@ -0,0 +1,88 @@ +package io.papermc.paper.text; + +import net.kyori.adventure.text.Component; +import net.kyori.adventure.text.flattener.ComponentFlattener; +import net.kyori.adventure.text.format.NamedTextColor; +import net.kyori.adventure.text.serializer.gson.GsonComponentSerializer; +import net.kyori.adventure.text.serializer.legacy.LegacyComponentSerializer; +import net.kyori.adventure.text.serializer.plain.PlainComponentSerializer; +import org.bukkit.Bukkit; +import org.checkerframework.checker.nullness.qual.NonNull; + +/** + * Paper API-specific methods for working with {@link Component}s and related. + */ +public final class PaperComponents { + private PaperComponents() { + throw new RuntimeException("PaperComponents is not to be instantiated!"); + } + + /** + * Return a component flattener that can use game data to resolve extra information about components. + * + * @return a component flattener + */ + public static @NonNull ComponentFlattener flattener() { + return Bukkit.getUnsafe().componentFlattener(); + } + + /** + * Get a serializer for {@link Component}s that will convert components to + * a plain-text string. + * + *

Implementations may provide a serializer capable of processing any + * information that requires access to implementation details.

+ * + * @return a serializer to plain text + */ + public static @NonNull PlainComponentSerializer plainSerializer() { + return Bukkit.getUnsafe().plainComponentSerializer(); + } + + /** + * Get a serializer for {@link Component}s that will convert to and from the + * standard JSON serialization format using Gson. + * + *

Implementations may provide a serializer capable of processing any + * information that requires implementation details, such as legacy + * (pre-1.16) hover events.

+ * + * @return a json component serializer + */ + public static @NonNull GsonComponentSerializer gsonSerializer() { + return Bukkit.getUnsafe().gsonComponentSerializer(); + } + + /** + * Get a serializer for {@link Component}s that will convert to and from the + * standard JSON serialization format using Gson, downsampling any RGB colors + * to their nearest {@link NamedTextColor} counterpart. + * + *

Implementations may provide a serializer capable of processing any + * information that requires implementation details, such as legacy + * (pre-1.16) hover events.

+ * + * @return a json component serializer + */ + public static @NonNull GsonComponentSerializer colorDownsamplingGsonSerializer() { + return Bukkit.getUnsafe().colorDownsamplingGsonComponentSerializer(); + } + + /** + * Get a serializer for {@link Component}s that will convert to and from the + * legacy component format used by Bukkit. This serializer uses the + * {@link LegacyComponentSerializer.Builder#useUnusualXRepeatedCharacterHexFormat()} + * option to match upstream behavior. + * + *

This legacy serializer uses the standard section symbol to mark + * formatting characters.

+ * + *

Implementations may provide a serializer capable of processing any + * information that requires access to implementation details.

+ * + * @return a section serializer + */ + public static @NonNull LegacyComponentSerializer legacySectionSerializer() { + return Bukkit.getUnsafe().legacyComponentSerializer(); + } +} diff --git a/src/main/java/org/bukkit/Bukkit.java b/src/main/java/org/bukkit/Bukkit.java index 7c715fdc11ab7837552b1fe3ffd08b31cec0a63b..426b1e83226e674ee4bf3ec05ddcd3ac4376b06d 100644 --- a/src/main/java/org/bukkit/Bukkit.java +++ b/src/main/java/org/bukkit/Bukkit.java @@ -278,7 +278,9 @@ public final class Bukkit { * * @param message the message * @return the number of players + * @deprecated in favour of {@link Server#sendMessage(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public static int broadcastMessage(@NotNull String message) { return server.broadcastMessage(message); } @@ -836,6 +838,7 @@ public final class Bukkit { server.shutdown(); } + // Paper start /** * Broadcasts the specified message to every user with the given * permission name. @@ -845,6 +848,21 @@ public final class Bukkit { * permissibles} must have to receive the broadcast * @return number of message recipients */ + public static int broadcast(@NotNull net.kyori.adventure.text.Component message, @NotNull String permission) { + return server.broadcast(message, permission); + } + // Paper end + /** + * Broadcasts the specified message to every user with the given + * permission name. + * + * @param message message to broadcast + * @param permission the required permission {@link Permissible + * permissibles} must have to receive the broadcast + * @return number of message recipients + * @deprecated in favour of {@link #broadcast(net.kyori.adventure.text.Component, String)} + */ + @Deprecated // Paper public static int broadcast(@NotNull String message, @NotNull String permission) { return server.broadcast(message, permission); } @@ -1044,6 +1062,7 @@ public final class Bukkit { return server.createInventory(owner, type); } + // Paper start /** * Creates an empty inventory with the specified type and title. If the type * is {@link InventoryType#CHEST}, the new inventory has a size of 27; @@ -1069,6 +1088,38 @@ public final class Bukkit { * @see InventoryType#isCreatable() */ @NotNull + public static Inventory createInventory(@Nullable InventoryHolder owner, @NotNull InventoryType type, @NotNull net.kyori.adventure.text.Component title) { + return server.createInventory(owner, type, title); + } + // Paper end + + /** + * Creates an empty inventory with the specified type and title. If the type + * is {@link InventoryType#CHEST}, the new inventory has a size of 27; + * otherwise the new inventory has the normal size for its type.
+ * It should be noted that some inventory types do not support titles and + * may not render with said titles on the Minecraft client. + *
+ * {@link InventoryType#WORKBENCH} will not process crafting recipes if + * created with this method. Use + * {@link Player#openWorkbench(Location, boolean)} instead. + *
+ * {@link InventoryType#ENCHANTING} will not process {@link ItemStack}s + * for possible enchanting results. Use + * {@link Player#openEnchanting(Location, boolean)} instead. + * + * @param owner The holder of the inventory; can be null if there's no holder. + * @param type The type of inventory to create. + * @param title The title of the inventory, to be displayed when it is viewed. + * @return The new inventory. + * @throws IllegalArgumentException if the {@link InventoryType} cannot be + * viewed. + * @deprecated in favour of {@link #createInventory(InventoryHolder, InventoryType, net.kyori.adventure.text.Component)} + * + * @see InventoryType#isCreatable() + */ + @Deprecated // Paper + @NotNull public static Inventory createInventory(@Nullable InventoryHolder owner, @NotNull InventoryType type, @NotNull String title) { return server.createInventory(owner, type, title); } @@ -1087,6 +1138,7 @@ public final class Bukkit { return server.createInventory(owner, size); } + // Paper start /** * Creates an empty inventory of type {@link InventoryType#CHEST} with the * specified size and title. @@ -1099,10 +1151,30 @@ public final class Bukkit { * @throws IllegalArgumentException if the size is not a multiple of 9 */ @NotNull + public static Inventory createInventory(@Nullable InventoryHolder owner, int size, @NotNull net.kyori.adventure.text.Component title) throws IllegalArgumentException { + return server.createInventory(owner, size, title); + } + // Paper end + + /** + * Creates an empty inventory of type {@link InventoryType#CHEST} with the + * specified size and title. + * + * @param owner the holder of the inventory, or null to indicate no holder + * @param size a multiple of 9 as the size of inventory to create + * @param title the title of the inventory, displayed when inventory is + * viewed + * @return a new inventory + * @throws IllegalArgumentException if the size is not a multiple of 9 + * @deprecated in favour of {@link #createInventory(InventoryHolder, InventoryType, net.kyori.adventure.text.Component)} + */ + @Deprecated // Paper + @NotNull public static Inventory createInventory(@Nullable InventoryHolder owner, int size, @NotNull String title) throws IllegalArgumentException { return server.createInventory(owner, size, title); } + // Paper start /** * Creates an empty merchant. * @@ -1110,7 +1182,20 @@ public final class Bukkit { * when the merchant inventory is viewed * @return a new merchant */ + public static @NotNull Merchant createMerchant(@Nullable net.kyori.adventure.text.Component title) { + return server.createMerchant(title); + } + // Paper start + /** + * Creates an empty merchant. + * + * @param title the title of the corresponding merchant inventory, displayed + * when the merchant inventory is viewed + * @return a new merchant + * @deprecated in favour of {@link #createMerchant(net.kyori.adventure.text.Component)} + */ @NotNull + @Deprecated // Paper public static Merchant createMerchant(@Nullable String title) { return server.createMerchant(title); } @@ -1181,22 +1266,47 @@ public final class Bukkit { return server.isPrimaryThread(); } + // Paper start + /** + * Gets the message that is displayed on the server list. + * + * @return the server's MOTD + */ + @NotNull public static net.kyori.adventure.text.Component motd() { + return server.motd(); + } + // Paper end + /** * Gets the message that is displayed on the server list. * * @return the servers MOTD + * @deprecated in favour of {@link #motd()} */ @NotNull + @Deprecated // Paper public static String getMotd() { return server.getMotd(); } + // Paper start + /** + * Gets the default message that is displayed when the server is stopped. + * + * @return the shutdown message + */ + public static @Nullable net.kyori.adventure.text.Component shutdownMessage() { + return server.shutdownMessage(); + } + // Paper end /** * Gets the default message that is displayed when the server is stopped. * * @return the shutdown message + * @deprecated in favour of {@link #shutdownMessage()} */ @Nullable + @Deprecated // Paper public static String getShutdownMessage() { return server.getShutdownMessage(); } diff --git a/src/main/java/org/bukkit/Nameable.java b/src/main/java/org/bukkit/Nameable.java index fee814e01a653d2b53c56e8b566383ca44aa5346..2acdf2a6d3955923c721222b9da784f3278f6418 100644 --- a/src/main/java/org/bukkit/Nameable.java +++ b/src/main/java/org/bukkit/Nameable.java @@ -4,6 +4,30 @@ import org.jetbrains.annotations.Nullable; public interface Nameable { + // Paper start + /** + * Gets the custom name. + * + *

This value has no effect on players, they will always use their real name.

+ * + * @return the custom name + */ + @Nullable net.kyori.adventure.text.Component customName(); + + /** + * Sets the custom name. + * + *

This name will be used in death messages and can be sent to the client as a nameplate over the mob.

+ * + *

Setting the name to {@code null} will clear it.

+ * + *

This value has no effect on players, they will always use their real name.

+ * + * @param customName the custom name to set + */ + void customName(final @Nullable net.kyori.adventure.text.Component customName); + // Paper end + /** * Gets the custom name on a mob or block. If there is no name this method * will return null. diff --git a/src/main/java/org/bukkit/NamespacedKey.java b/src/main/java/org/bukkit/NamespacedKey.java index 803fa0019869127ee8c7e4fb1777a59c43e66f8a..c65f0d6569c130b4920a9e71ad24af6427f1f030 100644 --- a/src/main/java/org/bukkit/NamespacedKey.java +++ b/src/main/java/org/bukkit/NamespacedKey.java @@ -19,7 +19,7 @@ import org.jetbrains.annotations.Nullable; * underscores, hyphens, and forward slashes. * */ -public final class NamespacedKey { +public final class NamespacedKey implements net.kyori.adventure.key.Key { // Paper - implement Key /** * The namespace representing all inbuilt keys. @@ -212,4 +212,24 @@ public final class NamespacedKey { public static NamespacedKey fromString(@NotNull String key) { return fromString(key, null); } + + // Paper start + @NotNull + @Override + public String namespace() { + return this.getNamespace(); + } + + @NotNull + @Override + public String value() { + return this.getKey(); + } + + @NotNull + @Override + public String asString() { + return this.namespace + ':' + this.key; + } + // Paper end } diff --git a/src/main/java/org/bukkit/Server.java b/src/main/java/org/bukkit/Server.java index a6b9e4f158583e5932bf8ca210d531857e9f5360..d9515a79dc7ed60c66960cd6c6bb4c108f206f3c 100644 --- a/src/main/java/org/bukkit/Server.java +++ b/src/main/java/org/bukkit/Server.java @@ -55,7 +55,7 @@ import org.jetbrains.annotations.Nullable; /** * Represents a server implementation. */ -public interface Server extends PluginMessageRecipient { +public interface Server extends PluginMessageRecipient, net.kyori.adventure.audience.ForwardingAudience { // Paper /** * Used for all administrative messages, such as an operator using a @@ -229,7 +229,9 @@ public interface Server extends PluginMessageRecipient { * * @param message the message * @return the number of players + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public int broadcastMessage(@NotNull String message); /** @@ -703,8 +705,22 @@ public interface Server extends PluginMessageRecipient { * @param permission the required permission {@link Permissible * permissibles} must have to receive the broadcast * @return number of message recipients + * @deprecated in favour of {@link #broadcast(net.kyori.adventure.text.Component, String)} */ + @Deprecated // Paper public int broadcast(@NotNull String message, @NotNull String permission); + // Paper start + /** + * Broadcasts the specified message to every user with the given + * permission name. + * + * @param message message to broadcast + * @param permission the required permission {@link Permissible + * permissibles} must have to receive the broadcast + * @return number of message recipients + */ + int broadcast(@NotNull net.kyori.adventure.text.Component message, @NotNull String permission); + // Paper end /** * Gets the player by the given name, regardless if they are offline or @@ -869,6 +885,7 @@ public interface Server extends PluginMessageRecipient { @NotNull Inventory createInventory(@Nullable InventoryHolder owner, @NotNull InventoryType type); + // Paper start /** * Creates an empty inventory with the specified type and title. If the type * is {@link InventoryType#CHEST}, the new inventory has a size of 27; @@ -894,6 +911,36 @@ public interface Server extends PluginMessageRecipient { * @see InventoryType#isCreatable() */ @NotNull + Inventory createInventory(@Nullable InventoryHolder owner, @NotNull InventoryType type, @NotNull net.kyori.adventure.text.Component title); + // Paper end + + /** + * Creates an empty inventory with the specified type and title. If the type + * is {@link InventoryType#CHEST}, the new inventory has a size of 27; + * otherwise the new inventory has the normal size for its type.
+ * It should be noted that some inventory types do not support titles and + * may not render with said titles on the Minecraft client. + *
+ * {@link InventoryType#WORKBENCH} will not process crafting recipes if + * created with this method. Use + * {@link Player#openWorkbench(Location, boolean)} instead. + *
+ * {@link InventoryType#ENCHANTING} will not process {@link ItemStack}s + * for possible enchanting results. Use + * {@link Player#openEnchanting(Location, boolean)} instead. + * + * @param owner The holder of the inventory; can be null if there's no holder. + * @param type The type of inventory to create. + * @param title The title of the inventory, to be displayed when it is viewed. + * @return The new inventory. + * @throws IllegalArgumentException if the {@link InventoryType} cannot be + * viewed. + * @deprecated in favour of {@link #createInventory(InventoryHolder, InventoryType, net.kyori.adventure.text.Component)} + * + * @see InventoryType#isCreatable() + */ + @Deprecated // Paper + @NotNull Inventory createInventory(@Nullable InventoryHolder owner, @NotNull InventoryType type, @NotNull String title); /** @@ -908,6 +955,22 @@ public interface Server extends PluginMessageRecipient { @NotNull Inventory createInventory(@Nullable InventoryHolder owner, int size) throws IllegalArgumentException; + // Paper start + /** + * Creates an empty inventory of type {@link InventoryType#CHEST} with the + * specified size and title. + * + * @param owner the holder of the inventory, or null to indicate no holder + * @param size a multiple of 9 as the size of inventory to create + * @param title the title of the inventory, displayed when inventory is + * viewed + * @return a new inventory + * @throws IllegalArgumentException if the size is not a multiple of 9 + */ + @NotNull + Inventory createInventory(@Nullable InventoryHolder owner, int size, @NotNull net.kyori.adventure.text.Component title) throws IllegalArgumentException; + // Paper end + /** * Creates an empty inventory of type {@link InventoryType#CHEST} with the * specified size and title. @@ -918,18 +981,32 @@ public interface Server extends PluginMessageRecipient { * viewed * @return a new inventory * @throws IllegalArgumentException if the size is not a multiple of 9 + * @deprecated in favour of {@link #createInventory(InventoryHolder, int, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper @NotNull Inventory createInventory(@Nullable InventoryHolder owner, int size, @NotNull String title) throws IllegalArgumentException; + // Paper start + /** + * Creates an empty merchant. + * + * @param title the title of the corresponding merchant inventory, displayed + * when the merchant inventory is viewed + * @return a new merchant + */ + @NotNull Merchant createMerchant(@Nullable net.kyori.adventure.text.Component title); + // Paper start /** * Creates an empty merchant. * * @param title the title of the corresponding merchant inventory, displayed * when the merchant inventory is viewed * @return a new merchant + * @deprecated in favour of {@link #createMerchant(net.kyori.adventure.text.Component)} */ @NotNull + @Deprecated // Paper Merchant createMerchant(@Nullable String title); /** @@ -986,20 +1063,41 @@ public interface Server extends PluginMessageRecipient { */ boolean isPrimaryThread(); + // Paper start + /** + * Gets the message that is displayed on the server list. + * + * @return the server's MOTD + */ + @NotNull net.kyori.adventure.text.Component motd(); + // Paper end + /** * Gets the message that is displayed on the server list. * * @return the servers MOTD + * @deprecated in favour of {@link #motd()} */ @NotNull + @Deprecated // Paper String getMotd(); + // Paper start + /** + * Gets the default message that is displayed when the server is stopped. + * + * @return the shutdown message + */ + @Nullable net.kyori.adventure.text.Component shutdownMessage(); + // Paper end /** * Gets the default message that is displayed when the server is stopped. * * @return the shutdown message + * @deprecated in favour of {@link #shutdownMessage()} */ @Nullable + @Deprecated // Paper String getShutdownMessage(); /** @@ -1368,7 +1466,9 @@ public interface Server extends PluginMessageRecipient { * Sends the component to the player * * @param component the components to send + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void broadcast(@NotNull net.md_5.bungee.api.chat.BaseComponent component) { throw new UnsupportedOperationException("Not supported yet."); } @@ -1377,7 +1477,9 @@ public interface Server extends PluginMessageRecipient { * Sends an array of components as a single message to the player * * @param components the components to send + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void broadcast(@NotNull net.md_5.bungee.api.chat.BaseComponent... components) { throw new UnsupportedOperationException("Not supported yet."); } diff --git a/src/main/java/org/bukkit/Sound.java b/src/main/java/org/bukkit/Sound.java index 768f35c19c4557236bded5f4a85f48a2b2b2a9e6..d0ce64412276512cde133937a85a3340a70eea6d 100644 --- a/src/main/java/org/bukkit/Sound.java +++ b/src/main/java/org/bukkit/Sound.java @@ -10,7 +10,7 @@ import org.jetbrains.annotations.NotNull; * guarantee values will not be removed from this Enum. As such, you should not * depend on the ordinal values of this class. */ -public enum Sound implements Keyed { +public enum Sound implements Keyed, net.kyori.adventure.sound.Sound.Type { // Paper - implement Sound.Type AMBIENT_BASALT_DELTAS_ADDITIONS("ambient.basalt_deltas.additions"), AMBIENT_BASALT_DELTAS_LOOP("ambient.basalt_deltas.loop"), @@ -1016,4 +1016,12 @@ public enum Sound implements Keyed { public NamespacedKey getKey() { return key; } + + // Paper start + @NotNull + @Override + public net.kyori.adventure.key.@org.checkerframework.checker.nullness.qual.NonNull Key key() { + return this.key; + } + // Paper end } diff --git a/src/main/java/org/bukkit/UnsafeValues.java b/src/main/java/org/bukkit/UnsafeValues.java index 945b8b030d1b2a13afc0c4efad76997eb7bf00ba..207c656c0a11a3a630bc70491efcf433b2681e18 100644 --- a/src/main/java/org/bukkit/UnsafeValues.java +++ b/src/main/java/org/bukkit/UnsafeValues.java @@ -17,6 +17,13 @@ import org.bukkit.plugin.PluginDescriptionFile; */ @Deprecated public interface UnsafeValues { + // Paper start + net.kyori.adventure.text.flattener.ComponentFlattener componentFlattener(); + net.kyori.adventure.text.serializer.plain.PlainComponentSerializer plainComponentSerializer(); + net.kyori.adventure.text.serializer.gson.GsonComponentSerializer gsonComponentSerializer(); + net.kyori.adventure.text.serializer.gson.GsonComponentSerializer colorDownsamplingGsonComponentSerializer(); + net.kyori.adventure.text.serializer.legacy.LegacyComponentSerializer legacyComponentSerializer(); + // Paper end void reportTimings(); // Paper Material toLegacy(Material material); diff --git a/src/main/java/org/bukkit/Warning.java b/src/main/java/org/bukkit/Warning.java index efb97712cc9dc7c1e12a59f5b94e4f2ad7c6b7d8..3024468af4c073324e536c1cb26beffb1e09f3f4 100644 --- a/src/main/java/org/bukkit/Warning.java +++ b/src/main/java/org/bukkit/Warning.java @@ -67,6 +67,7 @@ public @interface Warning { * */ public boolean printFor(@Nullable Warning warning) { + if (Boolean.getBoolean("paper.alwaysPrintWarningState")) return true; // Paper if (this == DEFAULT) { return warning == null || warning.value(); } diff --git a/src/main/java/org/bukkit/World.java b/src/main/java/org/bukkit/World.java index bc4417d8ffa92a78f690bfa5705d3e42cdc11fd2..d3519fa5b99e2888a194c6382415537785fbeef0 100644 --- a/src/main/java/org/bukkit/World.java +++ b/src/main/java/org/bukkit/World.java @@ -38,7 +38,7 @@ import org.jetbrains.annotations.Nullable; /** * Represents a world, which may contain entities, chunks and blocks */ -public interface World extends PluginMessageRecipient, Metadatable { +public interface World extends PluginMessageRecipient, Metadatable, net.kyori.adventure.audience.ForwardingAudience { // Paper /** * Gets the {@link Block} at the given coordinates @@ -640,6 +640,14 @@ public interface World extends PluginMessageRecipient, Metadatable { @NotNull public List getPlayers(); + // Paper start + @NotNull + @Override + default Iterable audiences() { + return this.getPlayers(); + } + // Paper end + /** * Returns a list of entities within a bounding box centered around a * Location. diff --git a/src/main/java/org/bukkit/block/Sign.java b/src/main/java/org/bukkit/block/Sign.java index 7e3cf00e49c66023bf46c298ef46c00e8c3c2caf..6ea9b54d95d80070c01a612c0ce2ab37f0b4ad41 100644 --- a/src/main/java/org/bukkit/block/Sign.java +++ b/src/main/java/org/bukkit/block/Sign.java @@ -7,13 +7,48 @@ import org.jetbrains.annotations.NotNull; * Represents a captured state of either a SignPost or a WallSign. */ public interface Sign extends TileState, Colorable { + // Paper start + /** + * Gets all the lines of text currently on this sign. + * + * @return Array of Strings containing each line of text + */ + @NotNull + public java.util.List lines(); + + /** + * Gets the line of text at the specified index. + *

+ * For example, getLine(0) will return the first line of text. + * + * @param index Line number to get the text from, starting at 0 + * @throws IndexOutOfBoundsException Thrown when the line does not exist + * @return Text on the given line + */ + @NotNull + public net.kyori.adventure.text.Component line(int index) throws IndexOutOfBoundsException; + + /** + * Sets the line of text at the specified index. + *

+ * For example, setLine(0, "Line One") will set the first line of text to + * "Line One". + * + * @param index Line number to set the text at, starting from 0 + * @param line New text to set at the specified index + * @throws IndexOutOfBoundsException If the index is out of the range 0..3 + */ + public void line(int index, @NotNull net.kyori.adventure.text.Component line) throws IndexOutOfBoundsException; + // Paper end /** * Gets all the lines of text currently on this sign. * * @return Array of Strings containing each line of text + * @deprecated in favour of {@link #lines()} */ @NotNull + @Deprecated // Paper public String[] getLines(); /** @@ -24,8 +59,10 @@ public interface Sign extends TileState, Colorable { * @param index Line number to get the text from, starting at 0 * @throws IndexOutOfBoundsException Thrown when the line does not exist * @return Text on the given line + * @deprecated in favour of {@link #line(int)} */ @NotNull + @Deprecated // Paper public String getLine(int index) throws IndexOutOfBoundsException; /** @@ -37,7 +74,9 @@ public interface Sign extends TileState, Colorable { * @param index Line number to set the text at, starting from 0 * @param line New text to set at the specified index * @throws IndexOutOfBoundsException If the index is out of the range 0..3 + * @deprecated in favour of {@link #line(int, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setLine(int index, @NotNull String line) throws IndexOutOfBoundsException; /** diff --git a/src/main/java/org/bukkit/command/CommandSender.java b/src/main/java/org/bukkit/command/CommandSender.java index ac772bf349e0ffe9cab1df165d9460b387f2fe69..c88418c7aa19b4fecdfa9af3d18ff202a5dc5763 100644 --- a/src/main/java/org/bukkit/command/CommandSender.java +++ b/src/main/java/org/bukkit/command/CommandSender.java @@ -6,12 +6,13 @@ import org.bukkit.permissions.Permissible; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; -public interface CommandSender extends Permissible { +public interface CommandSender extends net.kyori.adventure.audience.Audience, Permissible { // Paper /** * Sends this sender a message * * @param message Message to be displayed + * @see #sendMessage(net.kyori.adventure.text.Component) */ public void sendMessage(@NotNull String message); @@ -19,6 +20,7 @@ public interface CommandSender extends Permissible { * Sends this sender multiple messages * * @param messages An array of messages to be displayed + * @see #sendMessage(net.kyori.adventure.text.Component) */ public void sendMessage(@NotNull String[] messages); @@ -27,6 +29,7 @@ public interface CommandSender extends Permissible { * * @param message Message to be displayed * @param sender The sender of this message + * @see #sendMessage(net.kyori.adventure.identity.Identified, net.kyori.adventure.text.Component) */ public void sendMessage(@Nullable UUID sender, @NotNull String message); @@ -35,6 +38,7 @@ public interface CommandSender extends Permissible { * * @param messages An array of messages to be displayed * @param sender The sender of this message + * @see #sendMessage(net.kyori.adventure.identity.Identified, net.kyori.adventure.text.Component) */ public void sendMessage(@Nullable UUID sender, @NotNull String[] messages); @@ -61,7 +65,9 @@ public interface CommandSender extends Permissible { * Sends this sender a chat component. * * @param component the components to send + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void sendMessage(@NotNull net.md_5.bungee.api.chat.BaseComponent component) { throw new UnsupportedOperationException("Not supported yet."); } @@ -70,7 +76,9 @@ public interface CommandSender extends Permissible { * Sends an array of components as a single message to the sender. * * @param components the components to send + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void sendMessage(@NotNull net.md_5.bungee.api.chat.BaseComponent... components) { throw new UnsupportedOperationException("Not supported yet."); } @@ -80,7 +88,9 @@ public interface CommandSender extends Permissible { * * @param component the components to send * @param sender the sender of the message + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void sendMessage(@Nullable UUID sender, @NotNull net.md_5.bungee.api.chat.BaseComponent component) { throw new UnsupportedOperationException("Not supported yet."); } @@ -90,7 +100,9 @@ public interface CommandSender extends Permissible { * * @param components the components to send * @param sender the sender of the message + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void sendMessage(@Nullable UUID sender, @NotNull net.md_5.bungee.api.chat.BaseComponent... components) { throw new UnsupportedOperationException("Not supported yet."); } @@ -99,4 +111,11 @@ public interface CommandSender extends Permissible { @NotNull Spigot spigot(); // Spigot end + + // Paper start + @Override + default void sendMessage(final @NotNull net.kyori.adventure.identity.Identity identity, final @NotNull net.kyori.adventure.text.Component message, final @NotNull net.kyori.adventure.audience.MessageType type) { + this.sendMessage(org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(message)); + } + // Paper end } diff --git a/src/main/java/org/bukkit/command/ProxiedCommandSender.java b/src/main/java/org/bukkit/command/ProxiedCommandSender.java index fcc34b640265f4dccb46b9f09466ab8e1d96043e..74599b4ee0518481c0e3a5f6ab2f5302837f1ae3 100644 --- a/src/main/java/org/bukkit/command/ProxiedCommandSender.java +++ b/src/main/java/org/bukkit/command/ProxiedCommandSender.java @@ -3,7 +3,7 @@ package org.bukkit.command; import org.jetbrains.annotations.NotNull; -public interface ProxiedCommandSender extends CommandSender { +public interface ProxiedCommandSender extends CommandSender, net.kyori.adventure.audience.ForwardingAudience.Single { // Paper /** * Returns the CommandSender which triggered this proxied command @@ -21,4 +21,16 @@ public interface ProxiedCommandSender extends CommandSender { @NotNull CommandSender getCallee(); + // Paper start + @Override + default void sendMessage(final @NotNull net.kyori.adventure.identity.Identity source, final @NotNull net.kyori.adventure.text.Component message, final @NotNull net.kyori.adventure.audience.MessageType type) { + net.kyori.adventure.audience.ForwardingAudience.Single.super.sendMessage(source, message, type); + } + + @NotNull + @Override + default net.kyori.adventure.audience.Audience audience() { + return this.getCaller(); + } + // Paper end } diff --git a/src/main/java/org/bukkit/enchantments/Enchantment.java b/src/main/java/org/bukkit/enchantments/Enchantment.java index b833ef63fbe01271ceb2bd83a9eb4a84c9912761..8eb0497c81744874809ebc4bc2e28b128e66a926 100644 --- a/src/main/java/org/bukkit/enchantments/Enchantment.java +++ b/src/main/java/org/bukkit/enchantments/Enchantment.java @@ -294,6 +294,19 @@ public abstract class Enchantment implements Keyed { * @return True if the enchantment may be applied, otherwise False */ public abstract boolean canEnchantItem(@NotNull ItemStack item); + // Paper start + /** + * Get the name of the enchantment with its applied level. + *

+ * If the given {@code level} is either less than the {@link #getStartLevel()} or greater than the {@link #getMaxLevel()}, + * the level may not be shown in the numeral format one may otherwise expect. + *

+ * + * @param level the level of the enchantment to show + * @return the name of the enchantment with {@code level} applied + */ + public abstract @NotNull net.kyori.adventure.text.Component displayName(int level); + // Paper end @Override public boolean equals(Object obj) { diff --git a/src/main/java/org/bukkit/enchantments/EnchantmentWrapper.java b/src/main/java/org/bukkit/enchantments/EnchantmentWrapper.java index 9566e4306ada5e82dede0f002aa06da12c44996b..4d5f0837bd0e02a30c943d8969fb6b13452322e0 100644 --- a/src/main/java/org/bukkit/enchantments/EnchantmentWrapper.java +++ b/src/main/java/org/bukkit/enchantments/EnchantmentWrapper.java @@ -63,4 +63,11 @@ public class EnchantmentWrapper extends Enchantment { public boolean conflictsWith(@NotNull Enchantment other) { return getEnchantment().conflictsWith(other); } + // Paper start + @NotNull + @Override + public net.kyori.adventure.text.Component displayName(int level) { + return getEnchantment().displayName(level); + } + // Paper end } diff --git a/src/main/java/org/bukkit/entity/Entity.java b/src/main/java/org/bukkit/entity/Entity.java index b71530e94569f8ade829e655e3a89ec70215ea39..b21fe6c631d5a6cd52f133786270c8f5ad0dafc4 100644 --- a/src/main/java/org/bukkit/entity/Entity.java +++ b/src/main/java/org/bukkit/entity/Entity.java @@ -25,7 +25,7 @@ import org.jetbrains.annotations.Nullable; /** * Represents a base entity in the world */ -public interface Entity extends Metadatable, CommandSender, Nameable, PersistentDataHolder { +public interface Entity extends Metadatable, CommandSender, Nameable, PersistentDataHolder, net.kyori.adventure.text.event.HoverEventSource { // Paper /** * Gets the entity's current position @@ -602,4 +602,12 @@ public interface Entity extends Metadatable, CommandSender, Nameable, Persistent @Override Spigot spigot(); // Spigot end + + // Paper start + @NotNull + @Override + default net.kyori.adventure.text.event.HoverEvent asHoverEvent(final @NotNull java.util.function.UnaryOperator op) { + return net.kyori.adventure.text.event.HoverEvent.showEntity(op.apply(net.kyori.adventure.text.event.HoverEvent.ShowEntity.of(this.getType().getKey(), this.getUniqueId(), this.customName()))); + } + // Paper end } diff --git a/src/main/java/org/bukkit/entity/Player.java b/src/main/java/org/bukkit/entity/Player.java index 586fd9ebd02039ebd2c071cbbbf60f24879f96b9..171e2663301f654258dbd772a57688b5a2f8469c 100644 --- a/src/main/java/org/bukkit/entity/Player.java +++ b/src/main/java/org/bukkit/entity/Player.java @@ -30,7 +30,28 @@ import org.jetbrains.annotations.Nullable; /** * Represents a player, connected or not */ -public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginMessageRecipient { +public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginMessageRecipient, net.kyori.adventure.identity.Identified { // Paper + + // Paper start + @Override + default @NotNull net.kyori.adventure.identity.Identity identity() { + return net.kyori.adventure.identity.Identity.identity(this.getUniqueId()); + } + + /** + * Gets the "friendly" name to display of this player. + * + * @return the display name + */ + @NotNull net.kyori.adventure.text.Component displayName(); + + /** + * Sets the "friendly" name to display of this player. + * + * @param displayName the display name to set + */ + void displayName(final @Nullable net.kyori.adventure.text.Component displayName); + // Paper end /** * Gets the "friendly" name to display of this player. This may include @@ -40,7 +61,9 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * places defined by plugins. * * @return the friendly name + * @deprecated in favour of {@link #displayName()} */ + @Deprecated // Paper @NotNull public String getDisplayName(); @@ -52,15 +75,50 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * places defined by plugins. * * @param name The new display name. + * @deprecated in favour of {@link #displayName(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setDisplayName(@Nullable String name); + // Paper start + /** + * Sets the name that is shown on the in-game player list. + *

+ * If the value is null, the name will be identical to {@link #getName()}. + * + * @param name new player list name + */ + void playerListName(@Nullable net.kyori.adventure.text.Component name); + + /** + * Gets the name that is shown on the in-game player list. + * + * @return the player list name + */ + @Nullable net.kyori.adventure.text.Component playerListName(); + + /** + * Gets the currently displayed player list header for this player. + * + * @return player list header or null + */ + @Nullable net.kyori.adventure.text.Component playerListHeader(); + + /** + * Gets the currently displayed player list footer for this player. + * + * @return player list footer or null + */ + @Nullable net.kyori.adventure.text.Component playerListFooter(); + // Paper end /** * Gets the name that is shown on the player list. * * @return the player list name + * @deprecated in favour of {@link #playerListName()} */ @NotNull + @Deprecated // Paper public String getPlayerListName(); /** @@ -69,14 +127,18 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * If the value is null, the name will be identical to {@link #getName()}. * * @param name new player list name + * @deprecated in favour of {@link #playerListName(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setPlayerListName(@Nullable String name); /** * Gets the currently displayed player list header for this player. * * @return player list header or null + * @deprecated in favour of {@link #playerListHeader()} */ + @Deprecated // Paper @Nullable public String getPlayerListHeader(); @@ -84,7 +146,9 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * Gets the currently displayed player list footer for this player. * * @return player list header or null + * @deprecated in favour of {@link #playerListFooter()} */ + @Deprecated // Paper @Nullable public String getPlayerListFooter(); @@ -92,14 +156,18 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * Sets the currently displayed player list header for this player. * * @param header player list header, null for empty + * @deprecated in favour of {@link #sendPlayerListHeader(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setPlayerListHeader(@Nullable String header); /** * Sets the currently displayed player list footer for this player. * * @param footer player list footer, null for empty + * @deprecated in favour of {@link #sendPlayerListFooter(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setPlayerListFooter(@Nullable String footer); /** @@ -108,7 +176,9 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * * @param header player list header, null for empty * @param footer player list footer, null for empty + * @deprecated in favour of {@link #sendPlayerListHeaderAndFooter(net.kyori.adventure.text.Component, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setPlayerListHeaderFooter(@Nullable String header, @Nullable String footer); /** @@ -146,9 +216,20 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * Kicks player with custom kick message. * * @param message kick message + * @deprecated in favour of {@link #kick(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void kickPlayer(@Nullable String message); + // Paper start + /** + * Kicks player with custom kick message. + * + * @param message kick message + */ + void kick(final @Nullable net.kyori.adventure.text.Component message); + // Paper end + /** * Says a message (or runs a command). * @@ -448,6 +529,7 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM @Deprecated public boolean sendChunkChange(@NotNull Location loc, int sx, int sy, int sz, @NotNull byte[] data); + // Paper start /** * Send a sign change. This fakes a sign change packet for a user at * a certain location. This will not actually change the world in any way. @@ -463,6 +545,43 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * @throws IllegalArgumentException if location is null * @throws IllegalArgumentException if lines is non-null and has a length less than 4 */ + void sendSignChange(@NotNull Location loc, @Nullable java.util.List lines) throws IllegalArgumentException; + /** + * Send a sign change. This fakes a sign change packet for a user at + * a certain location. This will not actually change the world in any way. + * This method will use a sign at the location's block or a faked sign + * sent via + * {@link #sendBlockChange(org.bukkit.Location, org.bukkit.Material, byte)}. + *

+ * If the client does not have a sign at the given location it will + * display an error message to the user. + * + * @param loc the location of the sign + * @param lines the new text on the sign or null to clear it + * @param dyeColor the color of the sign + * @throws IllegalArgumentException if location is null + * @throws IllegalArgumentException if dyeColor is null + * @throws IllegalArgumentException if lines is non-null and has a length less than 4 + */ + void sendSignChange(@NotNull Location loc, @Nullable java.util.List lines, @NotNull DyeColor dyeColor) throws IllegalArgumentException; + // Paper end + /** + * Send a sign change. This fakes a sign change packet for a user at + * a certain location. This will not actually change the world in any way. + * This method will use a sign at the location's block or a faked sign + * sent via + * {@link #sendBlockChange(org.bukkit.Location, org.bukkit.Material, byte)}. + *

+ * If the client does not have a sign at the given location it will + * display an error message to the user. + * + * @param loc the location of the sign + * @param lines the new text on the sign or null to clear it + * @throws IllegalArgumentException if location is null + * @throws IllegalArgumentException if lines is non-null and has a length less than 4 + * @deprecated in favour of {@link #sendSignChange(org.bukkit.Location, java.util.List)} + */ + @Deprecated // Paper public void sendSignChange(@NotNull Location loc, @Nullable String[] lines) throws IllegalArgumentException; @@ -482,7 +601,9 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * @throws IllegalArgumentException if location is null * @throws IllegalArgumentException if dyeColor is null * @throws IllegalArgumentException if lines is non-null and has a length less than 4 + * @deprecated in favour of {@link #sendSignChange(org.bukkit.Location, java.util.List, org.bukkit.DyeColor)} */ + @Deprecated // Paper public void sendSignChange(@NotNull Location loc, @Nullable String[] lines, @NotNull DyeColor dyeColor) throws IllegalArgumentException; /** @@ -1220,6 +1341,14 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM */ public int getClientViewDistance(); + // Paper start + /** + * Gets the player's current locale. + * + * @return the player's locale + */ + @NotNull java.util.Locale locale(); + // Paper end /** * Gets the player's estimated ping in milliseconds. * @@ -1245,8 +1374,10 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * they wish. * * @return the player's locale + * @deprecated in favour of {@link #locale()} */ @NotNull + @Deprecated // Paper public String getLocale(); /** @@ -1264,6 +1395,14 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM */ public void openBook(@NotNull ItemStack book); + // Paper start + @NotNull + @Override + default net.kyori.adventure.text.event.HoverEvent asHoverEvent(final @NotNull java.util.function.UnaryOperator op) { + return net.kyori.adventure.text.event.HoverEvent.showEntity(op.apply(net.kyori.adventure.text.event.HoverEvent.ShowEntity.of(this.getType().getKey(), this.getUniqueId(), this.displayName()))); + } + // Paper end + // Spigot start public class Spigot extends Entity.Spigot { @@ -1318,11 +1457,13 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM throw new UnsupportedOperationException("Not supported yet."); } + @Deprecated // Paper @Override public void sendMessage(@NotNull net.md_5.bungee.api.chat.BaseComponent component) { throw new UnsupportedOperationException("Not supported yet."); } + @Deprecated // Paper @Override public void sendMessage(@NotNull net.md_5.bungee.api.chat.BaseComponent... components) { throw new UnsupportedOperationException("Not supported yet."); @@ -1333,7 +1474,9 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * * @param position the screen position * @param component the components to send + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void sendMessage(@NotNull net.md_5.bungee.api.ChatMessageType position, @NotNull net.md_5.bungee.api.chat.BaseComponent component) { throw new UnsupportedOperationException("Not supported yet."); } @@ -1343,7 +1486,9 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * * @param position the screen position * @param components the components to send + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void sendMessage(@NotNull net.md_5.bungee.api.ChatMessageType position, @NotNull net.md_5.bungee.api.chat.BaseComponent... components) { throw new UnsupportedOperationException("Not supported yet."); } @@ -1354,7 +1499,9 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * @param position the screen position * @param sender the sender of the message * @param component the components to send + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void sendMessage(@NotNull net.md_5.bungee.api.ChatMessageType position, @Nullable UUID sender, @NotNull net.md_5.bungee.api.chat.BaseComponent component) { throw new UnsupportedOperationException("Not supported yet."); } @@ -1365,7 +1512,9 @@ public interface Player extends HumanEntity, Conversable, OfflinePlayer, PluginM * @param position the screen position * @param sender the sender of the message * @param components the components to send + * @deprecated use {@code sendMessage} methods that accept {@link net.kyori.adventure.text.Component} */ + @Deprecated // Paper public void sendMessage(@NotNull net.md_5.bungee.api.ChatMessageType position, @Nullable UUID sender, @NotNull net.md_5.bungee.api.chat.BaseComponent... components) { throw new UnsupportedOperationException("Not supported yet."); diff --git a/src/main/java/org/bukkit/entity/minecart/CommandMinecart.java b/src/main/java/org/bukkit/entity/minecart/CommandMinecart.java index 63c80b4ee1f7adc8a9efc3b607993104b1991f90..91cab8b13d5bba34007f124838b32a1df58c5ac7 100644 --- a/src/main/java/org/bukkit/entity/minecart/CommandMinecart.java +++ b/src/main/java/org/bukkit/entity/minecart/CommandMinecart.java @@ -32,7 +32,9 @@ public interface CommandMinecart extends Minecart { * same as setting it to "@". * * @param name New name for this CommandMinecart. + * @deprecated in favour of {@link #customName(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setName(@Nullable String name); } diff --git a/src/main/java/org/bukkit/event/block/SignChangeEvent.java b/src/main/java/org/bukkit/event/block/SignChangeEvent.java index 7190db11eff7d48df8a99f405a9dbaefdfa76e3d..1f79f704abf339150df08900b8ea7da4cefef258 100644 --- a/src/main/java/org/bukkit/event/block/SignChangeEvent.java +++ b/src/main/java/org/bukkit/event/block/SignChangeEvent.java @@ -16,12 +16,25 @@ public class SignChangeEvent extends BlockEvent implements Cancellable { private static final HandlerList handlers = new HandlerList(); private boolean cancel = false; private final Player player; - private final String[] lines; + // Paper start + private final java.util.List adventure$lines; + public SignChangeEvent(@NotNull final Block theBlock, @NotNull final Player player, @NotNull final java.util.List adventure$lines) { + super(theBlock); + this.player = player; + this.adventure$lines = adventure$lines; + } + + @Deprecated // Paper end public SignChangeEvent(@NotNull final Block theBlock, @NotNull final Player thePlayer, @NotNull final String[] theLines) { super(theBlock); this.player = thePlayer; - this.lines = theLines; + // Paper start + this.adventure$lines = new java.util.ArrayList<>(); + for (String theLine : theLines) { + this.adventure$lines.add(org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(theLine)); + } + // Paper end } /** @@ -34,14 +47,52 @@ public class SignChangeEvent extends BlockEvent implements Cancellable { return player; } + // Paper start + /** + * Gets all of the lines of text from the sign involved in this event. + * + * @return the String array for the sign's lines new text + */ + public @NotNull java.util.List lines() { + return this.adventure$lines; + } + + /** + * Gets a single line of text from the sign involved in this event. + * + * @param index index of the line to get + * @return the String containing the line of text associated with the + * provided index + * @throws IndexOutOfBoundsException thrown when the provided index is {@literal > 3 + * or < 0} + */ + public @Nullable net.kyori.adventure.text.Component line(int index) throws IndexOutOfBoundsException { + return this.adventure$lines.get(index); + } + + /** + * Sets a single line for the sign involved in this event + * + * @param index index of the line to set + * @param line text to set + * @throws IndexOutOfBoundsException thrown when the provided index is {@literal > 3 + * or < 0} + */ + public void line(int index, @Nullable net.kyori.adventure.text.Component line) throws IndexOutOfBoundsException { + this.adventure$lines.set(index, line); + } + // Paper end + /** * Gets all of the lines of text from the sign involved in this event. * * @return the String array for the sign's lines new text + * @deprecated in favour of {@link #lines()} */ @NotNull + @Deprecated // Paper public String[] getLines() { - return lines; + return adventure$lines.stream().map(org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer()::serialize).toArray(String[]::new); // Paper } /** @@ -52,10 +103,12 @@ public class SignChangeEvent extends BlockEvent implements Cancellable { * provided index * @throws IndexOutOfBoundsException thrown when the provided index is {@literal > 3 * or < 0} + * @deprecated in favour of {@link #line(int)} */ @Nullable + @Deprecated // Paper public String getLine(int index) throws IndexOutOfBoundsException { - return lines[index]; + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.adventure$lines.get(index)); // Paper } /** @@ -65,9 +118,11 @@ public class SignChangeEvent extends BlockEvent implements Cancellable { * @param line text to set * @throws IndexOutOfBoundsException thrown when the provided index is {@literal > 3 * or < 0} + * @deprecated in favour of {@link #line(int, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setLine(int index, @Nullable String line) throws IndexOutOfBoundsException { - lines[index] = line; + adventure$lines.set(index, line != null ? org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(line) : null); // Paper } @Override diff --git a/src/main/java/org/bukkit/event/entity/PlayerDeathEvent.java b/src/main/java/org/bukkit/event/entity/PlayerDeathEvent.java index 3c2ea8fec3a748cab7f5ad9100d12bd8213ec6c9..a01d4c21bedc7f1a54f5a330bb4c2909ce3a18e4 100644 --- a/src/main/java/org/bukkit/event/entity/PlayerDeathEvent.java +++ b/src/main/java/org/bukkit/event/entity/PlayerDeathEvent.java @@ -12,25 +12,48 @@ import org.jetbrains.annotations.Nullable; public class PlayerDeathEvent extends EntityDeathEvent { private int newExp = 0; private String deathMessage = ""; + private net.kyori.adventure.text.Component adventure$deathMessage; // Paper private int newLevel = 0; private int newTotalExp = 0; private boolean keepLevel = false; private boolean keepInventory = false; + // Paper start + public PlayerDeathEvent(@NotNull final Player player, @NotNull final List drops, final int droppedExp, @Nullable final net.kyori.adventure.text.Component adventure$deathMessage) { + this(player, drops, droppedExp, 0, adventure$deathMessage, null); + } + + public PlayerDeathEvent(@NotNull final Player player, @NotNull final List drops, final int droppedExp, final int newExp, @Nullable final net.kyori.adventure.text.Component adventure$deathMessage, @Nullable String deathMessage) { + this(player, drops, droppedExp, newExp, 0, 0, adventure$deathMessage, deathMessage); + } + public PlayerDeathEvent(@NotNull final Player player, @NotNull final List drops, final int droppedExp, final int newExp, final int newTotalExp, final int newLevel, @Nullable final net.kyori.adventure.text.Component adventure$deathMessage, @Nullable String deathMessage) { + super(player, drops, droppedExp); + this.newExp = newExp; + this.newTotalExp = newTotalExp; + this.newLevel = newLevel; + this.deathMessage = deathMessage; + this.adventure$deathMessage = adventure$deathMessage; + } + // Paper end + + @Deprecated // Paper public PlayerDeathEvent(@NotNull final Player player, @NotNull final List drops, final int droppedExp, @Nullable final String deathMessage) { this(player, drops, droppedExp, 0, deathMessage); } + @Deprecated // Paper public PlayerDeathEvent(@NotNull final Player player, @NotNull final List drops, final int droppedExp, final int newExp, @Nullable final String deathMessage) { this(player, drops, droppedExp, newExp, 0, 0, deathMessage); } + @Deprecated // Paper public PlayerDeathEvent(@NotNull final Player player, @NotNull final List drops, final int droppedExp, final int newExp, final int newTotalExp, final int newLevel, @Nullable final String deathMessage) { super(player, drops, droppedExp); this.newExp = newExp; this.newTotalExp = newTotalExp; this.newLevel = newLevel; this.deathMessage = deathMessage; + this.adventure$deathMessage = deathMessage != null ? org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(deathMessage) : net.kyori.adventure.text.Component.empty(); // Paper } @NotNull @@ -39,25 +62,55 @@ public class PlayerDeathEvent extends EntityDeathEvent { return (Player) entity; } + // Paper start + /** + * Set the death message that will appear to everyone on the server. + * + * @param deathMessage Message to appear to other players on the server. + */ + public void deathMessage(@Nullable net.kyori.adventure.text.Component deathMessage) { + this.deathMessage = null; + this.adventure$deathMessage = deathMessage; + } + + /** + * Get the death message that will appear to everyone on the server. + * + * @return Message to appear to other players on the server. + */ + public @Nullable net.kyori.adventure.text.Component deathMessage() { + return this.adventure$deathMessage; + } + // Paper end + /** * Set the death message that will appear to everyone on the server. * * @param deathMessage Message to appear to other players on the server. + * @deprecated in favour of {@link #deathMessage(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setDeathMessage(@Nullable String deathMessage) { this.deathMessage = deathMessage; + this.adventure$deathMessage = deathMessage != null ? org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(deathMessage) : net.kyori.adventure.text.Component.empty(); // Paper } /** * Get the death message that will appear to everyone on the server. * * @return Message to appear to other players on the server. + * @deprecated in favour of {@link #deathMessage()} */ @Nullable + @Deprecated // Paper public String getDeathMessage() { - return deathMessage; + return this.deathMessage != null ? this.deathMessage : (this.adventure$deathMessage != null ? getDeathMessageString(this.adventure$deathMessage) : null); // Paper } - + // Paper start //TODO: add translation API to drop String deathMessage in favor of just Adventure + private static String getDeathMessageString(net.kyori.adventure.text.Component component) { + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(component); + } + // Paper end /** * Gets how much EXP the Player should have at respawn. *

diff --git a/src/main/java/org/bukkit/event/inventory/InventoryType.java b/src/main/java/org/bukkit/event/inventory/InventoryType.java index f1e9bc9bc797b7216336d3470e3c696a06f2b21a..090d22bd30f7947103771aaaf09a2398970ac337 100644 --- a/src/main/java/org/bukkit/event/inventory/InventoryType.java +++ b/src/main/java/org/bukkit/event/inventory/InventoryType.java @@ -136,6 +136,18 @@ public enum InventoryType { private final String title; private final boolean isCreatable; + // Paper start + private final net.kyori.adventure.text.Component defaultTitleComponent; + + /** + * Gets the inventory's default title. + * + * @return the inventory's default title + */ + public @NotNull net.kyori.adventure.text.Component defaultTitle() { + return defaultTitleComponent; + } + // Paper end private InventoryType(int defaultSize, /*@NotNull*/ String defaultTitle) { this(defaultSize, defaultTitle, true); } @@ -144,6 +156,7 @@ public enum InventoryType { size = defaultSize; title = defaultTitle; this.isCreatable = isCreatable; + this.defaultTitleComponent = net.kyori.adventure.text.Component.text(defaultTitle); // Paper - Adventure } public int getDefaultSize() { @@ -151,6 +164,7 @@ public enum InventoryType { } @NotNull + @Deprecated // Paper public String getDefaultTitle() { return title; } diff --git a/src/main/java/org/bukkit/event/player/AsyncPlayerChatEvent.java b/src/main/java/org/bukkit/event/player/AsyncPlayerChatEvent.java index 9c68c3f2d61500479f48b80264f625aaae2f3204..399afcd19fcb6acd24857ed6ab48cf0d105a01a3 100644 --- a/src/main/java/org/bukkit/event/player/AsyncPlayerChatEvent.java +++ b/src/main/java/org/bukkit/event/player/AsyncPlayerChatEvent.java @@ -22,7 +22,11 @@ import org.jetbrains.annotations.NotNull; *

* Care should be taken to check {@link #isAsynchronous()} and treat the event * appropriately. + * + * @deprecated use {@link io.papermc.paper.event.player.AsyncChatEvent} instead */ +@Deprecated // Paper +@org.bukkit.Warning(value = false, reason = "Don't nag on old event yet") // Paper public class AsyncPlayerChatEvent extends PlayerEvent implements Cancellable { private static final HandlerList handlers = new HandlerList(); private boolean cancel = false; diff --git a/src/main/java/org/bukkit/event/player/AsyncPlayerPreLoginEvent.java b/src/main/java/org/bukkit/event/player/AsyncPlayerPreLoginEvent.java index c8384da69af61e1970f254a3a9c206ee81d7a989..d3b4219a57fff4519ef8d803c333c854fafa7859 100644 --- a/src/main/java/org/bukkit/event/player/AsyncPlayerPreLoginEvent.java +++ b/src/main/java/org/bukkit/event/player/AsyncPlayerPreLoginEvent.java @@ -14,7 +14,7 @@ import org.jetbrains.annotations.NotNull; public class AsyncPlayerPreLoginEvent extends Event { private static final HandlerList handlers = new HandlerList(); private Result result; - private String message; + private net.kyori.adventure.text.Component message; // Paper private final String name; private final InetAddress ipAddress; private final UUID uniqueId; @@ -27,7 +27,7 @@ public class AsyncPlayerPreLoginEvent extends Event { public AsyncPlayerPreLoginEvent(@NotNull final String name, @NotNull final InetAddress ipAddress, @NotNull final UUID uniqueId) { super(true); this.result = Result.ALLOWED; - this.message = ""; + this.message = net.kyori.adventure.text.Component.empty(); // Paper this.name = name; this.ipAddress = ipAddress; this.uniqueId = uniqueId; @@ -79,6 +79,7 @@ public class AsyncPlayerPreLoginEvent extends Event { this.result = result == null ? null : Result.valueOf(result.name()); } + // Paper start /** * Gets the current kick message that will be used if getResult() != * Result.ALLOWED @@ -86,7 +87,7 @@ public class AsyncPlayerPreLoginEvent extends Event { * @return Current kick message */ @NotNull - public String getKickMessage() { + public net.kyori.adventure.text.Component kickMessage() { return message; } @@ -95,16 +96,66 @@ public class AsyncPlayerPreLoginEvent extends Event { * * @param message New kick message */ - public void setKickMessage(@NotNull final String message) { + public void kickMessage(@NotNull final net.kyori.adventure.text.Component message) { + this.message = message; + } + + /** + * Disallows the player from logging in, with the given reason + * + * @param result New result for disallowing the player + * @param message Kick message to display to the user + */ + public void disallow(@NotNull final Result result, @NotNull final net.kyori.adventure.text.Component message) { + this.result = result; this.message = message; } + /** + * Disallows the player from logging in, with the given reason + * + * @param result New result for disallowing the player + * @param message Kick message to display to the user + * @deprecated This method uses a deprecated enum from {@link + * PlayerPreLoginEvent} + * @see #disallow(Result, String) + */ + @Deprecated + public void disallow(@NotNull final PlayerPreLoginEvent.Result result, @NotNull final net.kyori.adventure.text.Component message) { + this.result = result == null ? null : Result.valueOf(result.name()); + this.message = message; + } + // Paper end + /** + * Gets the current kick message that will be used if getResult() != + * Result.ALLOWED + * + * @return Current kick message + * @deprecated in favour of {@link #kickMessage()} + */ + @NotNull + @Deprecated // Paper + public String getKickMessage() { + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.message); // Paper + } + + /** + * Sets the kick message to display if getResult() != Result.ALLOWED + * + * @param message New kick message + * @deprecated in favour of {@link #kickMessage(net.kyori.adventure.text.Component)} + */ + @Deprecated // Paper + public void setKickMessage(@NotNull final String message) { + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); // Paper + } + /** * Allows the player to log in */ public void allow() { result = Result.ALLOWED; - message = ""; + message = net.kyori.adventure.text.Component.empty(); // Paper } /** @@ -112,10 +163,12 @@ public class AsyncPlayerPreLoginEvent extends Event { * * @param result New result for disallowing the player * @param message Kick message to display to the user + * @deprecated in favour of {@link #disallow(org.bukkit.event.player.AsyncPlayerPreLoginEvent.Result, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void disallow(@NotNull final Result result, @NotNull final String message) { this.result = result; - this.message = message; + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); // Paper } /** @@ -130,7 +183,7 @@ public class AsyncPlayerPreLoginEvent extends Event { @Deprecated public void disallow(@NotNull final PlayerPreLoginEvent.Result result, @NotNull final String message) { this.result = result == null ? null : Result.valueOf(result.name()); - this.message = message; + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); // Paper } /** diff --git a/src/main/java/org/bukkit/event/player/PlayerChatEvent.java b/src/main/java/org/bukkit/event/player/PlayerChatEvent.java index 8ea56aac752544f798728b429e7152afbee497e4..213837794c603cb9f152f917941b912326a08030 100644 --- a/src/main/java/org/bukkit/event/player/PlayerChatEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerChatEvent.java @@ -18,6 +18,7 @@ import org.jetbrains.annotations.NotNull; * Listening to this event forces chat to wait for the main thread which * causes delays for chat. {@link AsyncPlayerChatEvent} is the encouraged * alternative for thread safe implementations. + * @deprecated use {@link io.papermc.paper.event.player.ChatEvent} instead */ @Deprecated @Warning(reason = "Listening to this event forces chat to wait for the main thread, delaying chat messages.") diff --git a/src/main/java/org/bukkit/event/player/PlayerEvent.java b/src/main/java/org/bukkit/event/player/PlayerEvent.java index 793b661b6d2d05de3d7f4fc26a4c018a2af58e62..f6d3b817de3001f04ea4554c7c39a1290af3fd6d 100644 --- a/src/main/java/org/bukkit/event/player/PlayerEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerEvent.java @@ -14,7 +14,7 @@ public abstract class PlayerEvent extends Event { player = who; } - PlayerEvent(@NotNull final Player who, boolean async) { + public PlayerEvent(@NotNull final Player who, boolean async) { // Paper - public super(async); player = who; diff --git a/src/main/java/org/bukkit/event/player/PlayerJoinEvent.java b/src/main/java/org/bukkit/event/player/PlayerJoinEvent.java index d06684aba7688ce06777dbd837a46856a9d7767f..851a189d42e271679abc78f95049d8badf7a2b64 100644 --- a/src/main/java/org/bukkit/event/player/PlayerJoinEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerJoinEvent.java @@ -10,30 +10,60 @@ import org.jetbrains.annotations.Nullable; */ public class PlayerJoinEvent extends PlayerEvent { private static final HandlerList handlers = new HandlerList(); - private String joinMessage; + // Paper start + private net.kyori.adventure.text.Component joinMessage; + public PlayerJoinEvent(@NotNull final Player playerJoined, @Nullable final net.kyori.adventure.text.Component joinMessage) { + super(playerJoined); + this.joinMessage = joinMessage; + } + @Deprecated // Paper end public PlayerJoinEvent(@NotNull final Player playerJoined, @Nullable final String joinMessage) { super(playerJoined); + this.joinMessage = joinMessage != null ? org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(joinMessage) : null; // Paper end + } + + // Paper start + /** + * Gets the join message to send to all online players + * + * @return string join message. Can be null + */ + public @Nullable net.kyori.adventure.text.Component joinMessage() { + return this.joinMessage; + } + + /** + * Sets the join message to send to all online players + * + * @param joinMessage join message. If null, no message will be sent + */ + public void joinMessage(@Nullable net.kyori.adventure.text.Component joinMessage) { this.joinMessage = joinMessage; } + // Paper end /** * Gets the join message to send to all online players * * @return string join message. Can be null + * @deprecated in favour of {@link #joinMessage()} */ @Nullable + @Deprecated // Paper public String getJoinMessage() { - return joinMessage; + return this.joinMessage == null ? null : org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.joinMessage); // Paper } /** * Sets the join message to send to all online players * * @param joinMessage join message. If null, no message will be sent + * @deprecated in favour of {@link #joinMessage(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setJoinMessage(@Nullable String joinMessage) { - this.joinMessage = joinMessage; + this.joinMessage = joinMessage != null ? org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(joinMessage) : null; // Paper } @NotNull diff --git a/src/main/java/org/bukkit/event/player/PlayerKickEvent.java b/src/main/java/org/bukkit/event/player/PlayerKickEvent.java index 14c337f15fc804f52e52cb0a185aad38d89303a8..5c0efe74237dbe6803ce023fde99682ff70d1a92 100644 --- a/src/main/java/org/bukkit/event/player/PlayerKickEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerKickEvent.java @@ -10,35 +10,84 @@ import org.jetbrains.annotations.NotNull; */ public class PlayerKickEvent extends PlayerEvent implements Cancellable { private static final HandlerList handlers = new HandlerList(); - private String leaveMessage; - private String kickReason; + private net.kyori.adventure.text.Component leaveMessage; // Paper + private net.kyori.adventure.text.Component kickReason; // Paper private Boolean cancel; + @Deprecated // Paper public PlayerKickEvent(@NotNull final Player playerKicked, @NotNull final String kickReason, @NotNull final String leaveMessage) { + super(playerKicked); + this.kickReason = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(kickReason); // Paper + this.leaveMessage = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(leaveMessage); // Paper + this.cancel = false; + } + // Paper start + public PlayerKickEvent(@NotNull final Player playerKicked, @NotNull final net.kyori.adventure.text.Component kickReason, @NotNull final net.kyori.adventure.text.Component leaveMessage) { super(playerKicked); this.kickReason = kickReason; this.leaveMessage = leaveMessage; this.cancel = false; } + /** + * Gets the leave message send to all online players + * + * @return string kick reason + */ + public @NotNull net.kyori.adventure.text.Component leaveMessage() { + return this.leaveMessage; + } + + /** + * Sets the leave message send to all online players + * + * @param leaveMessage leave message + */ + public void leaveMessage(@NotNull net.kyori.adventure.text.Component leaveMessage) { + this.leaveMessage = leaveMessage; + } + /** * Gets the reason why the player is getting kicked * * @return string kick reason */ + public @NotNull net.kyori.adventure.text.Component reason() { + return this.kickReason; + } + + /** + * Sets the reason why the player is getting kicked + * + * @param kickReason kick reason + */ + public void reason(@NotNull net.kyori.adventure.text.Component kickReason) { + this.kickReason = kickReason; + } + // Paper end + + /** + * Gets the reason why the player is getting kicked + * + * @return string kick reason + * @deprecated in favour of {@link #reason()} + */ @NotNull + @Deprecated // Paper public String getReason() { - return kickReason; + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.kickReason); // Paper } /** * Gets the leave message send to all online players * * @return string kick reason + * @deprecated in favour of {@link #leaveMessage()} */ @NotNull + @Deprecated // Paper public String getLeaveMessage() { - return leaveMessage; + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.leaveMessage); // Paper } @Override @@ -55,18 +104,22 @@ public class PlayerKickEvent extends PlayerEvent implements Cancellable { * Sets the reason why the player is getting kicked * * @param kickReason kick reason + * @deprecated in favour of {@link #reason(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setReason(@NotNull String kickReason) { - this.kickReason = kickReason; + this.kickReason = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(kickReason); // Paper } /** * Sets the leave message send to all online players * * @param leaveMessage leave message + * @deprecated in favour of {@link #leaveMessage(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setLeaveMessage(@NotNull String leaveMessage) { - this.leaveMessage = leaveMessage; + this.leaveMessage = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(leaveMessage); // Paper } @NotNull diff --git a/src/main/java/org/bukkit/event/player/PlayerLocaleChangeEvent.java b/src/main/java/org/bukkit/event/player/PlayerLocaleChangeEvent.java index 1db386bb701cb6974daedc6bb5b93a3afbc42100..84521186404b8e43c81a2f9513dce2be40d27840 100644 --- a/src/main/java/org/bukkit/event/player/PlayerLocaleChangeEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerLocaleChangeEvent.java @@ -12,18 +12,32 @@ public class PlayerLocaleChangeEvent extends PlayerEvent { private static final HandlerList handlers = new HandlerList(); // private final String locale; + // Paper start + private final java.util.Locale adventure$locale; + /** + * @see Player#getLocale() + * + * @return the player's new locale + */ + public @NotNull java.util.Locale locale() { + return this.adventure$locale; + } + // Paper end public PlayerLocaleChangeEvent(@NotNull Player who, @NotNull String locale) { super(who); this.locale = locale; + this.adventure$locale = net.kyori.adventure.translation.Translator.parseLocale(locale); // Paper } /** * @see Player#getLocale() * * @return the player's new locale + * @deprecated in favour of {@link #locale()} */ @NotNull + @Deprecated // Paper public String getLocale() { return locale; } diff --git a/src/main/java/org/bukkit/event/player/PlayerLoginEvent.java b/src/main/java/org/bukkit/event/player/PlayerLoginEvent.java index 084ca8cfcb7381bfa4fa6280748cf9b81210a9c1..75cc54739ef841cd90568d74927d6002d4cfa7e0 100644 --- a/src/main/java/org/bukkit/event/player/PlayerLoginEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerLoginEvent.java @@ -17,7 +17,7 @@ public class PlayerLoginEvent extends PlayerEvent { private final InetAddress address; private final String hostname; private Result result = Result.ALLOWED; - private String message = ""; + private net.kyori.adventure.text.Component message = net.kyori.adventure.text.Component.empty(); private final InetAddress realAddress; // Spigot /** @@ -53,12 +53,52 @@ public class PlayerLoginEvent extends PlayerEvent { * @param result The result status for this event * @param message The message to be displayed if result denies login * @param realAddress the actual, unspoofed connecting address + * @deprecated in favour of {@link #PlayerLoginEvent(Player, String, InetAddress, Result, net.kyori.adventure.text.Component, InetAddress)} */ + @Deprecated // Paper public PlayerLoginEvent(@NotNull final Player player, @NotNull String hostname, @NotNull final InetAddress address, @NotNull final Result result, @NotNull final String message, @NotNull final InetAddress realAddress) { // Spigot this(player, hostname, address, realAddress); // Spigot this.result = result; + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); // Paper + } + + // Paper start + /** + * This constructor pre-configures the event with a result and message + * + * @param player The {@link Player} for this event + * @param hostname The hostname that was used to connect to the server + * @param address The address the player used to connect, provided for + * timing issues + * @param result The result status for this event + * @param message The message to be displayed if result denies login + * @param realAddress the actual, unspoofed connecting address + */ + public PlayerLoginEvent(@NotNull final Player player, @NotNull String hostname, @NotNull final InetAddress address, @NotNull final Result result, @NotNull final net.kyori.adventure.text.Component message, @NotNull final InetAddress realAddress) { // Spigot + this(player, hostname, address, realAddress); // Spigot + this.result = result; + this.message = message; + } + + /** + * Gets the current kick message that will be used if getResult() != + * Result.ALLOWED + * + * @return Current kick message + */ + public @NotNull net.kyori.adventure.text.Component kickMessage() { + return this.message; + } + + /** + * Sets the kick message to display if getResult() != Result.ALLOWED + * + * @param message New kick message + */ + public void kickMessage(@NotNull net.kyori.adventure.text.Component message) { this.message = message; } + // Paper end // Spigot start /** @@ -96,19 +136,23 @@ public class PlayerLoginEvent extends PlayerEvent { * Result.ALLOWED * * @return Current kick message + * @deprecated in favour of {@link #kickMessage()} */ @NotNull + @Deprecated // Paper public String getKickMessage() { - return message; + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.message); // Paper } /** * Sets the kick message to display if getResult() != Result.ALLOWED * * @param message New kick message + * @deprecated in favour of {@link #kickMessage(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setKickMessage(@NotNull final String message) { - this.message = message; + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); // Paper } /** @@ -127,7 +171,7 @@ public class PlayerLoginEvent extends PlayerEvent { */ public void allow() { result = Result.ALLOWED; - message = ""; + message = net.kyori.adventure.text.Component.empty(); // Paper } /** @@ -135,8 +179,21 @@ public class PlayerLoginEvent extends PlayerEvent { * * @param result New result for disallowing the player * @param message Kick message to display to the user + * @deprecated in favour of {@link #disallow(Result, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper start public void disallow(@NotNull final Result result, @NotNull final String message) { + this.result = result; + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); + } + /** + * Disallows the player from logging in, with the given reason + * + * @param result New result for disallowing the player + * @param message Kick message to display to the user + */ + public void disallow(@NotNull final Result result, @NotNull final net.kyori.adventure.text.Component message) { + // Paper end this.result = result; this.message = message; } diff --git a/src/main/java/org/bukkit/event/player/PlayerPreLoginEvent.java b/src/main/java/org/bukkit/event/player/PlayerPreLoginEvent.java index fb066251f793ec3b41bfc075b9478901b15ee549..123979ed64939d615b061f91c19c630e1e1db8c7 100644 --- a/src/main/java/org/bukkit/event/player/PlayerPreLoginEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerPreLoginEvent.java @@ -19,7 +19,7 @@ import org.jetbrains.annotations.NotNull; public class PlayerPreLoginEvent extends Event { private static final HandlerList handlers = new HandlerList(); private Result result; - private String message; + private net.kyori.adventure.text.Component message; // Paper private final String name; private final InetAddress ipAddress; private final UUID uniqueId; @@ -31,7 +31,7 @@ public class PlayerPreLoginEvent extends Event { public PlayerPreLoginEvent(@NotNull final String name, @NotNull final InetAddress ipAddress, @NotNull final UUID uniqueId) { this.result = Result.ALLOWED; - this.message = ""; + this.message = net.kyori.adventure.text.Component.empty(); // Paper this.name = name; this.ipAddress = ipAddress; this.uniqueId = uniqueId; @@ -56,6 +56,7 @@ public class PlayerPreLoginEvent extends Event { this.result = result; } + // Paper start /** * Gets the current kick message that will be used if getResult() != * Result.ALLOWED @@ -63,7 +64,7 @@ public class PlayerPreLoginEvent extends Event { * @return Current kick message */ @NotNull - public String getKickMessage() { + public net.kyori.adventure.text.Component kickMessage() { return message; } @@ -72,16 +73,51 @@ public class PlayerPreLoginEvent extends Event { * * @param message New kick message */ - public void setKickMessage(@NotNull final String message) { + public void kickMessage(@NotNull final net.kyori.adventure.text.Component message) { this.message = message; } + /** + * Disallows the player from logging in, with the given reason + * + * @param result New result for disallowing the player + * @param message Kick message to display to the user + */ + public void disallow(@NotNull final Result result, @NotNull final net.kyori.adventure.text.Component message) { + this.result = result; + this.message = message; + } + // Paper end + /** + * Gets the current kick message that will be used if getResult() != + * Result.ALLOWED + * + * @return Current kick message + * @deprecated in favour of {@link #kickMessage()} + */ + @Deprecated // Paper + @NotNull + public String getKickMessage() { + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.message); // Paper + } + + /** + * Sets the kick message to display if getResult() != Result.ALLOWED + * + * @param message New kick message + * @deprecated in favour of {@link #kickMessage(net.kyori.adventure.text.Component)} + */ + @Deprecated // Paper + public void setKickMessage(@NotNull final String message) { + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); // Paper + } + /** * Allows the player to log in */ public void allow() { result = Result.ALLOWED; - message = ""; + message = net.kyori.adventure.text.Component.empty(); // Paper } /** @@ -89,10 +125,12 @@ public class PlayerPreLoginEvent extends Event { * * @param result New result for disallowing the player * @param message Kick message to display to the user + * @deprecated in favour of {@link #disallow(org.bukkit.event.player.PlayerPreLoginEvent.Result, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void disallow(@NotNull final Result result, @NotNull final String message) { this.result = result; - this.message = message; + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); // Paper } /** diff --git a/src/main/java/org/bukkit/event/player/PlayerQuitEvent.java b/src/main/java/org/bukkit/event/player/PlayerQuitEvent.java index d70c25f404e994766a9ebce89a917c8d0719777c..849e8f10dd77e9fb46aab17752b8f1ff79e9d42e 100644 --- a/src/main/java/org/bukkit/event/player/PlayerQuitEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerQuitEvent.java @@ -10,30 +10,59 @@ import org.jetbrains.annotations.Nullable; */ public class PlayerQuitEvent extends PlayerEvent { private static final HandlerList handlers = new HandlerList(); - private String quitMessage; + private net.kyori.adventure.text.Component quitMessage; // Paper + @Deprecated // Paper public PlayerQuitEvent(@NotNull final Player who, @Nullable final String quitMessage) { super(who); + this.quitMessage = quitMessage != null ? org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(quitMessage) : null; // Paper + } + // Paper start + public PlayerQuitEvent(@NotNull final Player who, @Nullable final net.kyori.adventure.text.Component quitMessage) { + super(who); + this.quitMessage = quitMessage; + } + + /** + * Gets the quit message to send to all online players + * + * @return string quit message + */ + public @Nullable net.kyori.adventure.text.Component quitMessage() { + return quitMessage; + } + + /** + * Sets the quit message to send to all online players + * + * @param quitMessage quit message + */ + public void quitMessage(@Nullable net.kyori.adventure.text.Component quitMessage) { this.quitMessage = quitMessage; } + // Paper end /** * Gets the quit message to send to all online players * * @return string quit message + * @deprecated in favour of {@link #quitMessage()} */ @Nullable + @Deprecated // Paper public String getQuitMessage() { - return quitMessage; + return this.quitMessage == null ? null : org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.quitMessage); // Paper } /** * Sets the quit message to send to all online players * * @param quitMessage quit message + * @deprecated in favour of {@link #quitMessage(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setQuitMessage(@Nullable String quitMessage) { - this.quitMessage = quitMessage; + this.quitMessage = quitMessage != null ? org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(quitMessage) : null; // Paper } @NotNull diff --git a/src/main/java/org/bukkit/event/server/BroadcastMessageEvent.java b/src/main/java/org/bukkit/event/server/BroadcastMessageEvent.java index 03bfca9d368bbe4b7c1353d52c883e756bf69bda..4f8c85222c7bd33217c7db0ff5f47bf397f8f3e5 100644 --- a/src/main/java/org/bukkit/event/server/BroadcastMessageEvent.java +++ b/src/main/java/org/bukkit/event/server/BroadcastMessageEvent.java @@ -18,7 +18,7 @@ import org.jetbrains.annotations.NotNull; public class BroadcastMessageEvent extends ServerEvent implements Cancellable { private static final HandlerList handlers = new HandlerList(); - private String message; + private net.kyori.adventure.text.Component message; // Paper private final Set recipients; private boolean cancelled = false; @@ -27,29 +27,66 @@ public class BroadcastMessageEvent extends ServerEvent implements Cancellable { this(false, message, recipients); } + @Deprecated // Paper public BroadcastMessageEvent(boolean isAsync, @NotNull String message, @NotNull Set recipients) { + // Paper start + super(isAsync); + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); + this.recipients = recipients; + } + + @Deprecated + public BroadcastMessageEvent(@NotNull net.kyori.adventure.text.Component message, @NotNull Set recipients) { + this(false, message, recipients); + } + + public BroadcastMessageEvent(boolean isAsync, @NotNull net.kyori.adventure.text.Component message, @NotNull Set recipients) { + // Paper end super(isAsync); this.message = message; this.recipients = recipients; } + // Paper start + /** + * Get the broadcast message. + * + * @return Message to broadcast + */ + public @NotNull net.kyori.adventure.text.Component message() { + return this.message; + } + + /** + * Set the broadcast message. + * + * @param message New message to broadcast + */ + public void message(@NotNull net.kyori.adventure.text.Component message) { + this.message = message; + } + // Paper end /** * Get the message to broadcast. * * @return Message to broadcast + * @deprecated in favour of {@link #message()} */ @NotNull + @Deprecated // Paper public String getMessage() { - return message; + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.message); // Paper } /** * Set the message to broadcast. * * @param message New message to broadcast + * @deprecated in favour of {@link #message(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setMessage(@NotNull String message) { - this.message = message; + this.message = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(message); // Paper } /** diff --git a/src/main/java/org/bukkit/event/server/ServerListPingEvent.java b/src/main/java/org/bukkit/event/server/ServerListPingEvent.java index 7a2a58bac8e721c3f0c64f69f77be07a51f76d58..ede5a41bc071a9c9cea369b227b37a50222f295d 100644 --- a/src/main/java/org/bukkit/event/server/ServerListPingEvent.java +++ b/src/main/java/org/bukkit/event/server/ServerListPingEvent.java @@ -17,15 +17,16 @@ public class ServerListPingEvent extends ServerEvent implements Iterable private static final int MAGIC_PLAYER_COUNT = Integer.MIN_VALUE; private static final HandlerList handlers = new HandlerList(); private final InetAddress address; - private String motd; + private net.kyori.adventure.text.Component motd; // Paper private final int numPlayers; private int maxPlayers; + @Deprecated // Paper public ServerListPingEvent(@NotNull final InetAddress address, @NotNull final String motd, final int numPlayers, final int maxPlayers) { super(true); Validate.isTrue(numPlayers >= 0, "Cannot have negative number of players online", numPlayers); this.address = address; - this.motd = motd; + this.motd = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(motd); // Paper this.numPlayers = numPlayers; this.maxPlayers = maxPlayers; } @@ -38,14 +39,58 @@ public class ServerListPingEvent extends ServerEvent implements Iterable * @param address the address of the pinger * @param motd the message of the day * @param maxPlayers the max number of players + * @deprecated in favour of {@link #ServerListPingEvent(java.net.InetAddress, net.kyori.adventure.text.Component, int)} */ + @Deprecated // Paper protected ServerListPingEvent(@NotNull final InetAddress address, @NotNull final String motd, final int maxPlayers) { + super(true); + this.numPlayers = MAGIC_PLAYER_COUNT; + this.address = address; + this.motd = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(motd); // Paper + this.maxPlayers = maxPlayers; + } + // Paper start + public ServerListPingEvent(@NotNull final InetAddress address, @NotNull final net.kyori.adventure.text.Component motd, final int numPlayers, final int maxPlayers) { + super(true); + Validate.isTrue(numPlayers >= 0, "Cannot have negative number of players online", numPlayers); + this.address = address; + this.motd = motd; + this.numPlayers = numPlayers; + this.maxPlayers = maxPlayers; + } + /** + * This constructor is intended for implementations that provide the + * {@link #iterator()} method, thus provided the {@link #getNumPlayers()} + * count. + * + * @param address the address of the pinger + * @param motd the message of the day + * @param maxPlayers the max number of players + */ + protected ServerListPingEvent(@NotNull final InetAddress address, @NotNull final net.kyori.adventure.text.Component motd, final int maxPlayers) { super(true); this.numPlayers = MAGIC_PLAYER_COUNT; this.address = address; this.motd = motd; this.maxPlayers = maxPlayers; } + /** + * Get the message of the day message. + * + * @return the message of the day + */ + public @NotNull net.kyori.adventure.text.Component motd() { + return motd; + } + /** + * Change the message of the day message. + * + * @param motd the message of the day + */ + public void motd(@NotNull net.kyori.adventure.text.Component motd) { + this.motd = motd; + } + // Paper end /** * Get the address the ping is coming from. @@ -61,19 +106,23 @@ public class ServerListPingEvent extends ServerEvent implements Iterable * Get the message of the day message. * * @return the message of the day + * @deprecated in favour of {@link #motd()} */ @NotNull + @Deprecated // Paper public String getMotd() { - return motd; + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.motd); // Paper } /** * Change the message of the day message. * * @param motd the message of the day + * @deprecated in favour of {@link #motd(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setMotd(@NotNull String motd) { - this.motd = motd; + this.motd = org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(motd); // Paper } /** diff --git a/src/main/java/org/bukkit/inventory/InventoryView.java b/src/main/java/org/bukkit/inventory/InventoryView.java index 14346d83bc99581b18e53d19af03708c0bf22cf7..664de64b020cf9090a2fbee0afe2bfaf150adc3c 100644 --- a/src/main/java/org/bukkit/inventory/InventoryView.java +++ b/src/main/java/org/bukkit/inventory/InventoryView.java @@ -446,11 +446,25 @@ public abstract class InventoryView { return getPlayer().setWindowProperty(prop, value); } + // Paper start /** * Get the title of this inventory window. * * @return The title. */ @NotNull + public /*abstract*/ net.kyori.adventure.text.Component title() { + return org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(this.getTitle()); + } + // Paper end + + /** + * Get the title of this inventory window. + * + * @return The title. + * @deprecated in favour of {@link #title()} + */ + @Deprecated // Paper + @NotNull public abstract String getTitle(); } diff --git a/src/main/java/org/bukkit/inventory/ItemFactory.java b/src/main/java/org/bukkit/inventory/ItemFactory.java index 4ff149fd98895dd8ba45939a37c223b1f8d7281f..6cc4bad2ecd19f44a680ff03cbfb99d48ea5c337 100644 --- a/src/main/java/org/bukkit/inventory/ItemFactory.java +++ b/src/main/java/org/bukkit/inventory/ItemFactory.java @@ -141,4 +141,24 @@ public interface ItemFactory { @Deprecated @NotNull Material updateMaterial(@NotNull final ItemMeta meta, @NotNull final Material material) throws IllegalArgumentException; + + // Paper start + /** + * Creates a hover event for the given item. + * + * @param item The item + * @return A hover event + */ + @NotNull + net.kyori.adventure.text.event.HoverEvent asHoverEvent(final @NotNull ItemStack item, final @NotNull java.util.function.UnaryOperator op); + + /** + * Get the formatted display name of the {@link ItemStack}. + * + * @param itemStack the {@link ItemStack} + * @return display name of the {@link ItemStack} + */ + @NotNull + net.kyori.adventure.text.Component displayName(@NotNull ItemStack itemStack); + // Paper end } diff --git a/src/main/java/org/bukkit/inventory/ItemStack.java b/src/main/java/org/bukkit/inventory/ItemStack.java index f70a6a22b85ff0da76e67e9b223ad4e0b020b5c4..a15abec467bac70116a6fc21a300d4930b909f15 100644 --- a/src/main/java/org/bukkit/inventory/ItemStack.java +++ b/src/main/java/org/bukkit/inventory/ItemStack.java @@ -22,7 +22,7 @@ import org.jetbrains.annotations.Nullable; * use this class to encapsulate Materials for which {@link Material#isItem()} * returns false. */ -public class ItemStack implements Cloneable, ConfigurationSerializable { +public class ItemStack implements Cloneable, ConfigurationSerializable, net.kyori.adventure.text.event.HoverEventSource { // Paper private Material type = Material.AIR; private int amount = 0; private MaterialData data = null; @@ -595,4 +595,21 @@ public class ItemStack implements Cloneable, ConfigurationSerializable { return true; } + + // Paper start + @NotNull + @Override + public net.kyori.adventure.text.event.HoverEvent asHoverEvent(final @NotNull java.util.function.UnaryOperator op) { + return org.bukkit.Bukkit.getServer().getItemFactory().asHoverEvent(this, op); + } + + /** + * Get the formatted display name of the {@link ItemStack}. + * + * @return display name of the {@link ItemStack} + */ + public @NotNull net.kyori.adventure.text.Component displayName() { + return Bukkit.getServer().getItemFactory().displayName(this); + } + // Paper end } diff --git a/src/main/java/org/bukkit/inventory/meta/BookMeta.java b/src/main/java/org/bukkit/inventory/meta/BookMeta.java index 94852d50e88d0594b84b581cd627174043629995..fdbd32c4bb6428f2dde484ecb53acdaaba2a0d18 100644 --- a/src/main/java/org/bukkit/inventory/meta/BookMeta.java +++ b/src/main/java/org/bukkit/inventory/meta/BookMeta.java @@ -1,8 +1,11 @@ package org.bukkit.inventory.meta; import java.util.List; + +import net.kyori.adventure.inventory.Book; import net.md_5.bungee.api.chat.BaseComponent; import org.bukkit.Material; +import org.checkerframework.checker.nullness.qual.NonNull; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; @@ -10,7 +13,7 @@ import org.jetbrains.annotations.Nullable; * Represents a book ({@link Material#WRITABLE_BOOK} or {@link * Material#WRITTEN_BOOK}) that can have a title, an author, and pages. */ -public interface BookMeta extends ItemMeta { +public interface BookMeta extends ItemMeta, net.kyori.adventure.inventory.Book { // Paper /** * Represents the generation (or level of copying) of a written book @@ -119,6 +122,116 @@ public interface BookMeta extends ItemMeta { */ boolean hasPages(); + // Paper start + /** + * Gets the title of the book. + *

+ * Plugins should check that hasTitle() returns true before calling this + * method. + * + * @return the title of the book + */ + @Nullable + @Override + net.kyori.adventure.text.Component title(); + + /** + * Sets the title of the book. + *

+ * Limited to 32 characters. Removes title when given null. + * + * @param title the title to set + * @return true if the title was successfully set + */ + @NotNull + @Override + BookMeta title(@Nullable net.kyori.adventure.text.Component title); + + /** + * Gets the author of the book. + *

+ * Plugins should check that hasAuthor() returns true before calling this + * method. + * + * @return the author of the book + */ + @Nullable + @Override + net.kyori.adventure.text.Component author(); + + /** + * Sets the author of the book. Removes author when given null. + * + * @param author the author to set + */ + @NotNull + @Override + BookMeta author(@Nullable net.kyori.adventure.text.Component author); + /** + * Gets the specified page in the book. The page must exist. + *

+ * Pages are 1-indexed. + * + * @param page the page number to get, in range [1, getPageCount()] + * @return the page from the book + */ + @NotNull net.kyori.adventure.text.Component page(int page); + + /** + * Sets the specified page in the book. Pages of the book must be + * contiguous. + *

+ * The data can be up to 256 characters in length, additional characters + * are truncated. + *

+ * Pages are 1-indexed. + * + * @param page the page number to set, in range [1, getPageCount()] + * @param data the data to set for that page + */ + void page(int page, @NotNull net.kyori.adventure.text.Component data); + + /** + * Adds new pages to the end of the book. Up to a maximum of 50 pages with + * 256 characters per page. + * + * @param pages A list of strings, each being a page + */ + void addPages(@NotNull net.kyori.adventure.text.Component... pages); + + interface BookMetaBuilder extends Builder { + + @NotNull + @Override + BookMetaBuilder title(@Nullable net.kyori.adventure.text.Component title); + + @NotNull + @Override + BookMetaBuilder author(@Nullable net.kyori.adventure.text.Component author); + + @NotNull + @Override + BookMetaBuilder addPage(@NotNull net.kyori.adventure.text.Component page); + + @NotNull + @Override + BookMetaBuilder pages(@NotNull net.kyori.adventure.text.Component... pages); + + @NotNull + @Override + BookMetaBuilder pages(@NotNull java.util.Collection pages); + + @NotNull + @Override + BookMeta build(); + } + + @Override + @NonNull + BookMetaBuilder toBuilder(); + + // Paper end + /** * Gets the specified page in the book. The given page must exist. *

@@ -126,8 +239,10 @@ public interface BookMeta extends ItemMeta { * * @param page the page number to get, in range [1, getPageCount()] * @return the page from the book + * @deprecated in favour of {@link #page(int)} */ @NotNull + @Deprecated // Paper String getPage(int page); /** @@ -141,15 +256,19 @@ public interface BookMeta extends ItemMeta { * * @param page the page number to set, in range [1, getPageCount()] * @param data the data to set for that page + * @deprecated in favour of {@link #page(int, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper void setPage(int page, @NotNull String data); /** * Gets all the pages in the book. * * @return list of all the pages in the book + * @deprecated in favour of {@link #pages()} */ @NotNull + @Deprecated // Paper List getPages(); /** @@ -157,7 +276,9 @@ public interface BookMeta extends ItemMeta { * pages. Maximum 100 pages with 256 characters per page. * * @param pages A list of pages to set the book to use + * @deprecated in favour of {@link #pages(List)} */ + @Deprecated // Paper void setPages(@NotNull List pages); /** @@ -165,7 +286,9 @@ public interface BookMeta extends ItemMeta { * pages. Maximum 50 pages with 256 characters per page. * * @param pages A list of strings, each being a page + * @deprecated in favour of {@link #pages(net.kyori.adventure.text.Component...)} */ + @Deprecated // Paper void setPages(@NotNull String... pages); /** @@ -173,7 +296,9 @@ public interface BookMeta extends ItemMeta { * 256 characters per page. * * @param pages A list of strings, each being a page + * @deprecated in favour of {@link #addPages(net.kyori.adventure.text.Component...)} */ + @Deprecated // Paper void addPage(@NotNull String... pages); /** @@ -195,8 +320,10 @@ public interface BookMeta extends ItemMeta { * * @param page the page number to get * @return the page from the book + * @deprecated in favour of {@link #page(int)} */ @NotNull + @Deprecated // Paper public BaseComponent[] getPage(int page) { throw new UnsupportedOperationException("Not supported yet."); } @@ -210,7 +337,9 @@ public interface BookMeta extends ItemMeta { * * @param page the page number to set * @param data the data to set for that page + * @deprecated in favour of {@link #page(int, net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setPage(int page, @Nullable BaseComponent... data) { throw new UnsupportedOperationException("Not supported yet."); } @@ -219,8 +348,10 @@ public interface BookMeta extends ItemMeta { * Gets all the pages in the book. * * @return list of all the pages in the book + * @deprecated in favour of {@link #pages()} */ @NotNull + @Deprecated // Paper public List getPages() { throw new UnsupportedOperationException("Not supported yet."); } @@ -230,7 +361,9 @@ public interface BookMeta extends ItemMeta { * pages. Maximum 50 pages with 256 characters per page. * * @param pages A list of pages to set the book to use + * @deprecated in favour of {@link #pages(java.util.List)} */ + @Deprecated // Paper public void setPages(@NotNull List pages) { throw new UnsupportedOperationException("Not supported yet."); } @@ -240,7 +373,9 @@ public interface BookMeta extends ItemMeta { * pages. Maximum 50 pages with 256 characters per page. * * @param pages A list of component arrays, each being a page + * @deprecated in favour of {@link #pages(net.kyori.adventure.text.Component...)} */ + @Deprecated // Paper public void setPages(@NotNull BaseComponent[]... pages) { throw new UnsupportedOperationException("Not supported yet."); } @@ -250,7 +385,9 @@ public interface BookMeta extends ItemMeta { * with 256 characters per page. * * @param pages A list of component arrays, each being a page + * @deprecated in favour of {@link #addPages(net.kyori.adventure.text.Component...)} */ + @Deprecated // Paper public void addPage(@NotNull BaseComponent[]... pages) { throw new UnsupportedOperationException("Not supported yet."); } diff --git a/src/main/java/org/bukkit/inventory/meta/ItemMeta.java b/src/main/java/org/bukkit/inventory/meta/ItemMeta.java index f2e9f2753ec92aa4a3e3f06ca6053bd70b9091d7..1c362636c56db0e6c118171ba367c43c4f7cff33 100644 --- a/src/main/java/org/bukkit/inventory/meta/ItemMeta.java +++ b/src/main/java/org/bukkit/inventory/meta/ItemMeta.java @@ -31,6 +31,24 @@ public interface ItemMeta extends Cloneable, ConfigurationSerializable, Persiste */ boolean hasDisplayName(); + // Paper start + /** + * Gets the display name. + * + *

Plugins should check that {@link #hasDisplayName()} returns true before calling this method.

+ * + * @return the display name + */ + @Nullable net.kyori.adventure.text.Component displayName(); + + /** + * Sets the display name. + * + * @param displayName the display name to set + */ + void displayName(final @Nullable net.kyori.adventure.text.Component displayName); + // Paper end + /** * Gets the display name that is set. *

@@ -38,7 +56,9 @@ public interface ItemMeta extends Cloneable, ConfigurationSerializable, Persiste * before calling this method. * * @return the display name that is set + * @deprecated in favour of {@link #displayName()} */ + @Deprecated // Paper @NotNull String getDisplayName(); @@ -46,7 +66,9 @@ public interface ItemMeta extends Cloneable, ConfigurationSerializable, Persiste * Sets the display name. * * @param name the name to set + * @deprecated in favour of {@link #displayName(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper void setDisplayName(@Nullable String name); /** @@ -81,6 +103,24 @@ public interface ItemMeta extends Cloneable, ConfigurationSerializable, Persiste */ boolean hasLore(); + // Paper start + /** + * Gets the lore. + * + *

Plugins should check that {@link #hasLore()} returns true before calling this method.

+ * + * @return the display name + */ + @Nullable List lore(); + + /** + * Sets the lore. + * + * @param lore the lore to set + */ + void lore(final @Nullable List lore); + // Paper end + /** * Gets the lore that is set. *

@@ -88,7 +128,9 @@ public interface ItemMeta extends Cloneable, ConfigurationSerializable, Persiste * calling this method. * * @return a list of lore that is set + * @deprecated in favour of {@link #lore()} */ + @Deprecated // Paper @Nullable List getLore(); @@ -97,7 +139,9 @@ public interface ItemMeta extends Cloneable, ConfigurationSerializable, Persiste * Removes lore when given null. * * @param lore the lore that will be set + * @deprecated in favour of {@link #lore(List)} */ + @Deprecated // Paper void setLore(@Nullable List lore); /** diff --git a/src/main/java/org/bukkit/map/MapCursor.java b/src/main/java/org/bukkit/map/MapCursor.java index 83354b2a38b6261b172b91c1008dcf3313cc4a8f..ed0bc2024a0bb85837e25f75ae89d1fe257b2e60 100644 --- a/src/main/java/org/bukkit/map/MapCursor.java +++ b/src/main/java/org/bukkit/map/MapCursor.java @@ -10,7 +10,7 @@ public final class MapCursor { private byte x, y; private byte direction, type; private boolean visible; - private String caption; + private net.kyori.adventure.text.Component caption; // Paper /** * Initialize the map cursor. @@ -24,7 +24,7 @@ public final class MapCursor { */ @Deprecated public MapCursor(byte x, byte y, byte direction, byte type, boolean visible) { - this(x, y, direction, type, visible, null); + this(x, y, direction, type, visible, (String) null); // Paper } /** @@ -37,7 +37,7 @@ public final class MapCursor { * @param visible Whether the cursor is visible by default. */ public MapCursor(byte x, byte y, byte direction, @NotNull Type type, boolean visible) { - this(x, y, direction, type, visible, null); + this(x, y, direction, type, visible, (String) null); // Paper } /** @@ -49,7 +49,7 @@ public final class MapCursor { * @param type The type (color/style) of the map cursor. * @param visible Whether the cursor is visible by default. * @param caption cursor caption - * @deprecated Magic value + * @deprecated Magic value. Use {@link #MapCursor(byte, byte, byte, byte, boolean, net.kyori.adventure.text.Component)} */ @Deprecated public MapCursor(byte x, byte y, byte direction, byte type, boolean visible, @Nullable String caption) { @@ -58,8 +58,42 @@ public final class MapCursor { setDirection(direction); setRawType(type); this.visible = visible; - this.caption = caption; + this.caption = caption == null ? null : org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(caption); // Paper } + // Paper start + /** + * Initialize the map cursor. + * + * @param x The x coordinate, from -128 to 127. + * @param y The y coordinate, from -128 to 127. + * @param direction The facing of the cursor, from 0 to 15. + * @param type The type (color/style) of the map cursor. + * @param visible Whether the cursor is visible by default. + * @param caption cursor caption + * @deprecated Magic value + */ + @Deprecated + public MapCursor(byte x, byte y, byte direction, byte type, boolean visible, @Nullable net.kyori.adventure.text.Component caption) { + this.x = x; this.y = y; this.visible = visible; this.caption = caption; + setDirection(direction); + setRawType(type); + } + /** + * Initialize the map cursor. + * + * @param x The x coordinate, from -128 to 127. + * @param y The y coordinate, from -128 to 127. + * @param direction The facing of the cursor, from 0 to 15. + * @param type The type (color/style) of the map cursor. + * @param visible Whether the cursor is visible by default. + * @param caption cursor caption + */ + public MapCursor(byte x, byte y, byte direction, @NotNull Type type, boolean visible, @Nullable net.kyori.adventure.text.Component caption) { + this.x = x; this.y = y; this.visible = visible; this.caption = caption; + setDirection(direction); + setType(type); + } + // Paper end /** * Initialize the map cursor. @@ -77,7 +111,7 @@ public final class MapCursor { setDirection(direction); setType(type); this.visible = visible; - this.caption = caption; + this.caption = caption == null ? null : org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(caption); // Paper } /** @@ -200,23 +234,45 @@ public final class MapCursor { this.visible = visible; } + // Paper start + /** + * Gets the caption on this cursor. + * + * @return caption + */ + public @Nullable net.kyori.adventure.text.Component caption() { + return this.caption; + } + /** + * Sets the caption on this cursor. + * + * @param caption new caption + */ + public void caption(@Nullable net.kyori.adventure.text.Component caption) { + this.caption = caption; + } + // Paper end /** * Gets the caption on this cursor. * * @return caption + * @deprecated in favour of {@link #caption()} */ @Nullable + @Deprecated // Paper public String getCaption() { - return caption; + return this.caption == null ? null : org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().serialize(this.caption); // Paper } /** * Sets the caption on this cursor. * * @param caption new caption + * @deprecated in favour of {@link #caption(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper public void setCaption(@Nullable String caption) { - this.caption = caption; + this.caption = caption == null ? null : org.bukkit.Bukkit.getUnsafe().legacyComponentSerializer().deserialize(caption); // Paper } /** diff --git a/src/main/java/org/bukkit/map/MapCursorCollection.java b/src/main/java/org/bukkit/map/MapCursorCollection.java index 4dba721aefe4fc6699b3b4bfa7ecb0b19c2a2a1a..01dec2c877df58c9dc22445e8b1f9ce2e53066da 100644 --- a/src/main/java/org/bukkit/map/MapCursorCollection.java +++ b/src/main/java/org/bukkit/map/MapCursorCollection.java @@ -117,4 +117,22 @@ public final class MapCursorCollection { public MapCursor addCursor(int x, int y, byte direction, byte type, boolean visible, @Nullable String caption) { return addCursor(new MapCursor((byte) x, (byte) y, direction, type, visible, caption)); } + // Paper start + /** + * Add a cursor to the collection. + * + * @param x The x coordinate, from -128 to 127. + * @param y The y coordinate, from -128 to 127. + * @param direction The facing of the cursor, from 0 to 15. + * @param type The type (color/style) of the map cursor. + * @param visible Whether the cursor is visible. + * @param caption banner caption + * @return The newly added MapCursor. + * @deprecated Magic value + */ + @Deprecated + public @NotNull MapCursor addCursor(int x, int y, byte direction, byte type, boolean visible, @Nullable net.kyori.adventure.text.Component caption) { + return addCursor(new MapCursor((byte) x, (byte) y, direction, type, visible, caption)); + } + // Paper end } diff --git a/src/main/java/org/bukkit/scoreboard/Objective.java b/src/main/java/org/bukkit/scoreboard/Objective.java index f5cbf6df32ef169cf0f2266f7c6e9c4f771ccb7d..58bddb11fd534e7c33a4ffd7b72b055ba92c767a 100644 --- a/src/main/java/org/bukkit/scoreboard/Objective.java +++ b/src/main/java/org/bukkit/scoreboard/Objective.java @@ -19,14 +19,35 @@ public interface Objective { */ @NotNull String getName() throws IllegalStateException; + // Paper start + /** + * Gets the name displayed to players for this objective + * + * @return this objective's display name + * @throws IllegalStateException if this objective has been unregistered + */ + @NotNull net.kyori.adventure.text.Component displayName() throws IllegalStateException; + /** + * Sets the name displayed to players for this objective. + * + * @param displayName Display name to set + * @throws IllegalStateException if this objective has been unregistered + * @throws IllegalArgumentException if displayName is null + * @throws IllegalArgumentException if displayName is longer than 128 + * characters. + */ + void displayName(@Nullable net.kyori.adventure.text.Component displayName) throws IllegalStateException, IllegalArgumentException; + // Paper end /** * Gets the name displayed to players for this objective * * @return this objective's display name * @throws IllegalStateException if this objective has been unregistered + * @deprecated in favour of {@link #displayName()} */ @NotNull + @Deprecated // Paper String getDisplayName() throws IllegalStateException; /** @@ -37,7 +58,9 @@ public interface Objective { * @throws IllegalArgumentException if displayName is null * @throws IllegalArgumentException if displayName is longer than 128 * characters. + * @deprecated in favour of {@link #displayName(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper void setDisplayName(@NotNull String displayName) throws IllegalStateException, IllegalArgumentException; /** diff --git a/src/main/java/org/bukkit/scoreboard/Scoreboard.java b/src/main/java/org/bukkit/scoreboard/Scoreboard.java index 4bfaaea78c9b6aa5d392629aa943d26dbe6a7d4a..f09ff32cc3ffc16af379a378b1948991435393e8 100644 --- a/src/main/java/org/bukkit/scoreboard/Scoreboard.java +++ b/src/main/java/org/bukkit/scoreboard/Scoreboard.java @@ -27,6 +27,48 @@ public interface Scoreboard { @Deprecated @NotNull Objective registerNewObjective(@NotNull String name, @NotNull String criteria) throws IllegalArgumentException; + // Paper start + /** + * Registers an Objective on this Scoreboard + * + * @param name Name of the Objective + * @param criteria Criteria for the Objective + * @param displayName Name displayed to players for the Objective. + * @return The registered Objective + * @throws IllegalArgumentException if name is null + * @throws IllegalArgumentException if name is longer than 16 + * characters. + * @throws IllegalArgumentException if criteria is null + * @throws IllegalArgumentException if displayName is null + * @throws IllegalArgumentException if displayName is longer than 128 + * characters. + * @throws IllegalArgumentException if an objective by that name already + * exists + */ + @NotNull + Objective registerNewObjective(@NotNull String name, @NotNull String criteria, @Nullable net.kyori.adventure.text.Component displayName) throws IllegalArgumentException; + /** + * Registers an Objective on this Scoreboard + * + * @param name Name of the Objective + * @param criteria Criteria for the Objective + * @param displayName Name displayed to players for the Objective. + * @param renderType Manner of rendering the Objective + * @return The registered Objective + * @throws IllegalArgumentException if name is null + * @throws IllegalArgumentException if name is longer than 16 + * characters. + * @throws IllegalArgumentException if criteria is null + * @throws IllegalArgumentException if displayName is null + * @throws IllegalArgumentException if displayName is longer than 128 + * characters. + * @throws IllegalArgumentException if renderType is null + * @throws IllegalArgumentException if an objective by that name already + * exists + */ + @NotNull + Objective registerNewObjective(@NotNull String name, @NotNull String criteria, @Nullable net.kyori.adventure.text.Component displayName, @NotNull RenderType renderType) throws IllegalArgumentException; + // Paper end /** * Registers an Objective on this Scoreboard @@ -44,8 +86,10 @@ public interface Scoreboard { * characters. * @throws IllegalArgumentException if an objective by that name already * exists + * @deprecated in favour of {@link #registerNewObjective(String, String, net.kyori.adventure.text.Component)} */ @NotNull + @Deprecated // Paper Objective registerNewObjective(@NotNull String name, @NotNull String criteria, @NotNull String displayName) throws IllegalArgumentException; /** @@ -66,8 +110,10 @@ public interface Scoreboard { * @throws IllegalArgumentException if renderType is null * @throws IllegalArgumentException if an objective by that name already * exists + * @deprecated in favour of {@link #registerNewObjective(String, String, net.kyori.adventure.text.Component, RenderType)} */ @NotNull + @Deprecated // Paper Objective registerNewObjective(@NotNull String name, @NotNull String criteria, @NotNull String displayName, @NotNull RenderType renderType) throws IllegalArgumentException; /** diff --git a/src/main/java/org/bukkit/scoreboard/Team.java b/src/main/java/org/bukkit/scoreboard/Team.java index da01d2926cc8a2485a3349ac1ebb32cad20e287c..f0af10a5b9ad048be197ed5ec6c8ed2672eb3dd5 100644 --- a/src/main/java/org/bukkit/scoreboard/Team.java +++ b/src/main/java/org/bukkit/scoreboard/Team.java @@ -22,14 +22,95 @@ public interface Team { */ @NotNull String getName() throws IllegalStateException; + // Paper start + /** + * Gets the name displayed to entries for this team + * + * @return Team display name + * @throws IllegalStateException if this team has been unregistered + */ + @NotNull net.kyori.adventure.text.Component displayName() throws IllegalStateException; + + /** + * Sets the name displayed to entries for this team + * + * @param displayName New display name + * @throws IllegalArgumentException if displayName is longer than 128 + * characters. + * @throws IllegalStateException if this team has been unregistered + */ + void displayName(@Nullable net.kyori.adventure.text.Component displayName) throws IllegalStateException, IllegalArgumentException; + + /** + * Gets the prefix prepended to the display of entries on this team. + * + * @return Team prefix + * @throws IllegalStateException if this team has been unregistered + */ + @NotNull net.kyori.adventure.text.Component prefix() throws IllegalStateException; + + /** + * Sets the prefix prepended to the display of entries on this team. + * + * @param prefix New prefix + * @throws IllegalArgumentException if prefix is null + * @throws IllegalArgumentException if prefix is longer than 64 + * characters + * @throws IllegalStateException if this team has been unregistered + */ + void prefix(@Nullable net.kyori.adventure.text.Component prefix) throws IllegalStateException, IllegalArgumentException; + + /** + * Gets the suffix appended to the display of entries on this team. + * + * @return the team's current suffix + * @throws IllegalStateException if this team has been unregistered + */ + @NotNull net.kyori.adventure.text.Component suffix() throws IllegalStateException; + + /** + * Sets the suffix appended to the display of entries on this team. + * + * @param suffix the new suffix for this team. + * @throws IllegalArgumentException if suffix is null + * @throws IllegalArgumentException if suffix is longer than 64 + * characters + * @throws IllegalStateException if this team has been unregistered + */ + void suffix(@Nullable net.kyori.adventure.text.Component suffix) throws IllegalStateException, IllegalArgumentException; + + /** + * Gets the color of the team. + *
+ * This only sets the team outline, other occurrences of colors such as in + * names are handled by prefixes / suffixes. + * + * @return team color, defaults to {@link ChatColor#RESET} + * @throws IllegalStateException if this team has been unregistered + */ + @NotNull net.kyori.adventure.text.format.TextColor color() throws IllegalStateException; + + /** + * Sets the color of the team. + *
+ * This only sets the team outline, other occurrences of colors such as in + * names are handled by prefixes / suffixes. + * + * @param color new color, must be non-null. Use {@link ChatColor#RESET} for + * no color + */ + void color(@Nullable net.kyori.adventure.text.format.NamedTextColor color); + // Paper end /** * Gets the name displayed to entries for this team * * @return Team display name * @throws IllegalStateException if this team has been unregistered + * @deprecated in favour of {@link #displayName()} */ @NotNull + @Deprecated // Paper String getDisplayName() throws IllegalStateException; /** @@ -39,7 +120,9 @@ public interface Team { * @throws IllegalArgumentException if displayName is longer than 128 * characters. * @throws IllegalStateException if this team has been unregistered + * @deprecated in favour of {@link #displayName(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper void setDisplayName(@NotNull String displayName) throws IllegalStateException, IllegalArgumentException; /** @@ -47,8 +130,10 @@ public interface Team { * * @return Team prefix * @throws IllegalStateException if this team has been unregistered + * @deprecated in favour of {@link #prefix()} */ @NotNull + @Deprecated // Paper String getPrefix() throws IllegalStateException; /** @@ -59,7 +144,9 @@ public interface Team { * @throws IllegalArgumentException if prefix is longer than 64 * characters * @throws IllegalStateException if this team has been unregistered + * @deprecated in favour of {@link #prefix(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper void setPrefix(@NotNull String prefix) throws IllegalStateException, IllegalArgumentException; /** @@ -67,8 +154,10 @@ public interface Team { * * @return the team's current suffix * @throws IllegalStateException if this team has been unregistered + * @deprecated in favour of {@link #suffix()} */ @NotNull + @Deprecated // Paper String getSuffix() throws IllegalStateException; /** @@ -79,7 +168,9 @@ public interface Team { * @throws IllegalArgumentException if suffix is longer than 64 * characters * @throws IllegalStateException if this team has been unregistered + * @deprecated in favour of {@link #suffix(net.kyori.adventure.text.Component)} */ + @Deprecated // Paper void setSuffix(@NotNull String suffix) throws IllegalStateException, IllegalArgumentException; /** @@ -90,8 +181,10 @@ public interface Team { * * @return team color, defaults to {@link ChatColor#RESET} * @throws IllegalStateException if this team has been unregistered + * @deprecated in favour of {@link #color()} */ @NotNull + @Deprecated // Paper ChatColor getColor() throws IllegalStateException; /** @@ -102,7 +195,9 @@ public interface Team { * * @param color new color, must be non-null. Use {@link ChatColor#RESET} for * no color + * @deprecated in favour of {@link #color(net.kyori.adventure.text.format.NamedTextColor)} */ + @Deprecated // Paper void setColor(@NotNull ChatColor color); /** diff --git a/src/test/java/org/bukkit/AnnotationTest.java b/src/test/java/org/bukkit/AnnotationTest.java index 0c7377247ad9251c9e498039511e7220370aba2d..c62919f18f318fec15a6c364d8b6d562c2b04762 100644 --- a/src/test/java/org/bukkit/AnnotationTest.java +++ b/src/test/java/org/bukkit/AnnotationTest.java @@ -26,6 +26,12 @@ import org.objectweb.asm.tree.ParameterNode; public class AnnotationTest { private static final String[] ACCEPTED_ANNOTATIONS = { + // Paper start + "Lorg/checkerframework/checker/nullness/qual/Nullable;", + "Lorg/checkerframework/checker/nullness/qual/NonNull;", + "Lorg/checkerframework/checker/nullness/qual/PolyNull;", + "Lorg/checkerframework/checker/nullness/qual/MonotonicNonNull;", + // Paper end "Lorg/jetbrains/annotations/Nullable;", "Lorg/jetbrains/annotations/NotNull;", "Lorg/jetbrains/annotations/Contract;", @@ -66,7 +72,7 @@ public class AnnotationTest { continue; } - if (mustBeAnnotated(Type.getReturnType(method.desc)) && !isWellAnnotated(method.invisibleAnnotations)) { + if (mustBeAnnotated(Type.getReturnType(method.desc)) && !isWellAnnotated(method.invisibleAnnotations) && !isWellAnnotated(method.visibleTypeAnnotations)) { // Paper - also check visible type annotations warn(errors, clazz, method, "return value"); } @@ -174,7 +180,7 @@ public class AnnotationTest { return true; } - private static boolean isWellAnnotated(@Nullable List annotations) { + private static boolean isWellAnnotated(@Nullable List annotations) { // Paper - allow children of AnnotationNode if (annotations == null) { return false; }