From 86f18c9392e0c72f1e09abb8053a12ae7c5113a2 Mon Sep 17 00:00:00 2001
From: AJ Ferguson <AJ-Ferguson@users.noreply.github.com>
Date: Sat, 1 Aug 2020 10:41:59 -0800
Subject: [PATCH] Remove Y pos workaround in BedrockItemFrameDropItemTranslator
 (#1037)

---
 .../bedrock/BedrockItemFrameDropItemTranslator.java    | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/connector/src/main/java/org/geysermc/connector/network/translators/bedrock/BedrockItemFrameDropItemTranslator.java b/connector/src/main/java/org/geysermc/connector/network/translators/bedrock/BedrockItemFrameDropItemTranslator.java
index 168e552b..8f563c0e 100644
--- a/connector/src/main/java/org/geysermc/connector/network/translators/bedrock/BedrockItemFrameDropItemTranslator.java
+++ b/connector/src/main/java/org/geysermc/connector/network/translators/bedrock/BedrockItemFrameDropItemTranslator.java
@@ -40,15 +40,7 @@ public class BedrockItemFrameDropItemTranslator extends PacketTranslator<ItemFra
 
     @Override
     public void translate(ItemFrameDropItemPacket packet, GeyserSession session) {
-        // I hope that, when we die, God (or whoever is waiting for us) tells us exactly why this code exists
-        // The packet sends the Y coordinate (and just the Y coordinate) divided by two, and it's negative if it needs to be subtracted by one
-        int y;
-        if (packet.getBlockPosition().getY() > 0) {
-            y = packet.getBlockPosition().getY() * 2;
-        } else {
-            y = (packet.getBlockPosition().getY() * -2) - 1;
-        }
-        Vector3i position = Vector3i.from(packet.getBlockPosition().getX(), y, packet.getBlockPosition().getZ());
+        Vector3i position = Vector3i.from(packet.getBlockPosition().getX(), packet.getBlockPosition().getY(), packet.getBlockPosition().getZ());
         ClientPlayerInteractEntityPacket interactPacket = new ClientPlayerInteractEntityPacket((int) ItemFrameEntity.getItemFrameEntityId(session, position),
                 InteractAction.ATTACK, Hand.MAIN_HAND, session.isSneaking());
         session.sendDownstreamPacket(interactPacket);