No Interact module's block mine whitelist now actually works (#6365)

Author: moxvallix

src/main/java/meteordevelopment/meteorclient/systems/modules/player/NoInteract.java

@@ -149,13 +149,13 @@ private void onInteractEntity(InteractEntityEvent event) {
     }
 
     private boolean shouldAttackBlock(BlockPos blockPos) {
-        if (blockMineMode.get() == ListMode.WhiteList &&
-            blockMine.get().contains(mc.level.getBlockState(blockPos).getBlock())) {
-            return false;
-        }
+        boolean blockInList = blockMine.get().contains(mc.level.getBlockState(blockPos).getBlock());
 
-        return blockMineMode.get() != ListMode.BlackList ||
-            !blockMine.get().contains(mc.level.getBlockState(blockPos).getBlock());
+        if (blockMineMode.get() == ListMode.WhiteList) {
+            return blockInList;
+        } else {
+            return !blockInList;
+        }
     }
 
     private boolean shouldInteractBlock(BlockHitResult hitResult, InteractionHand hand) {
@@ -167,13 +167,13 @@ private boolean shouldInteractBlock(BlockHitResult hitResult, InteractionHand ha
         }
 
         // Blocks
-        if (blockInteractMode.get() == ListMode.BlackList &&
-            blockInteract.get().contains(mc.level.getBlockState(hitResult.getBlockPos()).getBlock())) {
-            return false;
-        }
+        boolean blockInList = blockInteract.get().contains(mc.level.getBlockState(hitResult.getBlockPos()).getBlock());
 
-        return blockInteractMode.get() != ListMode.WhiteList ||
-            blockInteract.get().contains(mc.level.getBlockState(hitResult.getBlockPos()).getBlock());
+        if (blockInteractMode.get() == ListMode.WhiteList) {
+            return blockInList;
+        } else {
+            return !blockInList;
+        }
     }
 
     private boolean shouldAttackEntity(Entity entity) {