Tidy up updating worldspawn

diff --git a/src/main/java/com/irtimaled/bbor/client/interop/ClientInterop.java b/src/main/java/com/irtimaled/bbor/client/interop/ClientInterop.java
index 59c569a23f64bc3bbd212c84caea98136d07d053..5fc0f973a5e165a4a8177266928c690c7fa0f05a 100644 (file)
--- a/src/main/java/com/irtimaled/bbor/client/interop/ClientInterop.java
+++ b/src/main/java/com/irtimaled/bbor/client/interop/ClientInterop.java
@@ -4,8 +4,10 @@ import com.irtimaled.bbor.client.PlayerCoords;
 import com.irtimaled.bbor.client.events.DisconnectedFromRemoteServer;
 import com.irtimaled.bbor.client.events.Render;
 import com.irtimaled.bbor.client.events.SeedCommandTyped;
+import com.irtimaled.bbor.client.events.UpdateWorldSpawnReceived;
 import com.irtimaled.bbor.common.EventBus;
 import net.minecraft.client.entity.EntityPlayerSP;
+import net.minecraft.util.math.BlockPos;
 
 public class ClientInterop {
     public static void disconnectedFromRemoteServer() {
@@ -39,4 +41,8 @@ public class ClientInterop {
             return null;
         }
     }
+
+    public static void updateWorldSpawnReceived(BlockPos blockPos) {
+        EventBus.publish(new UpdateWorldSpawnReceived(blockPos.getX(), blockPos.getZ()));
+    }
 }

diff --git a/src/main/java/com/irtimaled/bbor/mixin/network/play/server/MixinSPacketCustomPayload.java b/src/main/java/com/irtimaled/bbor/mixin/network/play/server/MixinSPacketCustomPayload.java
index adb72e7addef9db1e28d25e5d74e6677069248fb..a0ab1d64f72fd388222a030d9a4465a75a2b5982 100644 (file)
--- a/src/main/java/com/irtimaled/bbor/mixin/network/play/server/MixinSPacketCustomPayload.java
+++ b/src/main/java/com/irtimaled/bbor/mixin/network/play/server/MixinSPacketCustomPayload.java
@@ -19,32 +19,33 @@ public abstract class MixinSPacketCustomPayload {
 
     @Redirect(method = "processPacket", at = @At(value = "INVOKE", target = "Lnet/minecraft/network/play/INetHandlerPlayClient;handleCustomPayload(Lnet/minecraft/network/play/server/SPacketCustomPayload;)V"))
     private void processPacket(INetHandlerPlayClient netHandlerPlayClient, SPacketCustomPayload packet) {
-        PacketBuffer data = null;
-        try {
-            data = packet.getBufferData();
-            PayloadReader reader = new PayloadReader(data);
-            switch (channel.toString()) {
-                case InitializeClient.NAME: {
-                    EventBus.publish(InitializeClient.getEvent(reader));
-                    ((NetHandlerPlayClient) netHandlerPlayClient).sendPacket(SubscribeToServer.getPayload().build());
-                    break;
-                }
-                case AddBoundingBox.NAME: {
-                    EventBus.publish(AddBoundingBox.getEvent(reader));
-                    break;
-                }
-                case RemoveBoundingBox.NAME: {
-                    EventBus.publish(RemoveBoundingBox.getEvent(reader));
-                    break;
-                }
-                default: {
-                    netHandlerPlayClient.handleCustomPayload(packet);
-                    break;
+        String channelName = channel.toString();
+        if (channelName.startsWith("bbor:")) {
+            PacketBuffer data = null;
+            try {
+                data = packet.getBufferData();
+                PayloadReader reader = new PayloadReader(data);
+                switch (channelName) {
+                    case InitializeClient.NAME: {
+                        EventBus.publish(InitializeClient.getEvent(reader));
+                        ((NetHandlerPlayClient) netHandlerPlayClient).sendPacket(SubscribeToServer.getPayload().build());
+                        break;
+                    }
+                    case AddBoundingBox.NAME: {
+                        EventBus.publish(AddBoundingBox.getEvent(reader));
+                        break;
+                    }
+                    case RemoveBoundingBox.NAME: {
+                        EventBus.publish(RemoveBoundingBox.getEvent(reader));
+                        break;
+                    }
                 }
+            } finally {
+                if (data != null)
+                    data.release();
             }
-        } finally {
-            if (data != null)
-                data.release();
+        } else {
+            netHandlerPlayClient.handleCustomPayload(packet);
         }
     }
 }

diff --git a/src/main/java/com/irtimaled/bbor/mixin/network/play/server/MixinSPacketSpawnPosition.java b/src/main/java/com/irtimaled/bbor/mixin/network/play/server/MixinSPacketSpawnPosition.java
index d0b6e868794d6627250a2cd71a3b58920943c90d..6fcabe12086b69d059c6eef19dcabcc87dae6951 100644 (file)
--- a/src/main/java/com/irtimaled/bbor/mixin/network/play/server/MixinSPacketSpawnPosition.java
+++ b/src/main/java/com/irtimaled/bbor/mixin/network/play/server/MixinSPacketSpawnPosition.java
@@ -1,7 +1,6 @@
 package com.irtimaled.bbor.mixin.network.play.server;
 
-import com.irtimaled.bbor.client.events.UpdateWorldSpawnReceived;
-import com.irtimaled.bbor.common.EventBus;
+import com.irtimaled.bbor.client.interop.ClientInterop;
 import net.minecraft.network.play.server.SPacketSpawnPosition;
 import net.minecraft.util.math.BlockPos;
 import org.spongepowered.asm.mixin.Mixin;
@@ -17,6 +16,6 @@ public abstract class MixinSPacketSpawnPosition {
 
     @Inject(method = "processPacket", at = @At("RETURN"))
     private void afterProcessPacket(CallbackInfo ci) {
-        EventBus.publish(new UpdateWorldSpawnReceived(spawnBlockPos.getX(), spawnBlockPos.getZ()));
+        ClientInterop.updateWorldSpawnReceived(spawnBlockPos);
     }
 }