Update ClickTP

- add the paperclip exploit
- make the module work while using freecam
- fix the conditions where you shouldn't tp (using an item, placing a block, etc.)

closes #4549
closes #4146
closes #2775
closes #5595

Co-authored-by: tyler <[email protected]>

Author: Wide-Cat

src/main/java/meteordevelopment/meteorclient/systems/modules/movement/ClickTP.java

@@ -6,60 +6,83 @@
 package meteordevelopment.meteorclient.systems.modules.movement;
 
 import meteordevelopment.meteorclient.events.world.TickEvent;
-import meteordevelopment.meteorclient.settings.DoubleSetting;
-import meteordevelopment.meteorclient.settings.Setting;
-import meteordevelopment.meteorclient.settings.SettingGroup;
 import meteordevelopment.meteorclient.systems.modules.Categories;
 import meteordevelopment.meteorclient.systems.modules.Module;
 import meteordevelopment.orbit.EventHandler;
 import net.minecraft.block.BlockState;
+import net.minecraft.client.render.Camera;
+import net.minecraft.item.BlockItem;
+import net.minecraft.item.consume.UseAction;
+import net.minecraft.network.packet.c2s.play.PlayerMoveC2SPacket;
 import net.minecraft.util.ActionResult;
 import net.minecraft.util.Hand;
 import net.minecraft.util.hit.BlockHitResult;
 import net.minecraft.util.hit.EntityHitResult;
 import net.minecraft.util.hit.HitResult;
 import net.minecraft.util.math.BlockPos;
 import net.minecraft.util.math.Direction;
+import net.minecraft.util.math.Vec3d;
 import net.minecraft.util.shape.VoxelShape;
+import net.minecraft.world.RaycastContext;
 
 public class ClickTP extends Module {
-    private final SettingGroup sgGeneral = settings.getDefaultGroup();
-
-    private final Setting<Double> maxDistance = sgGeneral.add(new DoubleSetting.Builder()
-        .name("max-distance")
-        .description("The maximum distance you can teleport.")
-        .defaultValue(5)
-        .build()
-    );
 
     public ClickTP() {
         super(Categories.Movement, "click-tp", "Teleports you to the block you click on.");
     }
 
     @EventHandler
     private void onTick(TickEvent.Post event) {
-        if (mc.player.isUsingItem()) return;
+        if (mc.player.getInventory().getSelectedStack().getUseAction() != UseAction.NONE) return;
+        if (!mc.options.useKey.isPressed()) return;
+
+        if (mc.crosshairTarget != null) {
+            if (mc.crosshairTarget.getType() == HitResult.Type.ENTITY && mc.player.interact(((EntityHitResult) mc.crosshairTarget).getEntity(), Hand.MAIN_HAND) != ActionResult.PASS) return;
+            if (mc.crosshairTarget.getType() == HitResult.Type.BLOCK && mc.player.getMainHandStack().getItem() instanceof BlockItem) return;
+        }
+
+        Camera camera = mc.gameRenderer.getCamera();
+        Vec3d cameraPos = camera.getPos();
 
-        if (mc.options.useKey.isPressed()) {
-            HitResult hitResult = mc.player.raycast(maxDistance.get(), 1f / 20f, false);
+        // Calculate the direction the camera is looking based on its pitch and yaw, and extend this direction 210 units away from the camera position
+        // 210 is used here as the maximum distance for this exploit is 200 blocks
+        // This is done to be able to click tp while in freecam
+        Vec3d direction = Vec3d.fromPolar(camera.getPitch(), camera.getYaw()).multiply(210);
+        Vec3d targetPos = cameraPos.add(direction);
 
-            if (hitResult.getType() == HitResult.Type.ENTITY && mc.player.interact(((EntityHitResult) hitResult).getEntity(), Hand.MAIN_HAND) != ActionResult.PASS) return;
+        RaycastContext context = new RaycastContext(
+            cameraPos,   // start position of the ray
+            targetPos,   // end position of the ray
+            RaycastContext.ShapeType.OUTLINE,
+            RaycastContext.FluidHandling.NONE,
+            mc.player
+        );
 
-            if (hitResult.getType() == HitResult.Type.BLOCK) {
-                BlockPos pos = ((BlockHitResult) hitResult).getBlockPos();
-                Direction side = ((BlockHitResult) hitResult).getSide();
+        BlockHitResult hitResult = mc.world.raycast(context);
 
-                if (mc.world.getBlockState(pos).onUse(mc.world, mc.player, (BlockHitResult) hitResult) != ActionResult.PASS) return;
+        if (hitResult.getType() == HitResult.Type.BLOCK) {
+            BlockPos pos = hitResult.getBlockPos();
+            Direction side = hitResult.getSide();
 
-                BlockState state = mc.world.getBlockState(pos);
+            if (mc.world.getBlockState(pos).onUse(mc.world, mc.player, hitResult) != ActionResult.PASS) return;
 
-                VoxelShape shape = state.getCollisionShape(mc.world, pos);
-                if (shape.isEmpty()) shape = state.getOutlineShape(mc.world, pos);
+            BlockState state = mc.world.getBlockState(pos);
 
-                double height = shape.isEmpty() ? 1 : shape.getMax(Direction.Axis.Y);
+            VoxelShape shape = state.getCollisionShape(mc.world, pos);
+            if (shape.isEmpty()) shape = state.getOutlineShape(mc.world, pos);
 
-                mc.player.setPosition(pos.getX() + 0.5 + side.getOffsetX(), pos.getY() + height, pos.getZ() + 0.5 + side.getOffsetZ());
+            double height = shape.isEmpty() ? 1 : shape.getMax(Direction.Axis.Y);
+
+            Vec3d newPos = new Vec3d(pos.getX() + 0.5 + side.getOffsetX(), pos.getY() + height, pos.getZ() + 0.5 + side.getOffsetZ());
+            int packetsRequired = (int) Math.ceil(mc.player.getPos().distanceTo(newPos) / 10) - 1; // subtract 1 to account for the final packet with movement
+            if (packetsRequired > 19) packetsRequired = 0;
+
+            for (int packetNumber = 0; packetNumber < (packetsRequired); packetNumber++) {
+                mc.player.networkHandler.sendPacket(new PlayerMoveC2SPacket.OnGroundOnly(true, true));
             }
+
+            mc.player.networkHandler.sendPacket(new PlayerMoveC2SPacket.PositionAndOnGround(newPos.x, newPos.y, newPos.z, true, true));
+            mc.player.setPosition(newPos);
         }
     }
 }