GeyserMC-Mirror/Geyser
1
0
Fork 0
mirror of https://github.com/GeyserMC/Geyser.git synced 2025-12-22 00:09:32 +00:00
Code Issues Releases Activity

Fix chunks not loading when riding a vehicle, fix world border corrections not applying (#5410)

Browse Source
This commit is contained in:
chris
2025-03-16 01:28:46 +01:00
committed by GitHub
parent f27290a8b3
commit 32160c5c64
1 changed files with 73 additions and 57 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
core/src/main/java/org/geysermc/geyser/translator/protocol/bedrock/entity/player/input/BedrockMovePlayer.java
View File

@@ -76,7 +76,7 @@ final class BedrockMovePlayer {
boolean hasVehicle = entity.getVehicle() != null; boolean hasVehicle = entity.getVehicle() != null;
// shouldSendPositionReminder also increments a tick counter, so make sure it's always called unless the player is on a vehicle. // shouldSendPositionReminder also increments a tick counter, so make sure it's always called unless the player is on a vehicle.
boolean positionChanged = !hasVehicle && (session.getInputCache().shouldSendPositionReminder() || actualPositionChanged); boolean positionChangedAndShouldUpdate = !hasVehicle && (session.getInputCache().shouldSendPositionReminder() || actualPositionChanged);
boolean rotationChanged = hasVehicle || (entity.getYaw() != yaw || entity.getPitch() != pitch || entity.getHeadYaw() != headYaw); boolean rotationChanged = hasVehicle || (entity.getYaw() != yaw || entity.getPitch() != pitch || entity.getHeadYaw() != headYaw);
if (session.getLookBackScheduledFuture() != null) { if (session.getLookBackScheduledFuture() != null) {
@@ -87,15 +87,23 @@ final class BedrockMovePlayer {
} }
// Client is telling us it wants to move down, but something is blocking it from doing so. // Client is telling us it wants to move down, but something is blocking it from doing so.
boolean isOnGround = packet.getInputData().contains(PlayerAuthInputData.VERTICAL_COLLISION) && packet.getDelta().getY() < 0; boolean isOnGround;
if (hasVehicle) {
// VERTICAL_COLLISION is not accurate while in a vehicle (as of 1.21.62)
isOnGround = Math.abs(packet.getDelta().getY()) < 0.1;
} else {
isOnGround = packet.getInputData().contains(PlayerAuthInputData.VERTICAL_COLLISION) && packet.getDelta().getY() < 0;
}
// This takes into account no movement sent from the client, but the player is trying to move anyway. // This takes into account no movement sent from the client, but the player is trying to move anyway.
// (Press into a wall in a corner - you're trying to move but nothing actually happens) // (Press into a wall in a corner - you're trying to move but nothing actually happens)
// This isn't sent when a player is riding a vehicle (as of 1.21.62)
boolean horizontalCollision = packet.getInputData().contains(PlayerAuthInputData.HORIZONTAL_COLLISION); boolean horizontalCollision = packet.getInputData().contains(PlayerAuthInputData.HORIZONTAL_COLLISION);
// If only the pitch and yaw changed // If only the pitch and yaw changed
// This isn't needed, but it makes the packets closer to vanilla // This isn't needed, but it makes the packets closer to vanilla
// It also means you can't "lag back" while only looking, in theory // It also means you can't "lag back" while only looking, in theory
if (!positionChanged && rotationChanged) { if (!positionChangedAndShouldUpdate && rotationChanged) {
ServerboundMovePlayerRotPacket playerRotationPacket = new ServerboundMovePlayerRotPacket(isOnGround, horizontalCollision, yaw, pitch); ServerboundMovePlayerRotPacket playerRotationPacket = new ServerboundMovePlayerRotPacket(isOnGround, horizontalCollision, yaw, pitch);
entity.setYaw(yaw); entity.setYaw(yaw);
@@ -103,8 +111,15 @@ final class BedrockMovePlayer {
entity.setHeadYaw(headYaw); entity.setHeadYaw(headYaw);
session.sendDownstreamGamePacket(playerRotationPacket); session.sendDownstreamGamePacket(playerRotationPacket);
} else if (positionChanged) {
// Player position MUST be updated on our end, otherwise e.g. chunk loading breaks
if (hasVehicle) {
entity.setPositionManual(packet.getPosition());
session.getSkullCache().updateVisibleSkulls();
}
} else if (positionChangedAndShouldUpdate) {
if (isValidMove(session, entity.getPosition(), packet.getPosition())) { if (isValidMove(session, entity.getPosition(), packet.getPosition())) {
if (!session.getWorldBorder().isPassingIntoBorderBoundaries(entity.getPosition(), true)) {
CollisionResult result = session.getCollisionManager().adjustBedrockPosition(packet.getPosition(), isOnGround, packet.getInputData().contains(PlayerAuthInputData.HANDLE_TELEPORT)); CollisionResult result = session.getCollisionManager().adjustBedrockPosition(packet.getPosition(), isOnGround, packet.getInputData().contains(PlayerAuthInputData.HANDLE_TELEPORT));
if (result != null) { // A null return value cancels the packet if (result != null) { // A null return value cancels the packet
Vector3d position = result.correctedMovement(); Vector3d position = result.correctedMovement();
@@ -169,6 +184,7 @@ final class BedrockMovePlayer {
session.getInputCache().markPositionPacketSent(); session.getInputCache().markPositionPacketSent();
session.getSkullCache().updateVisibleSkulls(); session.getSkullCache().updateVisibleSkulls();
} }
}
} else { } else {
// Not a valid move // Not a valid move
session.getGeyser().getLogger().debug("Recalculating position..."); session.getGeyser().getLogger().debug("Recalculating position...");