From 6cc4695e8cc8792da4a5a082be161aa2e79c52d8 Mon Sep 17 00:00:00 2001 From: FN-FAL113 <88238718+FN-FAL113@users.noreply.github.com> Date: Mon, 31 Jan 2022 10:11:00 +0800 Subject: [PATCH] Remove nested condition --- .../items/multiblocks/miner/MiningTask.java | 20 +++++++++---------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/multiblocks/miner/MiningTask.java b/src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/multiblocks/miner/MiningTask.java index 0cbebe109..8db4e1375 100644 --- a/src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/multiblocks/miner/MiningTask.java +++ b/src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/multiblocks/miner/MiningTask.java @@ -311,18 +311,16 @@ class MiningTask implements Runnable { for (MachineFuel fuelType : miner.fuelTypes) { ItemStack item = inv.getContents()[i]; - if (fuelType.test(item)) { - /* - * Fixes #3336 - * Respects the state of the miner if there are - * no any errors during #setPistonState - */ - if (running) { - ItemUtils.consumeItem(item, false); + /* + * Fixes #3336 + * Respects the state of the miner if there are + * no any errors during #setPistonState + */ + if (fuelType.test(item) && running) { + ItemUtils.consumeItem(item, false); - if (miner instanceof AdvancedIndustrialMiner) { - inv.addItem(new ItemStack(Material.BUCKET)); - } + if (miner instanceof AdvancedIndustrialMiner) { + inv.addItem(new ItemStack(Material.BUCKET)); } return fuelType.getTicks();