diff --git a/src/main/java/com/gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java b/src/main/java/com/gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java index c1c5bdf31..e2e994b4b 100644 --- a/src/main/java/com/gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java +++ b/src/main/java/com/gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java @@ -101,18 +101,20 @@ public final class AlchemyPotionBrewer { } for (int i = 0; i < 3; i++) { - ItemStack item = inventory.getItem(i); + ItemStack item = inventory.getItem(i).clone(); - if (!isEmpty(item) && PotionConfig.getInstance().isValidPotion(item)) { - AlchemyPotion input = PotionConfig.getInstance().getPotion(item.getDurability()); - AlchemyPotion output = PotionConfig.getInstance().getPotion(input.getChildDataValue(ingredient)); + if (isEmpty(item) || !PotionConfig.getInstance().isValidPotion(item)) { + continue; + } - if (output != null) { - inventory.setItem(i, output.toItemStack(item.getAmount()).clone()); + AlchemyPotion input = PotionConfig.getInstance().getPotion(item.getDurability()); + AlchemyPotion output = PotionConfig.getInstance().getPotion(input.getChildDataValue(ingredient)); - if (player != null) { - UserManager.getPlayer(player).getAlchemyManager().handlePotionBrewSuccesses(1); - } + if (output != null) { + inventory.setItem(i, output.toItemStack(item.getAmount()).clone()); + + if (player != null) { + UserManager.getPlayer(player).getAlchemyManager().handlePotionBrewSuccesses(1); } } }