gradle.properties

@@ -8,4 +8,4 @@ mcp_channel=official
 mcp_mappings=1.17.1
 botania=1.16.2-405
 version_major=2
-version_minor=4
+version_minor=3

src/main/java/thetadev/constructionwand/basics/WandUtil.java

@@ -11,11 +11,9 @@ import net.minecraft.world.entity.player.Player;
 import net.minecraft.world.item.BlockItem;
 import net.minecraft.world.item.Item;
 import net.minecraft.world.item.ItemStack;
-import net.minecraft.world.item.Items;
 import net.minecraft.world.level.Level;
 import net.minecraft.world.level.block.state.BlockState;
 import net.minecraft.world.phys.AABB;
-import net.minecraft.world.phys.BlockHitResult;
 import net.minecraft.world.phys.Vec3;
 import net.minecraft.world.phys.shapes.VoxelShape;
 import net.minecraftforge.common.MinecraftForge;
@@ -24,7 +22,6 @@ import net.minecraftforge.event.world.BlockEvent;
 import thetadev.constructionwand.ConstructionWand;
 import thetadev.constructionwand.containers.ContainerManager;
 import thetadev.constructionwand.items.wand.ItemWand;
-import thetadev.constructionwand.wand.WandItemUseContext;
 
 import javax.annotation.Nullable;
 import java.util.ArrayList;
@@ -190,19 +187,13 @@ public class WandUtil
 
     /**
      * Tests if a wand can place a block at a certain position.
-     * This check is independent of the used block.
+     * This check is independent from the used block.
      */
     public static boolean isPositionPlaceable(Level world, Player player, BlockPos pos, boolean replace) {
         if(!isPositionModifiable(world, player, pos)) return false;
 
         // If replace mode is off, target has to be air
-        if(world.isEmptyBlock(pos)) return true;
-
-        // Otherwise, check if the block can be replaced by a generic block
-        return replace && world.getBlockState(pos).canBeReplaced(
-                new WandItemUseContext(world, player,
-                        new BlockHitResult(new Vec3(0, 0, 0), Direction.DOWN, pos, false),
-                        pos, (BlockItem) Items.STONE));
+        return replace || world.isEmptyBlock(pos);
     }
 
     public static boolean isBlockRemovable(Level world, Player player, BlockPos pos) {
@@ -214,10 +205,6 @@ public class WandUtil
         return true;
     }
 
-    public static boolean isBlockPermeable(Level world, BlockPos pos) {
-        return world.isEmptyBlock(pos) || world.getBlockState(pos).getCollisionShape(world, pos).isEmpty();
-    }
-
     public static boolean entitiesCollidingWithBlock(Level world, BlockState blockState, BlockPos pos) {
         VoxelShape shape = blockState.getCollisionShape(world, pos);
         if(!shape.isEmpty()) {

src/main/java/thetadev/constructionwand/wand/action/ActionDestruction.java

@@ -10,7 +10,6 @@ import net.minecraft.world.phys.BlockHitResult;
 import thetadev.constructionwand.api.IWandAction;
 import thetadev.constructionwand.api.IWandSupplier;
 import thetadev.constructionwand.basics.ConfigServer;
-import thetadev.constructionwand.basics.WandUtil;
 import thetadev.constructionwand.basics.option.WandOptions;
 import thetadev.constructionwand.wand.undo.DestroySnapshot;
 import thetadev.constructionwand.wand.undo.ISnapshot;
@@ -39,14 +38,14 @@ public class ActionDestruction implements IWandAction
         HashSet<BlockPos> allCandidates = new HashSet<>();
 
         // Block face the wand was pointed at
-        Direction breakFace = rayTraceResult.getDirection();
+        Direction breakDirection = rayTraceResult.getDirection();
         // Block the wand was pointed at
         BlockPos startingPoint = rayTraceResult.getBlockPos();
         BlockState targetBlock = world.getBlockState(rayTraceResult.getBlockPos());
 
         // Is break direction allowed by lock?
         // Tried to break blocks from top/bottom face, so the wand should allow breaking in NS/EW direction
-        if(breakFace == Direction.UP || breakFace == Direction.DOWN) {
+        if(breakDirection == Direction.UP || breakDirection == Direction.DOWN) {
             if(options.testLock(WandOptions.LOCK.NORTHSOUTH) || options.testLock(WandOptions.LOCK.EASTWEST))
                 candidates.add(startingPoint);
         }
@@ -57,10 +56,6 @@ public class ActionDestruction implements IWandAction
         // Process current candidates, stop when none are avaiable or block limit is reached
         while(!candidates.isEmpty() && destroySnapshots.size() < limit) {
             BlockPos currentCandidate = candidates.removeFirst();
-
-            // Only break blocks facing the player, with no collidable blocks in between
-            if(!WandUtil.isBlockPermeable(world, currentCandidate.offset(breakFace.getNormal()))) continue;
-
             try {
                 BlockState candidateBlock = world.getBlockState(currentCandidate);
 
@@ -71,7 +66,7 @@ public class ActionDestruction implements IWandAction
                     if(snapshot == null) continue;
                     destroySnapshots.add(snapshot);
 
-                    switch(breakFace) {
+                    switch(breakDirection) {
                         case DOWN:
                         case UP:
                             if(options.testLock(WandOptions.LOCK.NORTHSOUTH)) {

src/main/java/thetadev/constructionwand/wand/supplier/SupplierInventory.java

@@ -6,7 +6,6 @@ import net.minecraft.world.entity.player.Player;
 import net.minecraft.world.item.BlockItem;
 import net.minecraft.world.item.Item;
 import net.minecraft.world.item.ItemStack;
-import net.minecraft.world.item.Items;
 import net.minecraft.world.level.Level;
 import net.minecraft.world.level.block.state.BlockState;
 import net.minecraft.world.phys.BlockHitResult;
@@ -52,7 +51,7 @@ public class SupplierInventory implements IWandSupplier
             addBlockItem((BlockItem) offhandStack.getItem());
         }
         // Otherwise use target block
-        else if(target != null && target != Items.AIR) {
+        else {
             addBlockItem(target);
 
             // Add replacement items