From 972da0e5aea1866c7d3a672f5d29d909ed0ccf81 Mon Sep 17 00:00:00 2001 From: themode Date: Tue, 27 Oct 2020 01:53:56 +0100 Subject: [PATCH] Added warning for ConnectionManager#addPlayerInitialization to prevent calling too early code --- .../java/net/minestom/server/network/ConnectionManager.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/main/java/net/minestom/server/network/ConnectionManager.java b/src/main/java/net/minestom/server/network/ConnectionManager.java index c2b1fc811..1b8015b36 100644 --- a/src/main/java/net/minestom/server/network/ConnectionManager.java +++ b/src/main/java/net/minestom/server/network/ConnectionManager.java @@ -216,6 +216,11 @@ public final class ConnectionManager { /** * Adds a new player initialization consumer. Those are called when a {@link Player} join, * mainly to add event callbacks to the player. + *

+ * This callback should be exclusively used to add event listeners since it is called directly after a + * player join (before any chunk is sent) and the client behavior can therefore be unpredictable. + * You can add your "init" code in {@link net.minestom.server.event.player.PlayerLoginEvent} + * or even {@link net.minestom.server.event.player.PlayerPreLoginEvent}. * * @param playerInitialization the {@link Player} initialization consumer */