From 06663bcafd8ac19f4072b813fe091e5dbbf4d3de Mon Sep 17 00:00:00 2001 From: chris Date: Mon, 6 Nov 2023 16:17:41 +0100 Subject: [PATCH] Fix: Don't mistakingly assume that Geyser-Velocity initialized (#4276) * Fix: ListenerBoundEvent could set `INITIALIZED` to true, even if we did not initialize yet - move that to postStartup() instead. * only assign INITIALIZED once, and only after the correct listenertype bound. --- .../geyser/platform/velocity/GeyserVelocityPlugin.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bootstrap/velocity/src/main/java/org/geysermc/geyser/platform/velocity/GeyserVelocityPlugin.java b/bootstrap/velocity/src/main/java/org/geysermc/geyser/platform/velocity/GeyserVelocityPlugin.java index 93b57d712..d7c6c588e 100644 --- a/bootstrap/velocity/src/main/java/org/geysermc/geyser/platform/velocity/GeyserVelocityPlugin.java +++ b/bootstrap/velocity/src/main/java/org/geysermc/geyser/platform/velocity/GeyserVelocityPlugin.java @@ -206,9 +206,9 @@ public class GeyserVelocityPlugin implements GeyserBootstrap { // After this bound, we know that the channel initializer cannot change without it being ineffective for Velocity, too geyserInjector.initializeLocalChannel(this); } - } - INITIALIZED = true; + INITIALIZED = true; + } } @Override