fix(barrel): Use different method to discern direction

Thorinwasher · Thorinwasher · commit f861b205ea21 · 2025-07-02T09:02:49.000+02:00
diff --git a/src/main/java/com/dre/brewery/BarrelBody.java b/src/main/java/com/dre/brewery/BarrelBody.java
@@ -135,13 +135,7 @@ public BarrelWoodType getWood() {
         if (direction == null) {
             return BarrelWoodType.ANY;
         }
-        // TODO: replace this with Block#getRelative(BlockFace)
-        Block wood = switch (direction) {
-            case WEST -> spigot.getRelative(1, 0, 0);
-            case EAST -> spigot.getRelative(-1, 0, 0);
-            case SOUTH -> spigot.getRelative(0, 0, 1);
-            case NORTH -> spigot.getRelative(0, 0, -1);
-        };
+        Block wood = spigot.getRelative(direction.getFace());
         return BarrelWoodType.fromMaterial(wood.getType());
     }
 

Original file line number	Diff line number	Diff line change
`@@ -135,13 +135,7 @@ public BarrelWoodType getWood() {`
`135`	`135`	`if (direction == null) {`
`136`	`136`	`return BarrelWoodType.ANY;`
`137`	`137`	`}`
`138`		`- // TODO: replace this with Block#getRelative(BlockFace)`
`139`		`- Block wood = switch (direction) {`
`140`		`- case WEST -> spigot.getRelative(1, 0, 0);`
`141`		`- case EAST -> spigot.getRelative(-1, 0, 0);`
`142`		`- case SOUTH -> spigot.getRelative(0, 0, 1);`
`143`		`- case NORTH -> spigot.getRelative(0, 0, -1);`
`144`		`- };`
	`138`	`+ Block wood = spigot.getRelative(direction.getFace());`
`145`	`139`	`return BarrelWoodType.fromMaterial(wood.getType());`
`146`	`140`	`}`
`147`	`141`