append new unit repair as minecraft namespace

src/main/java/xyz/alexcrea/cuanvil/api/UnitRepairApi.java

@@ -3,18 +3,16 @@ package xyz.alexcrea.cuanvil.api;
 import io.delilaheve.CustomAnvil;
 import kotlin.Triple;
 import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.configuration.file.FileConfiguration;
 import org.jetbrains.annotations.NotNull;
 import xyz.alexcrea.cuanvil.config.ConfigHolder;
 import xyz.alexcrea.cuanvil.dependency.DependencyManager;
 import xyz.alexcrea.cuanvil.gui.config.global.UnitRepairConfigGui;
-import xyz.alexcrea.cuanvil.gui.config.list.MappedGuiListConfigGui;
 import xyz.alexcrea.cuanvil.gui.config.list.UnitRepairElementListGui;
 
-import java.util.ArrayList;
+import java.util.*;
-import java.util.Collections;
-import java.util.List;
 
 /**
  * Custom Anvil api for unit repair.
@@ -22,7 +20,8 @@ import java.util.List;
 @SuppressWarnings("unused")
 public class UnitRepairApi {
 
-    private UnitRepairApi(){}
+    private UnitRepairApi() {
+    }
 
     private static Object saveChangeTask = null;
 
@@ -39,6 +38,19 @@ public class UnitRepairApi {
         return addUnitRepair(unit, repairable, 0.25, false);
     }
 
+    /**
+     * Write and add a custom anvil unit repair recipe.
+     * Will not write the recipe if it already exists or was deleted.
+     * Set the value to minecraft default value (0.25 = 25%)
+     *
+     * @param unit       The unit material used to repair the bellow item.
+     * @param repairable The item to be repaired.
+     * @return true if successful.
+     */
+    public static boolean addUnitRepair(@NotNull NamespacedKey unit, @NotNull NamespacedKey repairable) {
+        return addUnitRepair(unit, repairable, 0.25, false);
+    }
+
     /**
      * Write and add a custom anvil unit repair recipe.
      * Will not write the recipe if it already exists or was deleted.
@@ -63,8 +75,22 @@ public class UnitRepairApi {
      * @return true if successful.
      */
     public static boolean addUnitRepair(@NotNull Material unit, @NotNull Material repairable, double value, boolean overrideDeleted) {
+        return addUnitRepair(unit.getKey(), repairable.getKey(), value, overrideDeleted);
+    }
+
+    /**
+     * Write and add a custom anvil unit repair recipe.
+     * Will not write the recipe if it already exists.
+     *
+     * @param unit            The unit material used to repair the bellow item.
+     * @param repairable      The item to be repaired.
+     * @param value           The amount to be repaired by every unit. (1% = 0.01)
+     * @param overrideDeleted If we should write even if the recipe was previously deleted.
+     * @return true if successful.
+     */
+    public static boolean addUnitRepair(@NotNull NamespacedKey unit, @NotNull NamespacedKey repairable, double value, boolean overrideDeleted) {
         FileConfiguration config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
-        String path = unit.name().toLowerCase() + "." + repairable.name().toLowerCase();
+        String path = unit.toString().toLowerCase() + "." + repairable.toString().toLowerCase();
 
         if (!overrideDeleted && ConfigHolder.UNIT_REPAIR_HOLDER.isDeleted(path)) return false;
         if (config.contains(path)) return false;
@@ -83,10 +109,23 @@ public class UnitRepairApi {
      * @return true if successful.
      */
     public static boolean setUnitRepair(@NotNull Material unit, @NotNull Material repairable, double value) {
+        return setUnitRepair(unit.getKey(), repairable.getKey(), value);
+    }
+
+    /**
+     * Write and add a custom anvil unit repair recipe.
+     * Do not check if it previously existed or exist.
+     *
+     * @param unit       The unit material used to repair the bellow item.
+     * @param repairable The item to be repaired.
+     * @param value      The amount to be repaired by every unit. (1% = 0.01)
+     * @return true if successful.
+     */
+    public static boolean setUnitRepair(@NotNull NamespacedKey unit, @NotNull NamespacedKey repairable, double value) {
         FileConfiguration config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
 
-        String repairableName = repairable.name().toLowerCase();
+        String repairableName = repairable.toString().toLowerCase();
-        String path = unit.name().toLowerCase() + "." + repairableName;
+        String path = unit.toString().toLowerCase() + "." + repairableName;
 
         // Add to config then prepare save
         config.set(path, value);
@@ -97,7 +136,7 @@ public class UnitRepairApi {
         if (repairConfigGui != null) {
             UnitRepairElementListGui elementGui = repairConfigGui.getInstanceOrCreate(unit).getStored();
 
-            if(elementGui != null) elementGui.updateValueForGeneric(repairableName, true);
+            if (elementGui != null) elementGui.updateValueForGeneric(repairable, true);
             repairConfigGui.updateValueForGeneric(unit, true);
         }
 
@@ -112,38 +151,46 @@ public class UnitRepairApi {
      * @return true if successful.
      */
     public static boolean removeUnitRepair(@NotNull Material unit, @NotNull Material repairable) {
+        return removeUnitRepair(unit.getKey(), repairable.getKey());
+    }
+
+    /**
+     * Remove a custom anvil unit repair recipe.
+     *
+     * @param unit       The unit material used to repair the bellow item.
+     * @param repairable The item used to be repaired.
+     * @return true if successful.
+     */
+    public static boolean removeUnitRepair(@NotNull NamespacedKey unit, @NotNull NamespacedKey repairable) {
         // Delete every possible variation and save to file
-        String unitName = unit.name();
-        String repairableName = repairable.name();
 
         FileConfiguration config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
-        config.set(unitName.toLowerCase() + "." + repairableName.toUpperCase(), null);
+        config.set(unit.getKey() + "." + repairable.getKey(), null);
-        config.set(unitName.toUpperCase() + "." + repairableName.toLowerCase(), null);
+        config.set(unit.getKey() + "." + repairable, null);
-        config.set(unitName.toUpperCase() + "." + repairableName.toUpperCase(), null);
+        config.set(unit + "." + repairable.getKey(), null);
-        config.set(unitName.toLowerCase() + "." + repairableName.toLowerCase(), null);
+        config.set(unit + "." + repairable, null);
 
         // Test if it was the last value of this section
         boolean lastValue = false;
-        if(config.isConfigurationSection(unitName.toLowerCase())) {
+        if (config.isConfigurationSection(unit.toString())) {
-            ConfigurationSection section = config.getConfigurationSection(unitName.toLowerCase());
+            ConfigurationSection section = config.getConfigurationSection(unit.toString());
 
             if (section != null && section.getKeys(false).isEmpty()) {
                 lastValue = true;
-                config.set(unitName.toLowerCase(), null);
+                config.set(unit.toString(), null);
             }
 
-        } else if (config.isConfigurationSection(unitName.toUpperCase())) {
+        } else if (config.isConfigurationSection(unit.getKey())) {
-            ConfigurationSection section = config.getConfigurationSection(unitName.toUpperCase());
+            ConfigurationSection section = config.getConfigurationSection(unit.getKey());
             if (section != null && section.getKeys(false).isEmpty()) {
                 lastValue = true;
-                config.set(unitName.toUpperCase(), null);
+                config.set(unit.getKey(), null);
             }
 
         } else lastValue = true;
 
 
-        // We only need to "delete" as the lower case to be counted as deleted
+        ConfigHolder.UNIT_REPAIR_HOLDER.delete(unit.toString().toLowerCase() + "." + repairable.toString().toLowerCase());
-        ConfigHolder.UNIT_REPAIR_HOLDER.delete(unitName.toLowerCase() + "." + repairableName.toLowerCase());
         prepareSaveTask();
 
         // Remove from gui
@@ -151,7 +198,7 @@ public class UnitRepairApi {
         if (repairConfigGui != null) {
             UnitRepairElementListGui elementGui = repairConfigGui.getInstanceOrCreate(unit).getStored();
 
-            if(elementGui != null) elementGui.removeGeneric(repairableName);
+            if (elementGui != null) elementGui.removeGeneric(repairable);
             if (lastValue) {
                 repairConfigGui.removeGeneric(unit);
             }
@@ -174,6 +221,7 @@ public class UnitRepairApi {
 
     /**
      * Get every unit repair recipes.
+     *
      * @return An immutable collection of unit repair recipes.
      * <p>
      * Each element of the provided triple represent a part of the recipe
@@ -182,7 +230,10 @@ public class UnitRepairApi {
      *    <li>Second object is the item to be repaired.
      *    <li>Last object is the amount to be repaired by every unit. (1% = 0.01)
      * </ul>
+     * @deprecated some may be missing. use {@link #getModernUnitRepairs()}
+     *
      */
+    @Deprecated
     @NotNull
     public static List<Triple<Material, Material, Double>> getUnitRepairs() {
         List<Triple<Material, Material, Double>> mutableList = new ArrayList<>();
@@ -215,4 +266,53 @@ public class UnitRepairApi {
         return Collections.unmodifiableList(mutableList);
     }
 
+    /**
+     * Get every unit repair recipes.
+     *
+     * @return An immutable collection of unit repair recipes.
+     * <p>
+     * <li>First map contain a key the unit material used to repair the bellow item and value the second map
+     * <li>Second map contain as key the item to be repaired and as value the amount to be repaired by every unit. (1% = 0.01)
+     * </ul>
+     */
+    @NotNull
+    public static Map<NamespacedKey, Map<NamespacedKey, Double>> getModernUnitRepairs() {
+        Map<NamespacedKey, Map<NamespacedKey, Double>> mutableList = new HashMap<>();
+
+        FileConfiguration config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
+        for (String unitKey : config.getKeys(false)) {
+            // Test if config section exist
+            if (!config.isConfigurationSection(unitKey)) continue;
+
+            // Test if unit is a material
+            NamespacedKey unit = NamespacedKey.fromString(unitKey.toLowerCase());
+            if (unit == null) continue;
+
+            Map<NamespacedKey, Double> map;
+            if (!mutableList.containsKey(unit)) {
+                map = new HashMap<>();
+                mutableList.put(unit, map);
+            } else {
+                map = mutableList.get(unit);
+            }
+
+            // Iterate over reparable items
+            ConfigurationSection section = config.getConfigurationSection(unitKey);
+            for (String repairableKey : section.getKeys(false)) {
+                // Test if value section exist
+                if (!section.isDouble(repairableKey)) continue;
+
+                // Test if repairable is valid a material
+                NamespacedKey repairable = NamespacedKey.fromString(repairableKey.toLowerCase());
+                if (repairable == null) continue;
+
+                // Add the values
+                map.put(repairable, section.getDouble(repairableKey));
+
+            }
+        }
+
+        return mutableList;
+    }
+
 }

src/main/java/xyz/alexcrea/cuanvil/gui/config/global/AbstractEnchantConfigGui.java

@@ -5,7 +5,6 @@ import com.github.stefvanschie.inventoryframework.pane.util.Pattern;
 import org.bukkit.event.inventory.InventoryClickEvent;
 import xyz.alexcrea.cuanvil.enchant.CAEnchantment;
 import xyz.alexcrea.cuanvil.enchant.CAEnchantmentRegistry;
-import xyz.alexcrea.cuanvil.gui.ValueUpdatableGui;
 import xyz.alexcrea.cuanvil.gui.config.list.SettingGuiListConfigGui;
 import xyz.alexcrea.cuanvil.gui.config.settings.SettingGui;
 import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
@@ -37,7 +36,7 @@ public abstract class AbstractEnchantConfigGui<T extends SettingGui.SettingGuiFa
     }
 
     @Override
-    protected Collection<CAEnchantment> getEveryDisplayableInstanceOfGeneric() {
+    protected Collection<CAEnchantment> getEveryInstanceOfGeneric() {
         return CAEnchantmentRegistry.getInstance().getNameSortedEnchantments();
     }
 
@@ -73,7 +72,7 @@ public abstract class AbstractEnchantConfigGui<T extends SettingGui.SettingGuiFa
             this.pages.clear();
             this.pages.add(this.firstPage);
 
-            for (CAEnchantment enchantment : getEveryDisplayableInstanceOfGeneric()) {
+            for (CAEnchantment enchantment : getDisplayableInstanceOfGeneric()) {
                 GuiItem item = this.guiItemMap.get(enchantment);
 
                 if(item == null) {

src/main/java/xyz/alexcrea/cuanvil/gui/config/global/CustomRecipeConfigGui.java

@@ -113,7 +113,7 @@ public class CustomRecipeConfigGui extends MappedGuiListConfigGui<AnvilCustomRec
 
 
     @Override
-    protected Collection<AnvilCustomRecipe> getEveryDisplayableInstanceOfGeneric() {
+    protected Collection<AnvilCustomRecipe> getEveryInstanceOfGeneric() {
         return ConfigHolder.CUSTOM_RECIPE_HOLDER.getRecipeManager().getRecipeList();
     }
 }

src/main/java/xyz/alexcrea/cuanvil/gui/config/global/EnchantConflictGui.java

@@ -1,6 +1,7 @@
 package xyz.alexcrea.cuanvil.gui.config.global;
 
 import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
 import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.inventory.ItemFlag;
 import org.bukkit.inventory.ItemStack;
@@ -35,6 +36,10 @@ public class EnchantConflictGui extends MappedGuiListConfigGui<EnchantConflictGr
         return INSTANCE;
     }
 
+    // Need to init myself
+    public EnchantConflictGui(Gui parent) {
+        super("Conflict Config", parent);
+    }
 
     private EnchantConflictGui() {
         super("Conflict Config");
@@ -97,7 +102,7 @@ public class EnchantConflictGui extends MappedGuiListConfigGui<EnchantConflictGr
     }
 
     @Override
-    protected Collection<EnchantConflictGroup> getEveryDisplayableInstanceOfGeneric() {
+    protected Collection<EnchantConflictGroup> getEveryInstanceOfGeneric() {
         return ConfigHolder.CONFLICT_HOLDER.getConflictManager().getConflictList();
     }
 

src/main/java/xyz/alexcrea/cuanvil/gui/config/global/GroupConfigGui.java

@@ -1,6 +1,7 @@
 package xyz.alexcrea.cuanvil.gui.config.global;
 
 import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.inventory.ItemFlag;
 import org.bukkit.inventory.ItemStack;
@@ -43,6 +44,10 @@ public class GroupConfigGui extends MappedGuiListConfigGui<IncludeGroup, MappedG
         init();
     }
 
+    public GroupConfigGui(Gui parent) {
+        super("Group Config", parent);
+    }
+
     @Override
     protected ItemStack createItemForGeneric(IncludeGroup group) {
         ItemStack item = new ItemStack(group.getRepresentativeMaterial());
@@ -62,7 +67,7 @@ public class GroupConfigGui extends MappedGuiListConfigGui<IncludeGroup, MappedG
     }
 
     @Override
-    protected Collection<IncludeGroup> getEveryDisplayableInstanceOfGeneric() {
+    protected Collection<IncludeGroup> getEveryInstanceOfGeneric() {
         ArrayList<IncludeGroup> includeGroups = new ArrayList<>();
 
         for (AbstractMaterialGroup group : ConfigHolder.ITEM_GROUP_HOLDER.getItemGroupsManager().getGroupMap().values()) {

src/main/java/xyz/alexcrea/cuanvil/gui/config/global/ItemConfigGui.java

@@ -0,0 +1,133 @@
+package xyz.alexcrea.cuanvil.gui.config.global;
+
+import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import com.github.stefvanschie.inventoryframework.gui.type.ChestGui;
+import com.github.stefvanschie.inventoryframework.pane.PatternPane;
+import com.github.stefvanschie.inventoryframework.pane.util.Pattern;
+import io.delilaheve.CustomAnvil;
+import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
+import org.bukkit.inventory.ItemStack;
+import org.bukkit.inventory.meta.ItemMeta;
+import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions;
+import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
+
+import java.util.Collections;
+
+public class ItemConfigGui extends ChestGui {
+
+    public EnchantConflictGui enchantConflictGui;
+    public GroupConfigGui groupConfigGui;
+    public UnitRepairConfigGui unitRepairConfigGui;
+    public CustomRecipeConfigGui customRecipeConfigGui;
+
+    public ItemConfigGui(Material display, NamespacedKey material) {
+        super(3, material.getKey() + " Config", CustomAnvil.instance);
+
+        Pattern pattern = new Pattern(
+                "0000D0000",
+                "056000780",
+                "000000000"
+        );
+        PatternPane pane = new PatternPane(0, 0, 9, 3, pattern);
+        addPane(pane);
+
+        GuiGlobalItems.addBackgroundItem(pane);
+
+        ItemStack displayItemstack = new ItemStack(display);
+        ItemMeta displayMeta = displayItemstack.getItemMeta();
+        assert displayMeta != null;
+
+        displayMeta.setDisplayName("§aConfiguring " + material);
+        displayItemstack.setItemMeta(displayMeta);
+        pane.bindItem('D',  new GuiItem(displayItemstack, GuiGlobalActions.stayInPlace, CustomAnvil.instance));
+
+        // Enchantment Conflicts item
+        ItemStack enchantConflictItemstack = new ItemStack(Material.OAK_FENCE);
+        ItemMeta enchantConflictMeta = enchantConflictItemstack.getItemMeta();
+        assert enchantConflictMeta != null;
+
+        enchantConflictMeta.setDisplayName("§aEnchantment Conflict");
+        enchantConflictMeta.setLore(Collections.singletonList("§7Click here to open enchantment conflict menu"));
+        enchantConflictItemstack.setItemMeta(enchantConflictMeta);
+
+        GuiItem enchantConflictItem = GuiGlobalItems.goToGuiItem(enchantConflictItemstack, getEnchantConflictGui(material));
+        pane.bindItem('5', enchantConflictItem);
+
+        // Group config items
+        ItemStack groupItemstack = new ItemStack(Material.CHEST);
+        ItemMeta groupMeta = groupItemstack.getItemMeta();
+        assert groupMeta != null;
+
+        groupMeta.setDisplayName("§aItem Groups");
+        groupMeta.setLore(Collections.singletonList("§7Click here to open item group menu"));
+        groupItemstack.setItemMeta(groupMeta);
+
+        GuiItem groupConfigItem = GuiGlobalItems.goToGuiItem(groupItemstack, getGroupConfigGui(material));
+
+        pane.bindItem('6', groupConfigItem);
+
+        // Unit repair item
+        ItemStack unirRepairItemstack = new ItemStack(Material.DIAMOND);
+        ItemMeta unitRepairMeta = unirRepairItemstack.getItemMeta();
+        assert unitRepairMeta != null;
+
+        unitRepairMeta.setDisplayName("§aUnit Repair");
+        unitRepairMeta.setLore(Collections.singletonList("§7Click here to open anvil unit repair menu"));
+        unirRepairItemstack.setItemMeta(unitRepairMeta);
+
+        //TODO
+        GuiItem unitRepairItem = GuiGlobalItems.goToGuiItem(unirRepairItemstack, UnitRepairConfigGui.getInstance());
+        pane.bindItem('7', unitRepairItem);
+
+        // Custom recipe item
+        ItemStack customRecipeItemstack = new ItemStack(Material.CRAFTING_TABLE);
+        ItemMeta customRecipeMeta = customRecipeItemstack.getItemMeta();
+        assert customRecipeMeta != null;
+
+        customRecipeMeta.setDisplayName("§aCustom recipes");
+        customRecipeMeta.setLore(Collections.singletonList("§7Click here to open anvil custom recipe menu"));
+        customRecipeItemstack.setItemMeta(customRecipeMeta);
+
+        //TODO
+        GuiItem customRecipeItem = GuiGlobalItems.goToGuiItem(customRecipeItemstack, CustomRecipeConfigGui.getInstance());
+        pane.bindItem('8', customRecipeItem);
+    }
+
+    private EnchantConflictGui getEnchantConflictGui(NamespacedKey material) {
+        if (enchantConflictGui == null) {
+            enchantConflictGui = new EnchantConflictGui(this);
+            enchantConflictGui.setFilter(group ->
+                    group.getCantConflictGroup().contain(material)
+            );
+            enchantConflictGui.init();
+        }
+
+        return enchantConflictGui;
+    }
+
+    private GroupConfigGui getGroupConfigGui(NamespacedKey material) {
+        if (groupConfigGui == null) {
+            groupConfigGui = new GroupConfigGui(this);
+            groupConfigGui.setFilter(group ->
+                    group.contain(material)
+            );
+            groupConfigGui.init();
+        }
+
+        return groupConfigGui;
+    }
+
+    /*private UnitRepairConfigGui getUnitRepairConfigGui(NamespacedKey material) {
+        if (unitRepairConfigGui == null) {
+            unitRepairConfigGui = new UnitRepairConfigGui(this);
+            unitRepairConfigGui.setFilter(otherMat ->
+                    group.contain(material) //TODO check material & what inside
+            );
+            unitRepairConfigGui.init();
+        }
+
+        return unitRepairConfigGui;
+    }*/
+
+}

src/main/java/xyz/alexcrea/cuanvil/gui/config/global/UnitRepairConfigGui.java

@@ -1,9 +1,10 @@
 package xyz.alexcrea.cuanvil.gui.config.global;
 
 import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
 import io.delilaheve.CustomAnvil;
 import org.bukkit.Material;
-import org.bukkit.configuration.ConfigurationSection;
+import org.bukkit.NamespacedKey;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.jetbrains.annotations.NotNull;
@@ -13,13 +14,14 @@ import xyz.alexcrea.cuanvil.gui.config.ask.SelectItemTypeGui;
 import xyz.alexcrea.cuanvil.gui.config.list.MappedGuiListConfigGui;
 import xyz.alexcrea.cuanvil.gui.config.list.UnitRepairElementListGui;
 import xyz.alexcrea.cuanvil.util.CasedStringUtil;
+import xyz.alexcrea.cuanvil.util.MaterialUtil;
 
-import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
+import java.util.HashSet;
 
 public class UnitRepairConfigGui extends
-        MappedGuiListConfigGui<Material, MappedGuiListConfigGui.LazyElement<UnitRepairElementListGui>> {
+        MappedGuiListConfigGui<NamespacedKey, MappedGuiListConfigGui.LazyElement<UnitRepairElementListGui>> {
 
     private static UnitRepairConfigGui INSTANCE;
 
@@ -41,8 +43,12 @@ public class UnitRepairConfigGui extends
         init();
     }
 
+    public UnitRepairConfigGui(Gui parent) {
+        super("Unit Repair Config", parent);
+    }
+
     @Override
-    protected LazyElement<UnitRepairElementListGui> newInstanceOfGui(Material material, GuiItem item) {
+    protected LazyElement<UnitRepairElementListGui> newInstanceOfGui(NamespacedKey material, GuiItem item) {
         return new LazyElement<>(item, () -> {
             UnitRepairElementListGui element = new UnitRepairElementListGui(material, this);
             element.init();
@@ -51,17 +57,28 @@ public class UnitRepairConfigGui extends
     }
 
     @Override
-    protected ItemStack createItemForGeneric(Material material) {
+    protected ItemStack createItemForGeneric(@NotNull NamespacedKey material) {
-        ConfigurationSection materialSection = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().getConfigurationSection(material.name().toLowerCase());
+        var unitConfig = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
-        String materialName = CasedStringUtil.snakeToUpperSpacedCase(material.name().toLowerCase());
+        var section = unitConfig.getConfigurationSection(material.toString().toLowerCase());
+        var legacySection = unitConfig.getConfigurationSection(material.toString().toLowerCase());
 
-        if(material.isAir()){
+        String materialName = CasedStringUtil.snakeToUpperSpacedCase(material.getKey().toLowerCase());
-            material = Material.BARRIER;
+
+        var display = MaterialUtil.INSTANCE.getMatFromKey(material);
+
+        if (display == null || display.isAir()) {
+            display = Material.BARRIER;
         }
 
-        int reparableItemCount = materialSection == null ? 0 : materialSection.getKeys(false).size(); // Probably an expensive call but... why not
+        var reparable = new HashSet<String>();
+        if (section != null)
+            reparable.addAll(section.getKeys(false));
+        if (legacySection != null)
+            reparable.addAll(legacySection.getKeys(false));
 
-        ItemStack item = new ItemStack(material);
+        var reparableItemCount = reparable.size();
+
+        ItemStack item = new ItemStack(display);
         ItemMeta meta = item.getItemMeta();
         assert meta != null;
 
@@ -77,13 +94,16 @@ public class UnitRepairConfigGui extends
     }
 
     @Override
-    protected Collection<Material> getEveryDisplayableInstanceOfGeneric() {
+    protected Collection<NamespacedKey> getEveryInstanceOfGeneric() {
-        ArrayList<Material> materials = new ArrayList<>();
+        var materials = new HashSet<NamespacedKey>();
+        var config = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig();
 
-        for (String matName : ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().getKeys(false)) {
+        for (String matName : config.getKeys(false)) {
-            Material mat = Material.getMaterial(matName.toUpperCase());
+            if(!config.isConfigurationSection(matName)) continue;
-            if(mat != null){
+
-                materials.add(mat);
+            NamespacedKey material = NamespacedKey.fromString(matName.toLowerCase());
+            if (material != null) {
+                materials.add(material);
             }
         }
         return materials;
@@ -113,7 +133,7 @@ public class UnitRepairConfigGui extends
                             "§7You like to be an unit repair item",
                     this,
                     (itemStack, player) -> {
-                        Material type = itemStack.getType();
+                        NamespacedKey type = MaterialUtil.INSTANCE.getCustomType(itemStack);
                         // Add new material
                         updateValueForGeneric(type, true);
 
@@ -126,7 +146,7 @@ public class UnitRepairConfigGui extends
     }
 
     @NotNull
-    public LazyElement<UnitRepairElementListGui> getInstanceOrCreate(Material mat){
+    public LazyElement<UnitRepairElementListGui> getInstanceOrCreate(NamespacedKey mat) {
         LazyElement<UnitRepairElementListGui> element = this.elementGuiMap.get(mat);
         if (element == null) {
             updateValueForGeneric(mat, false);
@@ -141,8 +161,10 @@ public class UnitRepairConfigGui extends
     protected String genericDisplayedName() {
         return "this function Should not be used.";
     }
+
     @Override // Not used in this implementation.
-    protected Material createAndSaveNewEmptyGeneric(String name) {
+    protected NamespacedKey createAndSaveNewEmptyGeneric(String name) {
         return null;
     }
+
 }

src/main/java/xyz/alexcrea/cuanvil/gui/config/list/ElementListConfigGui.java

@@ -23,18 +23,21 @@ import java.util.ArrayList;
 import java.util.Collection;
 import java.util.HashMap;
 import java.util.UUID;
+import java.util.function.Predicate;
 
 public abstract class ElementListConfigGui<T> extends ChestGui implements ValueUpdatableGui {
 
-    private final String namePrefix;
-
-    protected PatternPane backgroundPane;
-
     public static final int LIST_FILLER_START_X = 1;
     public static final int LIST_FILLER_START_Y = 1;
     public static final int LIST_FILLER_LENGTH = 7;
     public static final int LIST_FILLER_HEIGHT = 4;
 
+    private final String namePrefix;
+
+    protected PatternPane backgroundPane;
+
+    private Predicate<T> filter = (t) -> true;
+
     protected ElementListConfigGui(@NotNull String title, Gui parent) {
         super(6, title, CustomAnvil.instance);
         this.namePrefix = title;
@@ -46,6 +49,10 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
 
     }
 
+    public void setFilter(Predicate<T> filter) {
+        this.filter = filter;
+    }
+
     protected Pattern getBackgroundPattern() {
         return new Pattern(
                 GuiSharedConstant.UPPER_FILLER_FULL_PLANE,
@@ -61,7 +68,7 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
     protected ArrayList<OutlinePane> pages;
     protected HashMap<UUID, Integer> pageMap;
 
-    public void init() { // Why I'm using an init function ? //TODO determine why is it using a init function and not used on constructor.
+    public void init() {
         GuiGlobalItems.addBackgroundItem(this.backgroundPane);
         this.backgroundPane.bindItem('1', GuiSharedConstant.SECONDARY_BACKGROUND_ITEM);
         addPane(this.backgroundPane);
@@ -117,12 +124,13 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
             this.backgroundPane.bindItem('C', createNew);
         }
     }
+
     protected void reloadValues() {
         this.firstPage.clear();
         this.pages.clear();
         this.pages.add(this.firstPage);
 
-        for (T generic : getEveryDisplayableInstanceOfGeneric()) {
+        for (T generic : getDisplayableInstanceOfGeneric()) {
             updateValueForGeneric(generic, false);
         }
 
@@ -300,7 +308,11 @@ public abstract class ElementListConfigGui< T > extends ChestGui implements Valu
 
     protected abstract void updateGeneric(T generic, ItemStack usedItem);
 
-    protected abstract Collection<T> getEveryDisplayableInstanceOfGeneric();
+    protected abstract Collection<T> getEveryInstanceOfGeneric();
+
+    protected Collection<T> getDisplayableInstanceOfGeneric() {
+        return getEveryInstanceOfGeneric().stream().filter(filter).toList();
+    }
 
     @Override
     public void updateGuiValues() {

src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedElementListConfigGui.java

@@ -1,6 +1,7 @@
 package xyz.alexcrea.cuanvil.gui.config.list;
 
 import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
 import io.delilaheve.CustomAnvil;
 import org.bukkit.Material;
 import org.bukkit.entity.HumanEntity;
@@ -17,12 +18,17 @@ import java.util.function.Consumer;
 public abstract class MappedElementListConfigGui<T, S> extends ElementListConfigGui<T> {
 
     protected final HashMap<T, S> elementGuiMap;
-    protected MappedElementListConfigGui(@NotNull String title) {
+
-        super(title, MainConfigGui.getInstance());
+    protected MappedElementListConfigGui(@NotNull String title, @NotNull Gui parent) {
+        super(title, parent);
         this.elementGuiMap = new HashMap<>();
 
     }
 
+    protected MappedElementListConfigGui(@NotNull String title) {
+        this(title, MainConfigGui.getInstance());
+    }
+
     @Override
     protected GuiItem prepareCreateNewItem() {
         // Create new conflict item

src/main/java/xyz/alexcrea/cuanvil/gui/config/list/MappedGuiListConfigGui.java

@@ -1,6 +1,7 @@
 package xyz.alexcrea.cuanvil.gui.config.list;
 
 import com.github.stefvanschie.inventoryframework.gui.GuiItem;
+import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
 import io.delilaheve.CustomAnvil;
 import org.bukkit.entity.HumanEntity;
 import org.bukkit.event.inventory.InventoryClickEvent;
@@ -19,7 +20,10 @@ public abstract class MappedGuiListConfigGui< T, S extends MappedGuiListConfigGu
 
     protected MappedGuiListConfigGui(@NotNull String title) {
         super(title);
+    }
 
+    protected MappedGuiListConfigGui(@NotNull String title, @NotNull Gui parent) {
+        super(title, parent);
     }
 
     @Override
@@ -80,7 +84,7 @@ public abstract class MappedGuiListConfigGui< T, S extends MappedGuiListConfigGu
 
             // Try to find if it already exists in a for loop
             // Not the most efficient on large number of conflict, but it should not run often.
-            for (T generic : getEveryDisplayableInstanceOfGeneric()) {
+            for (T generic : getDisplayableInstanceOfGeneric()) {
                 if (generic.toString().equalsIgnoreCase(message)) {
                     player.sendMessage("§cPlease enter a " + genericDisplayedName() + " name that do not already exist...");
                     // wait next message.
@@ -117,6 +121,7 @@ public abstract class MappedGuiListConfigGui< T, S extends MappedGuiListConfigGu
 
         private final GuiItem parentItem;
         private final LazyValue<Consumer<InventoryClickEvent>> lazyOpenConsumer;
+
         public LazyElement(GuiItem parentItem, Supplier<T> valueSupplier) {
             super(valueSupplier);
             this.parentItem = parentItem;

src/main/java/xyz/alexcrea/cuanvil/gui/config/list/UnitRepairElementListGui.java

@@ -3,12 +3,14 @@ package xyz.alexcrea.cuanvil.gui.config.list;
 import com.github.stefvanschie.inventoryframework.gui.GuiItem;
 import com.github.stefvanschie.inventoryframework.gui.type.util.Gui;
 import org.bukkit.Material;
+import org.bukkit.NamespacedKey;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.entity.HumanEntity;
 import org.bukkit.event.inventory.InventoryClickEvent;
 import org.bukkit.inventory.meta.Damageable;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 import xyz.alexcrea.cuanvil.config.ConfigHolder;
 import xyz.alexcrea.cuanvil.gui.config.ask.SelectItemTypeGui;
 import xyz.alexcrea.cuanvil.gui.config.global.UnitRepairConfigGui;
@@ -17,26 +19,25 @@ import xyz.alexcrea.cuanvil.gui.config.settings.DoubleSettingGui;
 import xyz.alexcrea.cuanvil.gui.util.GuiGlobalItems;
 import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
 import xyz.alexcrea.cuanvil.util.CasedStringUtil;
+import xyz.alexcrea.cuanvil.util.MaterialUtil;
 
-import java.util.ArrayList;
+import java.util.*;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.List;
 import java.util.function.Consumer;
 
-public class UnitRepairElementListGui extends SettingGuiListConfigGui<String, DoubleSettingGui.DoubleSettingFactory> implements ElementMappedToListGui {
+public class UnitRepairElementListGui extends SettingGuiListConfigGui<NamespacedKey, DoubleSettingGui.DoubleSettingFactory> implements ElementMappedToListGui {
 
-    private final Material parentMaterial;
+    private final NamespacedKey parentMaterial;
     private final UnitRepairConfigGui parentGui;
     private final String materialName;
 
     private boolean shouldWork = true;
-    public UnitRepairElementListGui(@NotNull Material parentMaterial,
+
+    public UnitRepairElementListGui(@NotNull NamespacedKey parentMaterial,
                                     @NotNull UnitRepairConfigGui parentGui) {
-        super("§e" + CasedStringUtil.snakeToUpperSpacedCase(parentMaterial.name().toLowerCase()) + " §rUnit repair");
+        super("§e" + CasedStringUtil.snakeToUpperSpacedCase(parentMaterial.getKey().toLowerCase()) + " §rUnit repair");
         this.parentMaterial = parentMaterial;
         this.parentGui = parentGui;
-        this.materialName = CasedStringUtil.snakeToUpperSpacedCase(parentMaterial.name().toLowerCase());
+        this.materialName = CasedStringUtil.snakeToUpperSpacedCase(parentMaterial.getKey().toLowerCase());
 
         GuiGlobalItems.addBackItem(this.backgroundPane, parentGui);
     }
@@ -66,28 +67,28 @@ public class UnitRepairElementListGui extends SettingGuiListConfigGui<String, Do
                     this,
                     (itemStack, player) -> {
                         ItemMeta meta = itemStack.getItemMeta();
-                        Material type = itemStack.getType();
+                        NamespacedKey type = MaterialUtil.INSTANCE.getCustomType(itemStack);
 
-                        if(!(meta instanceof Damageable) || (type.getMaxDurability() <= 0)) {
+                        if (!(meta instanceof Damageable)) {
                             player.sendMessage("§cThis item can't be damaged, so it can't be repaired.");
                             return;
                         }
-                        if(type == this.parentMaterial){
+                        if (type.equals(this.parentMaterial)) {
                             player.sendMessage("§cItem can't repair something of the same type.");
                             return;
                         }
 
-                        String materialName = type.name().toLowerCase();
+                        String materialName = type.toString();
 
                         // Add new material
-                        ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().set(parentMaterial.name().toLowerCase() + "." + materialName,0.25);
+                        ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().set(parentMaterial.toString().toLowerCase() + "." + materialName, 0.25);
 
                         if (GuiSharedConstant.TEMPORARY_DO_SAVE_TO_DISK_EVERY_CHANGE) {
                             ConfigHolder.UNIT_REPAIR_HOLDER.saveToDisk(GuiSharedConstant.TEMPORARY_DO_BACKUP_EVERY_SAVE);
                         }
 
                         // Update gui
-                        updateValueForGeneric(materialName, true);
+                        updateValueForGeneric(type, true);
                         this.parentGui.updateValueForGeneric(this.parentMaterial, true);
 
 
@@ -106,14 +107,14 @@ public class UnitRepairElementListGui extends SettingGuiListConfigGui<String, Do
     }
 
     @Override
-    protected DoubleSettingGui.DoubleSettingFactory createFactory(String materialName) {
+    protected DoubleSettingGui.DoubleSettingFactory createFactory(NamespacedKey materialName) {
-        String materialDisplayName = CasedStringUtil.snakeToUpperSpacedCase(materialName);
+        String materialDisplayName = CasedStringUtil.snakeToUpperSpacedCase(materialName.getKey());
 
         return new DoubleSettingGui.DoubleSettingFactory(
                 "§0%§8" + materialDisplayName + " Repair",
                 this,
                 ConfigHolder.UNIT_REPAIR_HOLDER,
-                this.parentMaterial.name().toLowerCase()+"."+materialName,
+                this.parentMaterial.toString().toLowerCase() + "." + materialName,
                 Arrays.asList(
                         "§7Click here to change how many §e% §7of §a" + materialDisplayName,
                         "§7Should get repaired by §e" + this.materialName
@@ -128,28 +129,39 @@ public class UnitRepairElementListGui extends SettingGuiListConfigGui<String, Do
     }
 
     @Override
-    protected GuiItem itemFromFactory(String materialName, DoubleSettingGui.DoubleSettingFactory factory) {
+    protected GuiItem itemFromFactory(NamespacedKey materialName, DoubleSettingGui.DoubleSettingFactory factory) {
         return factory.getItem(materialFromName(materialName),
-                "§7%§a" + CasedStringUtil.snakeToUpperSpacedCase(materialName)+ " §erepaired by §a" + this.materialName);
+                "§7%§a" + CasedStringUtil.snakeToUpperSpacedCase(materialName.getKey()) + " §erepaired by §a" + this.materialName);
     }
 
     @Override
-    protected Collection<String> getEveryDisplayableInstanceOfGeneric() {
+    protected Collection<NamespacedKey> getEveryInstanceOfGeneric() {
-        ArrayList<String> keys = new ArrayList<>();
+        Set<NamespacedKey> keys = new HashSet<>();
         if (!this.shouldWork) {
             return keys;
         }
 
-        ConfigurationSection materialSection = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().getConfigurationSection(parentMaterial.name().toLowerCase());
+        ConfigurationSection legacySection = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().getConfigurationSection(parentMaterial.getKey().toLowerCase());
-        if(materialSection == null){
+        ConfigurationSection materialSection = ConfigHolder.UNIT_REPAIR_HOLDER.getConfig().getConfigurationSection(parentMaterial.toString().toLowerCase());
-            return keys;
+
-        }
+        addAllKeys(legacySection, keys);
-        keys.addAll(materialSection.getKeys(false));
+        addAllKeys(materialSection, keys);
+
         return keys;
     }
 
-    private Material materialFromName(String materialName){
+    private void addAllKeys(@Nullable ConfigurationSection section, @NotNull Set<NamespacedKey> keys) {
-        Material mat = Material.getMaterial(materialName.toUpperCase());
+        if (section == null) return;
+        for (var key : section.getKeys(false)) {
+            var material = NamespacedKey.fromString(key);
+            if (material == null) continue;
+
+            keys.add(material);
+        }
+    }
+
+    private Material materialFromName(NamespacedKey material) {
+        Material mat = MaterialUtil.INSTANCE.getMatFromKey(material);
         if (mat == null || mat.isAir()) return Material.BARRIER;
         return mat;
     }
@@ -163,7 +175,8 @@ public class UnitRepairElementListGui extends SettingGuiListConfigGui<String, Do
     // ElementMappedToListGui methods
 
     @Override // Not used in this implementation
-    public void updateLocal() {}
+    public void updateLocal() {
+    }
 
     @Override
     public void cleanAndBeUnusable() {

src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/EnchantSelectSettingGui.java

@@ -63,7 +63,7 @@ public class EnchantSelectSettingGui extends SettingGuiListConfigGui<CAEnchantme
     }
 
     @Override
-    protected Collection<CAEnchantment> getEveryDisplayableInstanceOfGeneric() {
+    protected Collection<CAEnchantment> getEveryInstanceOfGeneric() {
         Stream<CAEnchantment> toDisplayStream;
         if(this.displayUnselected){
             toDisplayStream = CAEnchantmentRegistry.getInstance().getNameSortedEnchantments().stream();

src/main/java/xyz/alexcrea/cuanvil/gui/config/settings/MaterialSelectSettingGui.java

@@ -217,7 +217,7 @@ public class MaterialSelectSettingGui extends MappedElementListConfigGui<Namespa
     }
 
     @Override
-    protected Collection<NamespacedKey> getEveryDisplayableInstanceOfGeneric() {
+    protected Collection<NamespacedKey> getEveryInstanceOfGeneric() {
         return this.defaultMaterials;
     }
 

src/main/java/xyz/alexcrea/cuanvil/update/minecraft/Update_1_21.java

@@ -1,6 +1,5 @@
 package xyz.alexcrea.cuanvil.update.minecraft;
 
-import io.delilaheve.CustomAnvil;
 import xyz.alexcrea.cuanvil.config.ConfigHolder;
 import xyz.alexcrea.cuanvil.update.UpdateUtils;
 import xyz.alexcrea.cuanvil.update.Version;

src/main/java/xyz/alexcrea/cuanvil/update/minecraft/Update_1_21_11.java

@@ -38,26 +38,26 @@ public class Update_1_21_11 extends MCUpdate{
         addAbsentToList(conflictConfig, "restriction_fire_aspect.notAffectedGroups", "spears");
 
         // Unit repair for spears
-        unitConfig.set("gold_ingot.golden_spear", 0.25);
+        unitConfig.set("minecraft:gold_ingot.minecraft:golden_spear", 0.25);
-        unitConfig.set("copper_ingot.copper_spear", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_spear", 0.25);
-        unitConfig.set("iron_ingot.iron_spear", 0.25);
+        unitConfig.set("minecraft:iron_ingot.minecraft:iron_spear", 0.25);
-        unitConfig.set("diamond.diamond_spear", 0.25);
+        unitConfig.set("minecraft:diamond.minecraft:diamond_spear", 0.25);
-        unitConfig.set("netherite_ingot.netherite_spear", 0.25);
+        unitConfig.set("minecraft:netherite_ingot.minecraft:netherite_spear", 0.25);
 
-        unitConfig.set("cobblestone.stone_spear", 0.25);
+        unitConfig.set("minecraft:cobblestone.stone_spear", 0.25);
-        unitConfig.set("cobbled_deepslate.stone_spear", 0.25);
+        unitConfig.set("minecraft:cobbled_deepslate.stone_spear", 0.25);
 
-        unitConfig.set("oak_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:oak_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("spruce_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:spruce_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("birch_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:birch_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("jungle_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:jungle_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("acacia_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:acacia_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("dark_oak_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:dark_oak_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("mangrove_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:mangrove_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("cherry_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:cherry_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("bamboo_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:bamboo_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("crimson_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:crimson_planks.minecraft:wooden_spear", 0.25);
-        unitConfig.set("warped_planks.wooden_spear", 0.25);
+        unitConfig.set("minecraft:warped_planks.minecraft:wooden_spear", 0.25);
 
         // Create lunge enchant value and group
         baseConfig.set("enchant_limits.minecraft:lunge", 3);

src/main/java/xyz/alexcrea/cuanvil/update/minecraft/Update_1_21_9.java

@@ -49,16 +49,16 @@ public class Update_1_21_9 extends MCUpdate{
 
     public static void addCopperUnitRepair(FileConfiguration unitConfig) {
         // Add unit repair
-        unitConfig.set("copper_ingot.copper_helmet", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_helmet", 0.25);
-        unitConfig.set("copper_ingot.copper_chestplate", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_chestplate", 0.25);
-        unitConfig.set("copper_ingot.copper_leggings", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_leggings", 0.25);
-        unitConfig.set("copper_ingot.copper_boots", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_boots", 0.25);
 
-        unitConfig.set("copper_ingot.copper_pickaxe", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_pickaxe", 0.25);
-        unitConfig.set("copper_ingot.copper_shovel", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_shovel", 0.25);
-        unitConfig.set("copper_ingot.copper_hoe", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_hoe", 0.25);
-        unitConfig.set("copper_ingot.copper_axe", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_axe", 0.25);
-        unitConfig.set("copper_ingot.copper_sword", 0.25);
+        unitConfig.set("minecraft:copper_ingot.minecraft:copper_sword", 0.25);
     }
 
 }

src/main/kotlin/xyz/alexcrea/cuanvil/command/CustomAnvilCommand.kt

@@ -20,6 +20,7 @@ class CustomAnvilCommand(plugin: CustomAnvil) : CommandExecutor, TabCompleter {
     private val helpCommand = HelpExecutor()
     private val commands = ImmutableMap.of(
         "gui", editConfigCommand,
+        "config", editConfigCommand,
         "reload", ReloadExecutor(),
         "diagnostic", DiagnosticExecutor(),
         "debug", DebugToggleExecutor(),
@@ -42,12 +43,16 @@ class CustomAnvilCommand(plugin: CustomAnvil) : CommandExecutor, TabCompleter {
     ): Boolean {
         // Find sub command to execute based on the provided command name
         val subcmd: CASubCommand?
+        val subcmdStr: String
+
         val newargs: Array<out String>
         if(args.isEmpty()) {
+            subcmdStr = "config"
             subcmd = editConfigCommand
             newargs = args
         }else {
-            subcmd = commands[args[0].lowercase()]
+            subcmdStr = args[0].lowercase()
+            subcmd = commands[subcmdStr]
             newargs = args.copyOfRange(1, args.size)
         }
 
@@ -57,7 +62,7 @@ class CustomAnvilCommand(plugin: CustomAnvil) : CommandExecutor, TabCompleter {
         }
 
         try {
-            return subcmd.executeCommand(sender, cmd, cmdstr, newargs)
+            return subcmd.executeCommand(sender, cmd, subcmdStr, newargs)
         } catch (e: Throwable) {
             MetricsUtil.trackError(e)
             sender.sendMessage("§cError running this command")

src/main/kotlin/xyz/alexcrea/cuanvil/command/EditConfigExecutor.kt

@@ -4,16 +4,32 @@ import io.delilaheve.CustomAnvil
 import org.bukkit.command.Command
 import org.bukkit.command.CommandSender
 import org.bukkit.entity.HumanEntity
+import xyz.alexcrea.cuanvil.api.EnchantmentApi
 import xyz.alexcrea.cuanvil.dependency.util.PlatformUtil
+import xyz.alexcrea.cuanvil.enchant.CAEnchantment
+import xyz.alexcrea.cuanvil.enchant.CAEnchantmentRegistry
 import xyz.alexcrea.cuanvil.gui.config.MainConfigGui
+import xyz.alexcrea.cuanvil.gui.config.global.ItemConfigGui
 import xyz.alexcrea.cuanvil.gui.util.GuiGlobalActions
+import xyz.alexcrea.cuanvil.util.MaterialUtil.customType
+import xyz.alexcrea.cuanvil.util.MaterialUtil.isAir
 
 class EditConfigExecutor : CASubCommand() {
 
-    override fun executeCommand(sender: CommandSender,
+    override fun allowed(sender: CommandSender): Boolean {
+        return sender.hasPermission(CustomAnvil.editConfigPermission)
+    }
+
+    override fun description(): String {
+        return "Gui to edit the plugin's config"
+    }
+
+    override fun executeCommand(
+        sender: CommandSender,
         cmd: Command,
         cmdstr: String,
-                                args: Array<out String>): Boolean {
+        args: Array<out String>
+    ): Boolean {
         if (sender !is HumanEntity) return false
 
         if (!allowed(sender)) {
@@ -25,21 +41,92 @@ class EditConfigExecutor: CASubCommand() {
             sender.sendMessage("§eIt is may come in a future version.")
             sender.sendMessage("")
             sender.sendMessage("§eCurrently you need to edit manually the config or copy from another server (spigot or better)")
-            sender.sendMessage("§eThen /anvilconfigreload after config file is edited")
+            sender.sendMessage("§eThen /ca reload after config file is edited")
             return false
         }
 
-        MainConfigGui.getInstance().show(sender)
+        if ("gui".equals(cmdstr, ignoreCase = true)) {
+            sender.sendMessage("§c/ca gui has been moved to /ca config")
+        }
+
+        if (args.isEmpty())
+            processOpen(sender)
+        else when (args[0].lowercase()) {
+            "open" -> processOpen(sender)
+            "enchant" -> processEnchant(sender, args)
+            "item" -> processItem(sender)
+            else -> sender.sendMessage("Unknown subcommand \"${args[0]}\"")
+        }
 
         return true
     }
 
-    override fun allowed(sender: CommandSender): Boolean {
+    private fun processEnchant(sender: HumanEntity, args: Array<out String>) {
-        return sender.hasPermission(CustomAnvil.editConfigPermission)
+        val enchantToFilter: Set<CAEnchantment>
+        if (args.size <= 1) {
+            val item = sender.inventory.itemInMainHand
+
+            enchantToFilter = EnchantmentApi.getEnchantments(item).keys
+            if (enchantToFilter.isEmpty()) {
+                sender.sendMessage("No enchantment found in the item you are holding")
+                return
+            }
+        } else {
+            enchantToFilter = HashSet(EnchantmentApi.getListByName(args[1].lowercase()))
+
+            if (enchantToFilter.isEmpty()) {
+                sender.sendMessage("No enchantment found with the name \"${args[1]}\"")
+                return
+            }
         }
 
-    override fun description(): String {
+        // TODO open enchantments edit gui
-        return "Gui to edit the plugin's config"
+
+    }
+
+    private fun processItem(sender: HumanEntity) {
+        val item = sender.inventory.itemInMainHand
+        if(item.isAir) {
+            sender.sendMessage("Cannot configure the item in hand")
+            return
+        }
+
+        ItemConfigGui(item.type, item.customType).show(sender)
+    }
+
+    private fun processOpen(sender: HumanEntity) {
+        MainConfigGui.getInstance().show(sender)
+    }
+
+    // ---------------
+    //  Tab completer
+    // ---------------
+
+    private fun allEnchantmentsByName(): Collection<String> {
+        val names = mutableSetOf<String>()
+        for (enchantment in CAEnchantmentRegistry.getInstance().values()) {
+            names.add(enchantment.name)
+            names.add(enchantment.key.toString())
+        }
+
+        return names
+    }
+
+    override fun tabCompleter(sender: CommandSender, args: Array<out String>, list: MutableList<String>) {
+        list.addAll(
+            when (args.size) {
+                1 -> listOf("item", "enchant", "open")
+                2 -> {
+                    when (args[0].lowercase()) {
+                        "enchant" -> allEnchantmentsByName()
+                        else -> listOf()
+                    }
+                }
+
+                else -> listOf()
+            }
+        )
+
     }
 
 }

src/main/kotlin/xyz/alexcrea/cuanvil/util/MaterialUtil.kt

@@ -6,6 +6,7 @@ import org.bukkit.NamespacedKey
 import org.bukkit.inventory.ItemStack
 import xyz.alexcrea.cuanvil.dependency.DependencyManager
 import xyz.alexcrea.cuanvil.dependency.plugins.EcoItemDependencyUtil
+import javax.annotation.Nullable
 
 object MaterialUtil {
 
@@ -35,11 +36,13 @@ object MaterialUtil {
             return this.type.key
         }
 
+    @Nullable
     private fun bukkitMaterialFromKey(key: NamespacedKey): Material? {
         //TODO on paper only transition Registry.MATERIAL.get(key)
         return Material.matchMaterial(key.toString())
     }
 
+    @Nullable
     fun getMatFromKey(key: NamespacedKey): Material? {
         if(DependencyManager.ecoEnchantCompatibility != null) {
             val result = EcoItemDependencyUtil.ecoItemMaterialFromKey(key)

src/main/kotlin/xyz/alexcrea/cuanvil/util/UnitRepairUtil.kt

@@ -1,6 +1,8 @@
 package xyz.alexcrea.cuanvil.util
 
+import org.bukkit.NamespacedKey
 import org.bukkit.configuration.ConfigurationSection
+import org.bukkit.configuration.file.FileConfiguration
 import org.bukkit.inventory.ItemStack
 import xyz.alexcrea.cuanvil.config.ConfigHolder
 import xyz.alexcrea.cuanvil.util.MaterialUtil.customType
@@ -11,7 +13,7 @@ object UnitRepairUtil {
     private const val DEFAULT_DEFAULT_UNIT_REPAIR = 0.25
 
     // Path to user default unit repair value
-    private const val UNIT_REPAIR_DEFAULT_PATH = "default_repair_amount"
+    public const val UNIT_REPAIR_DEFAULT_PATH = "default_repair_amount"
 
     /**
      * Get the % of repair by unit [other] will do to this [ItemStack].
@@ -22,40 +24,45 @@ object UnitRepairUtil {
     ): Double? {
         if (other == null) return null
         val config = ConfigHolder.UNIT_REPAIR_HOLDER.config
-        // Get configuration section if exist
-        val otherName = other.customType.key.lowercase()
-        var section = config.getConfigurationSection(otherName)
-        if (section == null) {
-            section = config.getConfigurationSection(otherName.uppercase())
-            if (section == null) return null
 
-        }
+        val result = findRepairValue(this, other, config) ?: return null
-        // Get repair amount
+
-        var userDefault = config.getDouble(UNIT_REPAIR_DEFAULT_PATH, DEFAULT_DEFAULT_UNIT_REPAIR)
+        if(result > 0) return result
-        if (userDefault <= 0) {
+
-            userDefault = DEFAULT_DEFAULT_UNIT_REPAIR
+        // Get default
+        val userDefault = config.getDouble(UNIT_REPAIR_DEFAULT_PATH, DEFAULT_DEFAULT_UNIT_REPAIR)
+        if (userDefault <= 0)
+            return DEFAULT_DEFAULT_UNIT_REPAIR
+        return userDefault
     }
 
-        return getRepairAmount(this, section, userDefault)
+    private fun findRepairValue(
+        self: ItemStack,
+        other: ItemStack,
+        config: FileConfiguration
+    ): Double? {
+        val material = other.customType
+        val selfType = self.customType
+
+        val result = checkSection(config, material.toString(), selfType)
+        if (result != null) return result
+
+        return checkSection(config, material.key, selfType)
     }
 
-    /**
+    fun checkSection(
-     * Get the item % repaired by this configuration section of a unit repair.
+        config: FileConfiguration,
-     * null if not found.
+        path: String,
-     * If value is set to less than or equal to 0 then it will be set to default
+        material: NamespacedKey
-     */
+    ): Double? {
-    private fun getRepairAmount(item: ItemStack, section: ConfigurationSection, default: Double): Double? {
+        val section = config.getConfigurationSection(path) ?: return null
-        val itemName = item.customType.key.lowercase()
+
-        val repairValue = if (section.isDouble(itemName)) {
+        if (section.isDouble(material.toString()))
-            section.getDouble(itemName)
+            return section.getDouble(material.toString())
-        } else if (section.isDouble(itemName.uppercase())) {
+        if (section.isDouble(material.key))
-            section.getDouble(itemName.uppercase())
+            return section.getDouble(material.key)
-        } else {
+
         return null
     }
-        if (repairValue <= 0)
-            return default
-        return repairValue
-    }
 
 }