Use vanilla teleportTo instead of reimplementing all its functionality

Author: Robotgiggle

Common/src/main/java/at/petrak/hexcasting/common/casting/actions/spells/great/OpTeleport.kt

@@ -12,7 +12,6 @@ import at.petrak.hexcasting.api.casting.mishaps.MishapImmuneEntity
 import at.petrak.hexcasting.api.misc.MediaConstants
 import at.petrak.hexcasting.api.mod.HexConfig
 import at.petrak.hexcasting.api.mod.HexTags
-import at.petrak.hexcasting.common.msgs.MsgBlinkS2C
 import at.petrak.hexcasting.xplat.IXplatAbstractions
 import net.minecraft.core.BlockPos
 import net.minecraft.server.level.ServerLevel
@@ -106,39 +105,13 @@ object OpTeleport : SpellAction {
             return
         }
 
-        val playersToUpdate = mutableListOf<ServerPlayer>()
         val target = teleportee.position().add(delta)
 
         // A "sticky" entity teleports itself and its riders
-        val sticky = teleportee.type.`is`(HexTags.Entities.STICKY_TELEPORTERS)
-
-        // TODO: this probably does funky things with stacks of passengers. I doubt this will come up in practice
-        // though
-        if (sticky) {
-            teleportee.stopRiding()
-            teleportee.indirectPassengers.filterIsInstance<ServerPlayer>().forEach(playersToUpdate::add)
-            // this handles teleporting the passengers
-            teleportee.teleportTo(target.x, target.y, target.z)
-        } else {
-            // Snap everyone off the stacks
-            teleportee.stopRiding()
+        // This is the default behavior for teleportTo(), so we remove the riders if the teleportee *isn't* sticky
+        teleportee.stopRiding()
+        if (!teleportee.type.`is`(HexTags.Entities.STICKY_TELEPORTERS)) 
             teleportee.passengers.forEach(Entity::stopRiding)
-            if (teleportee is ServerPlayer) {
-                playersToUpdate.add(teleportee)
-            } else {
-                teleportee.setPos(teleportee.position().add(delta))
-            }
-        }
-
-        for (player in playersToUpdate) {
-            // See TeleportCommand
-            val chunkPos = ChunkPos(BlockPos.containing(delta))
-            // the `1` is apparently for "distance." i'm not sure what it does but this is what
-            // /tp does
-            world.chunkSource.addRegionTicket(TicketType.POST_TELEPORT, chunkPos, 1, player.id)
-            player.connection.resetPosition()
-            player.setPos(target)
-            IXplatAbstractions.INSTANCE.sendPacketToPlayer(player, MsgBlinkS2C(delta))
-        }
+        teleportee.teleportTo(target.x, target.y, target.z)
     }
 }

Common/src/main/java/at/petrak/hexcasting/common/msgs/MsgBlinkS2C.java

@@ -27,8 +27,6 @@ private static <T> ServerPlayNetworking.PlayChannelHandler makeServerBoundHandle
     public static void initClient() {
         ClientPlayNetworking.registerGlobalReceiver(MsgNewSpellPatternS2C.ID,
             makeClientBoundHandler(MsgNewSpellPatternS2C::deserialize, MsgNewSpellPatternS2C::handle));
-        ClientPlayNetworking.registerGlobalReceiver(
-            MsgBlinkS2C.ID, makeClientBoundHandler(MsgBlinkS2C::deserialize, MsgBlinkS2C::handle));
         ClientPlayNetworking.registerGlobalReceiver(MsgCastParticleS2C.ID,
             makeClientBoundHandler(MsgCastParticleS2C::deserialize, MsgCastParticleS2C::handle));
         ClientPlayNetworking.registerGlobalReceiver(MsgOpenSpellGuiS2C.ID,

Fabric/src/main/java/at/petrak/hexcasting/fabric/network/FabricPacketHandler.java

@@ -39,8 +39,6 @@ public static void init() {
         // Server -> client
         NETWORK.registerMessage(messageIdx++, MsgNewSpellPatternS2C.class, MsgNewSpellPatternS2C::serialize,
             MsgNewSpellPatternS2C::deserialize, makeClientBoundHandler(MsgNewSpellPatternS2C::handle));
-        NETWORK.registerMessage(messageIdx++, MsgBlinkS2C.class, MsgBlinkS2C::serialize,
-            MsgBlinkS2C::deserialize, makeClientBoundHandler(MsgBlinkS2C::handle));
         NETWORK.registerMessage(messageIdx++, MsgSentinelStatusUpdateAck.class, MsgSentinelStatusUpdateAck::serialize,
             MsgSentinelStatusUpdateAck::deserialize, makeClientBoundHandler(MsgSentinelStatusUpdateAck::handle));
         NETWORK.registerMessage(messageIdx++, MsgPigmentUpdateAck.class, MsgPigmentUpdateAck::serialize,