diff --git a/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java b/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
index 8cc0f8c2b..d22ef80c8 100644
--- a/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
+++ b/src/main/java/org/bukkit/craftbukkit/block/CraftSign.java
@@ -93,14 +93,15 @@ public class CraftSign<T extends TileEntitySign> extends CraftBlockEntityState<T
         super.applyTo(sign);
     }
 
-    public static void openSign(Sign sign, Player player) {
+    public static void openSign(Sign sign, Player player, Side side) {
         Preconditions.checkArgument(sign != null, "sign == null");
+        Preconditions.checkArgument(side != null, "side == null");
         Preconditions.checkArgument(sign.isPlaced(), "Sign must be placed");
         Preconditions.checkArgument(sign.getWorld() == player.getWorld(), "Sign must be in same world as Player");
 
         TileEntitySign handle = ((CraftSign<?>) sign).getTileEntity();
 
-        ((CraftPlayer) player).getHandle().openTextEdit(handle, true);
+        ((CraftPlayer) player).getHandle().openTextEdit(handle, Side.FRONT == side);
     }
 
     public static IChatBaseComponent[] sanitizeLines(String[] lines) {
diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
index 7c4ea7f68..c600da19f 100644
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
@@ -2098,12 +2098,12 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
 
     @Override
     public void openSign(Sign sign) {
-        CraftSign.openSign(sign, this);
+        openSign(sign, Side.FRONT);
     }
 
     @Override
     public void openSign(@NotNull Sign sign, @NotNull Side side) {
-        openSign(sign); // todo implement per side
+        CraftSign.openSign(sign, this, side);
     }
 
     @Override