From c5fa3f0d4cd2698283dcc5b7c7c39940d0fe69ff Mon Sep 17 00:00:00 2001 From: Alvinn8 <42838560+Alvinn8@users.noreply.github.com> Date: Tue, 11 May 2021 02:56:36 +0200 Subject: [PATCH] Add Adventure message to PlayerAdvancementDoneEvent (#5030) * Add Adventure message to PlayerAdvancementDoneEvent * Add obfuscation helpers * Renumber patches --- ...essage-to-PlayerAdvancementDoneEvent.patch | 71 +++++++++++++++ ...essage-to-PlayerAdvancementDoneEvent.patch | 89 +++++++++++++++++++ 2 files changed, 160 insertions(+) create mode 100644 Spigot-API-Patches/0296-Add-Adventure-message-to-PlayerAdvancementDoneEvent.patch create mode 100644 Spigot-Server-Patches/0725-Add-Adventure-message-to-PlayerAdvancementDoneEvent.patch diff --git a/Spigot-API-Patches/0296-Add-Adventure-message-to-PlayerAdvancementDoneEvent.patch b/Spigot-API-Patches/0296-Add-Adventure-message-to-PlayerAdvancementDoneEvent.patch new file mode 100644 index 0000000000..16af3a7728 --- /dev/null +++ b/Spigot-API-Patches/0296-Add-Adventure-message-to-PlayerAdvancementDoneEvent.patch @@ -0,0 +1,71 @@ +From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 +From: Alvinn8 <42838560+Alvinn8@users.noreply.github.com> +Date: Fri, 8 Jan 2021 20:29:58 +0100 +Subject: [PATCH] Add Adventure message to PlayerAdvancementDoneEvent + + +diff --git a/src/main/java/org/bukkit/event/player/PlayerAdvancementDoneEvent.java b/src/main/java/org/bukkit/event/player/PlayerAdvancementDoneEvent.java +index 21ff095afb434d15babcdbe85fa0b94cbb7e75a7..6f618bb140a901c5c1cd08210978a6a651677ba4 100644 +--- a/src/main/java/org/bukkit/event/player/PlayerAdvancementDoneEvent.java ++++ b/src/main/java/org/bukkit/event/player/PlayerAdvancementDoneEvent.java +@@ -4,6 +4,10 @@ import org.bukkit.advancement.Advancement; + import org.bukkit.entity.Player; + import org.bukkit.event.HandlerList; + import org.jetbrains.annotations.NotNull; ++// Paper start ++import org.jetbrains.annotations.Nullable; ++import net.kyori.adventure.text.Component; ++// Paper end + + /** + * Called when a player has completed all criteria in an advancement. +@@ -13,10 +17,17 @@ public class PlayerAdvancementDoneEvent extends PlayerEvent { + private static final HandlerList handlers = new HandlerList(); + // + private final Advancement advancement; ++ private Component message; // Paper - Add Adventure message + + public PlayerAdvancementDoneEvent(@NotNull Player who, @NotNull Advancement advancement) { ++ // Paper start - Add Adventure message ++ this(who, advancement, null); ++ } ++ public PlayerAdvancementDoneEvent(@NotNull Player who, @NotNull Advancement advancement, @Nullable Component message) { ++ // Paper end + super(who); + this.advancement = advancement; ++ this.message = message; // Paper - Add Adventure message + } + + /** +@@ -29,6 +40,31 @@ public class PlayerAdvancementDoneEvent extends PlayerEvent { + return advancement; + } + ++ // Paper start - Add Adventure message ++ /** ++ * Gets the message to send to all online players. ++ *
++ * Will be null if the advancement does not announce to chat, for example if ++ * it is a recipe unlock or a root advancement. ++ * ++ * @return The announcement message, or null ++ */ ++ @Nullable ++ public Component message() { ++ return this.message; ++ } ++ ++ /** ++ * Sets the message to send to all online players. ++ *
++ * If set to null the message will not be sent. ++ * ++ * @param message The new message ++ */ ++ public void message(@Nullable Component message) { ++ this.message = message; ++ } ++ // Paper end + @NotNull + @Override + public HandlerList getHandlers() { diff --git a/Spigot-Server-Patches/0725-Add-Adventure-message-to-PlayerAdvancementDoneEvent.patch b/Spigot-Server-Patches/0725-Add-Adventure-message-to-PlayerAdvancementDoneEvent.patch new file mode 100644 index 0000000000..07309399fa --- /dev/null +++ b/Spigot-Server-Patches/0725-Add-Adventure-message-to-PlayerAdvancementDoneEvent.patch @@ -0,0 +1,89 @@ +From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 +From: Alvinn8 <42838560+Alvinn8@users.noreply.github.com> +Date: Fri, 8 Jan 2021 20:31:13 +0100 +Subject: [PATCH] Add Adventure message to PlayerAdvancementDoneEvent + + +diff --git a/src/main/java/net/minecraft/advancements/Advancement.java b/src/main/java/net/minecraft/advancements/Advancement.java +index e3d5d55a2fe5d86db621d3e0c9322f553386d731..2f3d9e5b849093027d3c2ef434494cd600f52a31 100644 +--- a/src/main/java/net/minecraft/advancements/Advancement.java ++++ b/src/main/java/net/minecraft/advancements/Advancement.java +@@ -76,6 +76,7 @@ public class Advancement { + return this.parent; + } + ++ public final @Nullable AdvancementDisplay getAdvancementDisplay() { return this.c(); } // Paper - OBFHELPER + @Nullable + public AdvancementDisplay c() { + return this.display; +@@ -125,6 +126,7 @@ public class Advancement { + return this.requirements; + } + ++ public final IChatBaseComponent getChatComponent() { return this.j(); } // Paper - OBFHELPER + public IChatBaseComponent j() { + return this.chatComponent; + } +diff --git a/src/main/java/net/minecraft/advancements/AdvancementDisplay.java b/src/main/java/net/minecraft/advancements/AdvancementDisplay.java +index 721d7fbab447117349994f710338ca616ab11067..adc6779e53e7b2ee04a80e2ea714e3378b8e6f39 100644 +--- a/src/main/java/net/minecraft/advancements/AdvancementDisplay.java ++++ b/src/main/java/net/minecraft/advancements/AdvancementDisplay.java +@@ -54,10 +54,12 @@ public class AdvancementDisplay { + return this.b; + } + ++ public final AdvancementFrameType getFrameType() { return this.e(); } // Paper - OBFHELPER + public AdvancementFrameType e() { + return this.e; + } + ++ public final boolean shouldAnnounceToChat() { return this.i(); } // Paper - OBFHELPER + public boolean i() { + return this.g; + } +diff --git a/src/main/java/net/minecraft/advancements/AdvancementFrameType.java b/src/main/java/net/minecraft/advancements/AdvancementFrameType.java +index f5d24950d317c78a971472821595fc1b44befc6e..32380346555e194227423999a79f1ebcbbe38d3b 100644 +--- a/src/main/java/net/minecraft/advancements/AdvancementFrameType.java ++++ b/src/main/java/net/minecraft/advancements/AdvancementFrameType.java +@@ -20,6 +20,7 @@ public enum AdvancementFrameType { + this.g = new ChatMessage("advancements.toast." + s); + } + ++ public final String getId() { return this.a(); } // Paper - OBFHELPER + public String a() { + return this.d; + } +diff --git a/src/main/java/net/minecraft/server/AdvancementDataPlayer.java b/src/main/java/net/minecraft/server/AdvancementDataPlayer.java +index 7a8a1960882e291c46301d07da3e1c5415516893..dfdbc028f68ced197ad179248ed3b1e9d70ba057 100644 +--- a/src/main/java/net/minecraft/server/AdvancementDataPlayer.java ++++ b/src/main/java/net/minecraft/server/AdvancementDataPlayer.java +@@ -51,6 +51,7 @@ import net.minecraft.util.datafix.DataFixTypes; + import net.minecraft.world.level.GameRules; + import org.apache.logging.log4j.LogManager; + import org.apache.logging.log4j.Logger; ++import io.papermc.paper.adventure.PaperAdventure; // Paper + + public class AdvancementDataPlayer { + +@@ -313,10 +314,18 @@ public class AdvancementDataPlayer { + this.j.add(advancement); + flag = true; + if (!flag1 && advancementprogress.isDone()) { +- this.player.world.getServer().getPluginManager().callEvent(new org.bukkit.event.player.PlayerAdvancementDoneEvent(this.player.getBukkitEntity(), advancement.bukkit)); // CraftBukkit ++ // Paper start - Add Adventure message to PlayerAdvancementDoneEvent ++ boolean announceToChat = advancement.getAdvancementDisplay() != null && advancement.getAdvancementDisplay().shouldAnnounceToChat(); ++ net.kyori.adventure.text.Component message = announceToChat ? PaperAdventure.asAdventure(new ChatMessage("chat.type.advancement." + advancement.getAdvancementDisplay().getFrameType().getId(), this.player.getScoreboardDisplayName(), advancement.getChatComponent())) : null; ++ org.bukkit.event.player.PlayerAdvancementDoneEvent event = new org.bukkit.event.player.PlayerAdvancementDoneEvent(this.player.getBukkitEntity(), advancement.bukkit, message); ++ this.player.world.getServer().getPluginManager().callEvent(event); ++ message = event.message(); ++ // Paper end + advancement.d().a(this.player); +- if (advancement.c() != null && advancement.c().i() && this.player.world.getGameRules().getBoolean(GameRules.ANNOUNCE_ADVANCEMENTS)) { +- this.e.sendMessage(new ChatMessage("chat.type.advancement." + advancement.c().e().a(), new Object[]{this.player.getScoreboardDisplayName(), advancement.j()}), ChatMessageType.SYSTEM, SystemUtils.b); ++ // Paper start - Add Adventure message to PlayerAdvancementDoneEvent ++ if (message != null && this.player.world.getGameRules().getBoolean(GameRules.ANNOUNCE_ADVANCEMENTS)) { ++ this.e.sendMessage(PaperAdventure.asVanilla(message), ChatMessageType.SYSTEM, SystemUtils.getNullUUID()); ++ // Paper end + } + } + }