From ea237f20c96b7e56cbd8e2f78e55959504ce5486 Mon Sep 17 00:00:00 2001
From: Camotoy <20743703+Camotoy@users.noreply.github.com>
Date: Sun, 6 Jun 2021 21:39:39 -0400
Subject: [PATCH] Shulker attach position is now irrelevant, it seems

---
 .../connector/entity/living/monster/ShulkerEntity.java     | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/connector/src/main/java/org/geysermc/connector/entity/living/monster/ShulkerEntity.java b/connector/src/main/java/org/geysermc/connector/entity/living/monster/ShulkerEntity.java
index b2e3f834e..142c0012b 100644
--- a/connector/src/main/java/org/geysermc/connector/entity/living/monster/ShulkerEntity.java
+++ b/connector/src/main/java/org/geysermc/connector/entity/living/monster/ShulkerEntity.java
@@ -48,13 +48,6 @@ public class ShulkerEntity extends GolemEntity {
             BlockFace blockFace = (BlockFace) entityMetadata.getValue();
             metadata.put(EntityData.SHULKER_ATTACH_FACE, (byte) blockFace.ordinal());
         }
-        //TODO - this was removed on Java Edition, but does Bedrock Edition still need it??
-//        if (entityMetadata.getId() == 16) {
-//            Position position = (Position) entityMetadata.getValue();
-//            if (position != null) {
-//                metadata.put(EntityData.SHULKER_ATTACH_POS, Vector3i.from(position.getX(), position.getY(), position.getZ()));
-//            }
-//        }
 
         if (entityMetadata.getId() == 17) {
             int height = (byte) entityMetadata.getValue();