Merge pull request #3749 from J3fftw1/feature/maxEnchants

added a way to set max enchants to go on an item

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/electric/machines/enchanting/AbstractEnchantmentMachine.java

@@ -7,6 +7,7 @@ import javax.annotation.Nonnull;
 import javax.annotation.ParametersAreNonnullByDefault;
 
 import org.bukkit.Material;
+import org.bukkit.entity.Item;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.ItemMeta;
 
@@ -28,6 +29,7 @@ import me.mrCookieSlime.Slimefun.api.inventory.BlockMenu;
  *
  * @author TheBusyBiscuit
  * @author Rothes
+ * @author J3fftw1
  *
  * @see AutoEnchanter
  * @see AutoDisenchanter
@@ -37,6 +39,8 @@ abstract class AbstractEnchantmentMachine extends AContainer {
 
     private final ItemSetting<Boolean> useLevelLimit = new ItemSetting<>(this, "use-enchant-level-limit", false);
     private final IntRangeSetting levelLimit = new IntRangeSetting(this, "enchant-level-limit", 0, 10, Short.MAX_VALUE);
+    private final ItemSetting<Integer> maxEnchants = new IntRangeSetting(this, "max-enchants", 0, 10, Short.MAX_VALUE);
+    private final ItemSetting<Boolean> useMaxEnchants= new ItemSetting<>(this, "use-max-encahnts", false);
     private final ItemSetting<Boolean> useIgnoredLores = new ItemSetting<>(this, "use-ignored-lores", false);
     private final ItemSetting<List<String>> ignoredLores = new ItemSetting<>(this, "ignored-lores", Collections.singletonList("&7- &cCan't be used in " + this.getItemName()));
 
@@ -48,6 +52,8 @@ abstract class AbstractEnchantmentMachine extends AContainer {
         addItemSetting(levelLimit);
         addItemSetting(useIgnoredLores);
         addItemSetting(ignoredLores);
+        addItemSetting(maxEnchants);
+        addItemSetting(useMaxEnchants);
     }
 
     protected boolean isEnchantmentLevelAllowed(int enchantmentLevel) {
@@ -84,4 +90,8 @@ abstract class AbstractEnchantmentMachine extends AContainer {
 
         return false;
     }
+
+    protected boolean isEnchantmentAmountAllowed(@Nonnull ItemStack item ) {
+        return !useMaxEnchants.getValue() || item.getEnchantments().size() >= maxEnchants.getValue();
+    }
 }

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/electric/machines/enchanting/AutoDisenchanter.java

@@ -94,6 +94,10 @@ public class AutoDisenchanter extends AbstractEnchantmentMachine {
             }
         }
 
+        if (isEnchantmentAmountAllowed(item)) {
+            return null;
+        }
+
         // Check if we found any valid enchantments
         if (!enchantments.isEmpty()) {
             ItemStack disenchantedItem = item.clone();

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/items/electric/machines/enchanting/AutoEnchanter.java

@@ -116,6 +116,14 @@ public class AutoEnchanter extends AbstractEnchantmentMachine {
             enchantments.entrySet().removeIf(e -> target.getEnchantmentLevel(e.getKey()) >= e.getValue());
         }
 
+        /*
+         * When maxEnchants is set to -1 it will be ignored. When it's set to 0 it will not allow any enchants to go
+         * on an item. When maxEnchants is set to any other value it will allow that many enchants to go on the item.
+         */
+        if (isEnchantmentAmountAllowed(target)) {
+            return null;
+        }
+
         // Check if we found any valid enchantments
         if (!enchantments.isEmpty()) {
             ItemStack enchantedItem = target.clone();