alpha version up & cleanup code.

build.gradle.kts

@@ -4,7 +4,7 @@ plugins {
 }
 
 group = "xyz.alexcrea"
-version = "1.3.2-A1"
+version = "1.3.2-A2"
 
 repositories {
     mavenCentral()

src/main/java/xyz/alexcrea/cuanvil/config/ConfigHolder.java

@@ -49,12 +49,15 @@ public abstract class ConfigHolder {
     private static final File BACKUP_FOLDER = new File(CustomAnvil.instance.getDataFolder(), "backup");
 
     protected FileConfiguration configuration;
+
     protected ConfigHolder() {
 
     }
 
     public abstract boolean reloadFromDisk(boolean hardFail);
+
     public abstract void reload();
+
     public FileConfiguration getConfig() {
         return configuration;
     }
@@ -65,12 +68,15 @@ public abstract class ConfigHolder {
     protected String getConfigFileExtension() {
         return ".yml";
     }
+
     protected File getConfigFile() {
         return new File(CustomAnvil.instance.getDataFolder(), getConfigFileName() + getConfigFileExtension());
     }
+
     protected File getFirstBackup() {
         return new File(BACKUP_FOLDER, getConfigFileName() + "-first" + getConfigFileExtension());
     }
+
     protected File getLastBackup() {
         return new File(BACKUP_FOLDER, getConfigFileName() + "-latest" + getConfigFileExtension());
     }
@@ -150,7 +156,8 @@ public abstract class ConfigHolder {
         }
 
         @Override
-        public void reload() {}// Nothing to do
+        public void reload() {
+        }// Nothing to do
 
     }
 
@@ -158,6 +165,7 @@ public abstract class ConfigHolder {
     public abstract static class ResourceConfigHolder extends ConfigHolder {
 
         String resourceName;
+
         private ResourceConfigHolder(String resourceName) {
             this.resourceName = resourceName;
         }
@@ -184,6 +192,7 @@ public abstract class ConfigHolder {
         private final static String FILE_NAME = "item_groups";
 
         ItemGroupManager itemGroupsManager;
+
         private ItemGroupConfigHolder() {
             super(FILE_NAME);
         }
@@ -210,6 +219,7 @@ public abstract class ConfigHolder {
         private final static String FILE_NAME = "enchant_conflict";
 
         EnchantConflictManager conflictManager;
+
         private ConflictConfigHolder() {
             super(FILE_NAME);
         }
@@ -235,8 +245,10 @@ public abstract class ConfigHolder {
         private UnitRepairHolder() {
             super(ITEM_GROUP_FILE_NAME);
         }
+
         @Override
-        public void reload() {} // Do nothing
+        public void reload() {
+        } // Do nothing
 
     }
 

src/main/java/xyz/alexcrea/cuanvil/enchant/EnchantmentProperties.java

@@ -41,11 +41,10 @@ public enum EnchantmentProperties {
     SWEEPING(EnchantmentRarity.RARE),
     THORNS(EnchantmentRarity.VERY_RARE),
     UNBREAKING(EnchantmentRarity.UNCOMMON),
-    VANISHING_CURSE(EnchantmentRarity.VERY_RARE)
+    VANISHING_CURSE(EnchantmentRarity.VERY_RARE);
-
-    ;
 
     private final EnchantmentRarity rarity;
+
     EnchantmentProperties(EnchantmentRarity rarity) {
         this.rarity = rarity;
     }

src/main/java/xyz/alexcrea/cuanvil/enchant/EnchantmentRarity.java

@@ -7,9 +7,7 @@ public enum EnchantmentRarity {
     COMMON(1),
     UNCOMMON(2),
     RARE(4),
-    VERY_RARE(8)
+    VERY_RARE(8);
-
-    ;
 
     private final int itemValue;
     private final int bookValue;
@@ -18,6 +16,7 @@ public enum EnchantmentRarity {
         this.itemValue = itemValue;
         this.bookValue = bookValue;
     }
+
     EnchantmentRarity(int itemValue) {
         this(itemValue, Math.max(1, itemValue / 2));
     }

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

@@ -14,12 +14,14 @@ import java.util.List;
 
 /**
  * Abstract Global Config gui for enchantment setting configuration.
+ *
  * @param <T> Type of the factory of the type of setting the gui should edit.
  */
 public abstract class AbstractEnchantConfigGui<T extends AbstractSettingGui.SettingGuiFactory> extends ValueUpdatableGui {
 
     /**
      * Constructor for a gui displaying available enchantment to edit a enchantment setting.
+     *
      * @param title Title of the gui.
      */
     protected AbstractEnchantConfigGui(String title) {

src/main/java/xyz/alexcrea/cuanvil/gui/config/ConfirmActionGui.java

@@ -24,6 +24,7 @@ public class ConfirmActionGui extends ChestGui {
 
     private static final ItemStack CANCEL_ITEM;
     private static final ItemStack CONFIRM_ITEM;
+
     static {
         CANCEL_ITEM = new ItemStack(Material.RED_TERRACOTTA);
         ItemMeta meta = CANCEL_ITEM.getItemMeta();

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

@@ -31,6 +31,7 @@ import java.util.function.Consumer;
 public class EnchantConflictGui extends ChestGui {
 
     public final static EnchantConflictGui INSTANCE = new EnchantConflictGui();
+
     static {
         INSTANCE.init();
     }
@@ -80,6 +81,7 @@ public class EnchantConflictGui extends ChestGui {
 
     private GuiItem goLeftItem;
     private GuiItem goRightItem;
+
     private void prepareOtherValues() {
         // Left item creation for consumer & bind
         this.goLeftItem = new GuiItem(new ItemStack(Material.RED_TERRACOTTA), event -> {

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

@@ -46,7 +46,8 @@ public class EnchantCostConfigGui extends AbstractEnchantConfigGui<EnchantCostSe
         EnchantmentRarity rarity = EnchantmentRarity.NO_RARITY;
         try {
             rarity = EnchantmentProperties.valueOf(key.toUpperCase(Locale.ENGLISH)).getRarity();
-        }catch (IllegalArgumentException ignored){}
+        } catch (IllegalArgumentException ignored) {
+        }
 
         return EnchantCostSettingsGui.enchantCostFactory(prettyKey + " Level Cost", this,
                 SECTION_NAME + '.' + key, ConfigHolder.DEFAULT_CONFIG, 0, 255,

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

@@ -20,6 +20,7 @@ public class MainConfigGui extends ChestGui {
     static {
         INSTANCE.init();
     }
+
     private MainConfigGui() {
         super(3, "\u00A78Anvil Config", CustomAnvil.instance);
 

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

@@ -26,6 +26,7 @@ public abstract class AbstractSettingGui extends ChestGui {
 
     /**
      * Prepare necessary object for a setting gui.
+     *
      * @param rows   Number of row for this gui.
      * @param title  Title of this gui.
      * @param parent Parent gui to go back when completed.
@@ -37,6 +38,7 @@ public abstract class AbstractSettingGui extends ChestGui {
 
     /**
      * Prepare necessary object for a setting gui.
+     *
      * @param rows   Number of row for this gui.
      * @param title  Title of this gui.
      * @param parent Parent gui to go back when completed.
@@ -50,6 +52,7 @@ public abstract class AbstractSettingGui extends ChestGui {
 
     /**
      * Initialise and prepare value for this gui.
+     *
      * @param parent Parent gui to go back when completed.
      */
     private void initBase(ValueUpdatableGui parent) {
@@ -75,6 +78,7 @@ public abstract class AbstractSettingGui extends ChestGui {
 
     /**
      * Get main pane for this setting gui.
+     *
      * @return Main pattern pain of this gui.
      */
     protected PatternPane getPane() {
@@ -89,12 +93,14 @@ public abstract class AbstractSettingGui extends ChestGui {
      * <li><b>B</b>: "back to previous gui" button.</li>
      * <li><b>0</b>: default background item.</li>
      * </ul>
+     *
      * @return The gui's pattern.
      */
     protected abstract Pattern getGuiPattern();
 
     /**
      * Called when the associated setting need to be saved.
+     *
      * @return true if the save was successful. false otherwise
      */
     public abstract boolean onSave();
@@ -102,6 +108,7 @@ public abstract class AbstractSettingGui extends ChestGui {
     /**
      * If this function return true
      * the gui assume the associated setting can be saved.
+     *
      * @return true if there is a change to the setting. false otherwise
      */
     public abstract boolean hadChange();
@@ -117,6 +124,7 @@ public abstract class AbstractSettingGui extends ChestGui {
 
         /**
          * Constructor for settings gui factory
+         *
          * @param configPath Configuration path of this setting.
          * @param config     Configuration holder of this setting.
          */
@@ -141,6 +149,7 @@ public abstract class AbstractSettingGui extends ChestGui {
 
         /**
          * Create a gui using setting parameters and current setting value.
+         *
          * @return A new instance of the implemented setting gui.
          */
         public abstract AbstractSettingGui create();

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

@@ -29,6 +29,7 @@ public class BoolSettingsGui extends AbstractSettingGui{
 
     /**
      * Create a boolean setting config gui.
+     *
      * @param holder Configuration factory of this setting.
      * @param now    The defined value of this setting.
      */
@@ -139,6 +140,7 @@ public class BoolSettingsGui extends AbstractSettingGui{
 
     /**
      * Create a bool setting factory from setting's parameters.
+     *
      * @param title      The title of the gui.
      * @param parent     Parent gui to go back when completed.
      * @param configPath Configuration path of this setting.
@@ -159,11 +161,14 @@ public class BoolSettingsGui extends AbstractSettingGui{
      * A factory for a boolean setting gui that hold setting's information.
      */
     public static class BoolSettingFactory extends SettingGuiFactory {
-        @NotNull String title; ValueUpdatableGui parent;
+        @NotNull
+        String title;
+        ValueUpdatableGui parent;
         boolean defaultVal;
 
         /**
          * Constructor for a boolean setting gui factory.
+         *
          * @param title      The title of the gui.
          * @param parent     Parent gui to go back when completed.
          * @param configPath Configuration path of this setting.

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

@@ -34,6 +34,7 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
 
     /**
      * Create an enchantment cost setting config gui.
+     *
      * @param holder  Configuration factory of this setting.
      * @param nowItem The defined value of this setting item's value.
      */
@@ -81,7 +82,8 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
         bookMeta.setDisplayName("\u00A7aCost of an Enchantment by Book");
         bookMeta.setLore(Arrays.asList(
                 "\u00A77Cost per result item level of an sacrifice enchantment",
-                "\u00A77Only apply if sacrificed item \u00A7cis \u00A77a book"));bookItemstack.setItemMeta(bookMeta);
+                "\u00A77Only apply if sacrificed item \u00A7cis \u00A77a book"));
+        bookItemstack.setItemMeta(bookMeta);
 
         // sword display
         ItemStack swordItemstack = new ItemStack(Material.WOODEN_SWORD);
@@ -221,6 +223,7 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
 
     /**
      * Create an int setting factory from setting's parameters.
+     *
      * @param title          The title of the gui.
      * @param parent         Parent gui to go back when completed.
      * @param configPath     Configuration path of this setting.
@@ -255,6 +258,7 @@ public class EnchantCostSettingsGui extends IntSettingsGui {
 
         /**
          * Constructor for an enchantment cost setting gui factory.
+         *
          * @param title          The title of the gui.
          * @param parent         Parent gui to go back when completed.
          * @param configPath     Configuration path of this setting.

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

@@ -20,7 +20,10 @@ import xyz.alexcrea.cuanvil.gui.config.SelectGroupContainer;
 import xyz.alexcrea.cuanvil.gui.util.GuiSharedConstant;
 import xyz.alexcrea.cuanvil.util.CasedStringUtil;
 
-import java.util.*;
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
 import java.util.function.Consumer;
 
 public class GroupSelectSettingGui extends AbstractSettingGui {

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

@@ -32,6 +32,7 @@ public class IntSettingsGui extends AbstractSettingGui{
 
     /**
      * Create an int setting config gui.
+     *
      * @param holder Configuration factory of this setting.
      * @param now    The defined value of this setting.
      */
@@ -59,6 +60,7 @@ public class IntSettingsGui extends AbstractSettingGui{
     }
 
     protected GuiItem returnToDefault;
+
     /**
      * Prepare "return to default value" gui item.
      */
@@ -186,6 +188,7 @@ public class IntSettingsGui extends AbstractSettingGui{
 
     /**
      * Step use lower case character from 'a' to a certain char.
+     *
      * @return The middle value of the character set for steps.
      */
     protected char getMidStepChar() {
@@ -194,6 +197,7 @@ public class IntSettingsGui extends AbstractSettingGui{
 
     /**
      * Create a step item from a step value.
+     *
      * @param stepIndex the index of the step item.
      * @return A step item corresponding to its index value.
      */
@@ -261,6 +265,7 @@ public class IntSettingsGui extends AbstractSettingGui{
 
     /**
      * Create an int setting factory from setting's parameters.
+     *
      * @param title      The title of the gui.
      * @param parent     Parent gui to go back when completed.
      * @param configPath Configuration path of this setting.
@@ -287,11 +292,17 @@ public class IntSettingsGui extends AbstractSettingGui{
      * A factory for an int setting gui that hold setting's information.
      */
     public static class IntSettingFactory extends SettingGuiFactory {
-        @NotNull String title; ValueUpdatableGui parent;
+        @NotNull
-        int min; int max; int defaultVal; int[] steps;
+        String title;
+        ValueUpdatableGui parent;
+        int min;
+        int max;
+        int defaultVal;
+        int[] steps;
 
         /**
          * Constructor for an int setting gui factory.
+         *
          * @param title      The title of the gui.
          * @param parent     Parent gui to go back when completed.
          * @param configPath Configuration path of this setting.

src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalActions.java

@@ -27,6 +27,7 @@ public class GuiGlobalActions {
     /**
      * Create a consumer to create and open a new GUI.
      * Used with InventoryClickEvent as the consumer argument as it is planned to be used on click on an GuiItem.
+     *
      * @param clazz    The class of the gui to open.
      *                 It is assumed this class contain a constructor requiring arguments of argClass in the same order as argClass array.
      * @param argClass Classes of the argument that will be passed to the constructor of the GUI class.
@@ -62,6 +63,7 @@ public class GuiGlobalActions {
     /**
      * Create a consumer to create and open a new GUI.
      * Used with InventoryClickEvent as the consumer argument as it is planned to be used on click on an GuiItem.
+     *
      * @param clazz The class of the gui to open.
      *              It is assumed this class contain a constructor with no argument.
      * @return A consumer to create a new gui and open it.
@@ -74,6 +76,7 @@ public class GuiGlobalActions {
     /**
      * Create a consumer to open a setting gui from a setting GUI factory.
      * Used with InventoryClickEvent as the consumer argument as it is planned to be used on click on an GuiItem.
+     *
      * @param factory The setting gui factory.
      * @return A consumer to create and open a new setting GUI.
      */
@@ -88,6 +91,7 @@ public class GuiGlobalActions {
     /**
      * Create a consumer to open a global GUI.
      * Used with InventoryClickEvent as the consumer argument as it is planned to be used on click on an GuiItem.
+     *
      * @param goal The gui to open when consumer is run.
      * @return A consumer to open a global GUI.
      */
@@ -109,6 +113,7 @@ public class GuiGlobalActions {
      * Create a consumer to update and open an updatable GUI.
      * Used with InventoryClickEvent as the consumer argument as it is planned to be used on click on an GuiItem.
      * This consumer check if the player who interacted with the item have the permission to save before saving.
+     *
      * @param setting The gui that contain the modified setting.
      * @param goal    The gui to update and open when consumer is run.
      * @return A consumer to open a global GUI.

src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiGlobalItems.java

@@ -23,6 +23,7 @@ public class GuiGlobalItems {
 
     // statically create default back itemstack
     private static final ItemStack BACK_ITEM;
+
     static {
         BACK_ITEM = new ItemStack(Material.BARRIER);
         ItemMeta meta = BACK_ITEM.getItemMeta();
@@ -32,6 +33,7 @@ public class GuiGlobalItems {
 
     /**
      * Create a GuiItem that open the given GUi.
+     *
      * @param item The item to display in the GUI.
      * @param goal The GUI to open on click.
      * @return An GuiItem that open goal on click.
@@ -43,6 +45,7 @@ public class GuiGlobalItems {
     /**
      * Create back button item from default back GuiItem.
      * The back item will open the goal inventory when clicked.
+     *
      * @param goal The GUI to go back to.
      * @return An GuiItem that go back to goal on click.
      */
@@ -53,6 +56,7 @@ public class GuiGlobalItems {
     /**
      * Add default back item to a GUI pattern with the reserved character key <strong>B</strong>.
      * The back item will open the target inventory when clicked.
+     *
      * @param target The pattern to add the back item.
      * @param goal   The GUI to go back to.
      */
@@ -66,6 +70,7 @@ public class GuiGlobalItems {
     /**
      * Get a background item with backgroundMat as the displayed material.
      * A background item is a GuiItem that do nothing when interacted with and have an empty name.
+     *
      * @param backgroundMat The material to which the background item should be made of.
      * @return A background item with backgroundMat as material.
      */
@@ -76,9 +81,11 @@ public class GuiGlobalItems {
         item.setItemMeta(meta);
         return new GuiItem(item, GuiGlobalActions.stayInPlace, CustomAnvil.instance);
     }
+
     /**
      * Get default background GuiItem.
      * A background item is a GuiItem that do nothing when interacted with and have an empty name.
+     *
      * @return A new instance of the default background item.
      */
     public static GuiItem backgroundItem() {
@@ -88,6 +95,7 @@ public class GuiGlobalItems {
     /**
      * Add default background item to a GUI pattern with the reserved character key <strong>0</strong>.
      * A background item is a GuiItem that do nothing when interacted with and have an empty name.
+     *
      * @param target        The pattern to add the background item.
      * @param backgroundMat The material of the background item.
      */
@@ -99,6 +107,7 @@ public class GuiGlobalItems {
     /**
      * Add default background item to a GUI pattern with the reserved character key <strong>0</strong>.
      * A background item is a GuiItem that do nothing when interacted with and have an empty name.
+     *
      * @param target The pattern to add the background item.
      */
     public static void addBackgroundItem(@NotNull PatternPane target) {
@@ -112,6 +121,7 @@ public class GuiGlobalItems {
      * Create a new save setting GuiItem.
      * A save setting item is a GuiItem that save a changed setting when clicked.
      * This item also check if the player who interacted with the item have the permission to save before saving.
+     *
      * @param setting The setting to change.
      * @param goal    Parent GUI of this setting GUI. as setting will be change the display of goal GUI will be updated.
      * @return A save setting item.
@@ -131,6 +141,7 @@ public class GuiGlobalItems {
 
     // Create static non change item
     private static final GuiItem NO_CHANGE_ITEM;
+
     static {
         ItemStack item = new ItemStack(DEFAULT_NO_CHANGE_ITEM);
         ItemMeta meta = item.getItemMeta();
@@ -142,6 +153,7 @@ public class GuiGlobalItems {
     /**
      * Get the global "no change" GuiItem.
      * The no change item do nothing when interacted, only the title is change to show there is no change.
+     *
      * @return The global "no change" item.
      */
     public static GuiItem noChangeItem() {
@@ -151,6 +163,7 @@ public class GuiGlobalItems {
     /**
      * Create a new "create and go to the setting GUI" GuiItem.
      * This item will create and open a setting GUI from the factory.
+     *
      * @param item    The item that will be displayed.
      * @param factory The setting's GUI factory.
      * @return A formatted GuiItem that will create and open a GUI for the setting.
@@ -169,6 +182,7 @@ public class GuiGlobalItems {
      * Create a new Boolean setting GuiItem.
      * This item will create and open a boolean setting GUI from the factory.
      * The item will have its value written in the lore part of the item.
+     *
      * @param factory The setting's GUI factory.
      * @param name    Name of the item.
      * @return A formatted GuiItem that will create and open a GUI for the boolean setting.
@@ -199,6 +213,7 @@ public class GuiGlobalItems {
      * This item will create and open a boolean setting GUI from the factory.
      * The item will have its value written in the lore part of the item.
      * Item's name will be the factory set title.
+     *
      * @param factory The setting's GUI factory.
      * @return A formatted GuiItem that will create and open a GUI for the boolean setting.
      */
@@ -213,6 +228,7 @@ public class GuiGlobalItems {
      * Create a new int setting GuiItem.
      * This item will create and open an int setting GUI from the factory.
      * The item will have its value written in the lore part of the item.
+     *
      * @param factory The setting's GUI factory.
      * @param itemMat Displayed material of the item.
      * @param name    Name of the item.
@@ -235,6 +251,7 @@ public class GuiGlobalItems {
      * This item will create and open an int setting GUI from the factory.
      * The item will have its value written in the lore part of the item.
      * Item's name will be the factory set title.
+     *
      * @param factory The setting's GUI factory.
      * @param itemMat Displayed material of the item.
      * @return A formatted GuiItem that will create and open a GUI for the int setting.
@@ -249,6 +266,7 @@ public class GuiGlobalItems {
 
     /**
      * Create an arbitrary GuiItem from a unique setting and item's property.
+     *
      * @param factory  The setting's GUI factory.
      * @param itemMat  Displayed material of the item.
      * @param itemName Name of the item.
@@ -277,6 +295,7 @@ public class GuiGlobalItems {
     /**
      * Get the setting name from the setting path.
      * For example: "gui.command.name" will return "name".
+     *
      * @param path The setting's path.
      * @return The setting's name.
      */

src/main/java/xyz/alexcrea/cuanvil/gui/util/GuiSharedConstant.java

@@ -31,6 +31,7 @@ public class GuiSharedConstant {
     public static final boolean TEMPORARY_DO_BACKUP_EVERY_SAVE = true;
 
     public static final PatternPane BACK_TO_MAIN_MENU_BIG_LIST_DISPLAY_BACKGROUND_PANE;
+
     static {
         Pattern pattern = new Pattern(
                 GuiSharedConstant.EMPTY_GUI_FULL_LINE,

src/main/java/xyz/alexcrea/cuanvil/util/CasedStringUtil.java

@@ -9,6 +9,7 @@ public class CasedStringUtil {
      * Transform a snake cased string to an upper-cased spaced string.
      * <p>
      * for example: if we use  "hello_world" as an input this function will return "Hello World".
+     *
      * @param snake_cased_string The input string.
      *                           This argument NEED to be a snake cased string, or it will not work
      * @return The input as an upper-cased string with space separator.

src/main/kotlin/io/delilaheve/AnvilEventListener.kt

@@ -39,6 +39,7 @@ class AnvilEventListener : Listener {
         private const val ANVIL_INPUT_LEFT = 0
         private const val ANVIL_INPUT_RIGHT = 1
         private const val ANVIL_OUTPUT_SLOT = 2
+
         // static slot container
         private val NO_SLOT = SlotContainer(SlotType.NO_SLOT, 0)
         private val CURSOR_SLOT = SlotContainer(SlotType.CURSOR, 0)
@@ -160,7 +161,9 @@ class AnvilEventListener : Listener {
         val player = event.whoClicked as? Player ?: return
         if (!player.hasPermission(CustomAnvil.affectedByPluginPermission)) return
         val inventory = event.inventory as? AnvilInventory ?: return
-        if (event.rawSlot != ANVIL_OUTPUT_SLOT) { return }
+        if (event.rawSlot != ANVIL_OUTPUT_SLOT) {
+            return
+        }
         val output = inventory.getItem(ANVIL_OUTPUT_SLOT) ?: return
         val leftItem = inventory.getItem(ANVIL_INPUT_LEFT) ?: return
         val rightItem = inventory.getItem(ANVIL_INPUT_RIGHT)
@@ -173,7 +176,8 @@ class AnvilEventListener : Listener {
 
         // True if there was no change or not allowed
         if ((output == inventory.getItem(ANVIL_INPUT_LEFT))
-            || !allowed){
+            || !allowed
+        ) {
 
             event.result = Event.Result.DENY
             return
@@ -185,23 +189,28 @@ class AnvilEventListener : Listener {
         if (canMerge) {
             event.result = Event.Result.ALLOW
         } else if (unitRepairResult != null) {
-            onUnitRepairExtract(leftItem, rightItem, output,
+            onUnitRepairExtract(
-                unitRepairResult, event, player, inventory)
+                leftItem, rightItem, output,
+                unitRepairResult, event, player, inventory
+            )
 
             return
         }
     }
 
-    private fun onUnitRepairExtract(leftItem: ItemStack,
+    private fun onUnitRepairExtract(
+        leftItem: ItemStack,
         rightItem: ItemStack,
         output: ItemStack,
         unitRepairResult: Double,
         event: InventoryClickEvent,
         player: Player,
-                                    inventory: AnvilInventory){
+        inventory: AnvilInventory
+    ) {
         val resultCopy = leftItem.clone()
         val resultAmount = resultCopy.unitRepair(
-            rightItem.amount, unitRepairResult)
+            rightItem.amount, unitRepairResult
+        )
 
         // To avoid vanilla, we cancel the event for unit repair
         event.result = Event.Result.DENY
@@ -228,7 +237,8 @@ class AnvilEventListener : Listener {
             repairCost += resultAmount * ConfigOptions.unitRepairCost
 
             if ((inventory.maximumRepairCost < repairCost)
-                        || (player.level < repairCost)) return
+                || (player.level < repairCost)
+            ) return
         }
         // If not creative middle click...
         if (event.click != ClickType.MIDDLE) {
@@ -291,8 +301,11 @@ class AnvilEventListener : Listener {
         // Calculate work penalty
         val leftPenalty = (left.itemMeta as? Repairable)?.repairCost ?: 0
         val rightPenalty =
-            if(right == null){ 0 }
+            if (right == null) {
-            else{ (right.itemMeta as? Repairable)?.repairCost ?: 0 }
+                0
+            } else {
+                (right.itemMeta as? Repairable)?.repairCost ?: 0
+            }
 
         // Try to set work penalty for the result item
         result.itemMeta?.let {
@@ -300,10 +313,12 @@ class AnvilEventListener : Listener {
             result.itemMeta = it
         }
 
-        CustomAnvil.log("Calculated penalty: " +
+        CustomAnvil.log(
+            "Calculated penalty: " +
                     "leftPenalty: $leftPenalty, " +
                     "rightPenalty: $rightPenalty, " +
-                "result penalty: ${(result.itemMeta as? Repairable)?.repairCost ?: "none"}")
+                    "result penalty: ${(result.itemMeta as? Repairable)?.repairCost ?: "none"}"
+        )
 
         return leftPenalty + rightPenalty
     }
@@ -325,7 +340,11 @@ class AnvilEventListener : Listener {
             // count enchant as illegal enchant if it conflicts with another enchant or not in result
             if ((enchantment.key !in resultEnchsKeys)) {
                 resultEnchsKeys.add(enchantment.key)
-                val conflictType = ConfigHolder.CONFLICT_HOLDER.conflictManager.isConflicting(resultEnchsKeys,result.type,enchantment.key)
+                val conflictType = ConfigHolder.CONFLICT_HOLDER.conflictManager.isConflicting(
+                    resultEnchsKeys,
+                    result.type,
+                    enchantment.key
+                )
                 resultEnchsKeys.remove(enchantment.key)
 
                 if (ConflictType.BIG_CONFLICT == conflictType) {
@@ -342,9 +361,11 @@ class AnvilEventListener : Listener {
             rightValue += value
 
         }
-        CustomAnvil.log("Calculated right values: " +
+        CustomAnvil.log(
+            "Calculated right values: " +
                     "rightValue: $rightValue, " +
-                "illegalPenalty: $illegalPenalty")
+                    "illegalPenalty: $illegalPenalty"
+        )
 
         return rightValue + illegalPenalty
     }
@@ -352,15 +373,16 @@ class AnvilEventListener : Listener {
     /**
      * Display xp needed for the work on the anvil inventory
      */
-    private fun handleAnvilXp(inventory: AnvilInventory,
+    private fun handleAnvilXp(
+        inventory: AnvilInventory,
         event: PrepareAnvilEvent,
-                              anvilCost: Int){
+        anvilCost: Int
+    ) {
         // Test repair cost limit
-        val finalAnvilCost: Int
+        val finalAnvilCost = if (ConfigOptions.limitRepairCost) {
-        if (ConfigOptions.limitRepairCost) {
+            min(anvilCost, ConfigOptions.limitRepairValue)
-            finalAnvilCost = min(anvilCost, ConfigOptions.limitRepairValue)
         } else {
-            finalAnvilCost = anvilCost
+            anvilCost
         }
 
         /* Because Minecraft likes to have the final say in the repair cost displayed

src/main/kotlin/io/delilaheve/CustomAnvil.kt

@@ -24,17 +24,22 @@ class CustomAnvil : JavaPlugin() {
 
         // Permission string required to use the plugin's features
         const val affectedByPluginPermission = "ca.affected"
+
         // Permission string required to bypass enchantment conflicts test
         const val bypassFusePermission = "ca.bypass.fuse"
+
         // Permission string required to bypass enchantment conflicts test
         const val bypassLevelPermission = "ca.bypass.level"
+
         // Permission string required to reload the config
         const val commandReloadPermission = "ca.command.reload"
+
         // Permission string required to edit the plugin's config
         const val editConfigPermission = "ca.config.edit"
 
         // Command Name to reload the config
         const val commandReloadName = "anvilconfigreload"
+
         // Test command name
         const val commandTestName = "customanvilconfig"
 
@@ -98,8 +103,10 @@ class CustomAnvil : JavaPlugin() {
         )
     }
 
-    fun reloadResource(resourceName: String,
+    fun reloadResource(
-                       hardFailSafe:Boolean = true): YamlConfiguration?{
+        resourceName: String,
+        hardFailSafe: Boolean = true
+    ): YamlConfiguration? {
         // Save default resource
         val file = File(dataFolder, resourceName)
         if (!file.exists()) {

src/main/kotlin/io/delilaheve/util/ConfigOptions.kt

@@ -12,67 +12,94 @@ object ConfigOptions {
 
     // Path for default enchantment limits
     private const val DEFAULT_LIMIT_PATH = "default_limit"
+
     // Path for limiting repair cost
     const val LIMIT_REPAIR_COST = "limit_repair_cost"
+
     // Path for repair value limit
     const val LIMIT_REPAIR_VALUE = "limit_repair_value"
+
     // Path for level cost on item repair
     const val ITEM_REPAIR_COST = "item_repair_cost"
+
     // Path for level cost on unit repair
     const val UNIT_REPAIR_COST = "unit_repair_cost"
+
     // Path for level cost on item renaming
     const val ITEM_RENAME_COST = "item_rename_cost"
+
     // Path for level cost on illegal enchantment on sacrifice
     const val SACRIFICE_ILLEGAL_COST = "sacrifice_illegal_enchant_cost"
+
     // Path for removing repair cost limits
     const val REMOVE_REPAIR_LIMIT = "remove_repair_limit"
+
     // Root path for enchantment limits
     const val ENCHANT_LIMIT_ROOT = "enchant_limits"
+
     // Root path for enchantment values
     const val ENCHANT_VALUES_ROOT = "enchant_values"
+
     // Keys for specific enchantment values
     private const val KEY_BOOK = "book"
     private const val KEY_ITEM = "item"
+
     // Debug logging toggle path
     private const val DEBUG_LOGGING = "debug_log"
+
     // Debug verbose logging toggle path
     private const val VERBOSE_DEBUG_LOGGING = "debug_log_verbose"
 
     // Default value for enchantment limits
     private const val DEFAULT_ENCHANT_LIMIT = 5
+
     // Default value for limiting repair cost
     const val DEFAULT_LIMIT_REPAIR = false
+
     // Default value for repair cost limit
     const val DEFAULT_LIMIT_REPAIR_VALUE = 39
+
     // Default value for level cost on item repair
     const val DEFAULT_ITEM_REPAIR_COST = 2
+
     // Default value for level cost per unit repair
     const val DEFAULT_UNIT_REPAIR_COST = 1
+
     // Default value for level cost on item renaming
     const val DEFAULT_ITEM_RENAME_COST = 1
+
     // Default value for level cost on illegal enchantment on sacrifice
     const val DEFAULT_SACRIFICE_ILLEGAL_COST = 1
+
     // Valid range for repair cost limit
     @JvmField
     val REPAIR_LIMIT_RANGE = 1..39
+
     // Valid range for repair cost
     @JvmField
     val REPAIR_COST_RANGE = 0..255
+
     // Valid range for rename cost
     @JvmField
     val ITEM_RENAME_COST_RANGE = 0..255
+
     // Valid range for illegal enchantment conflict cost
     @JvmField
     val SACRIFICE_ILLEGAL_COST_RANGE = 0..255
+
     // Default for removing repair cost limits
     const val DEFAULT_REMOVE_LIMIT = false
+
     // Valid range for an enchantment limit
     @JvmField
     val ENCHANT_LIMIT_RANGE = 1..255
+
     // Default value for an enchantment multiplier
     private const val DEFAULT_ENCHANT_VALUE = 0
+
     // Default value for debug logging
     private const val DEFAULT_DEBUG_LOG = false
+
     // Default value for debug logging
     private const val DEFAULT_VERBOSE_DEBUG_LOG = false
 
@@ -155,6 +182,7 @@ object ConfigOptions {
                 .takeIf { it in SACRIFICE_ILLEGAL_COST_RANGE }
                 ?: DEFAULT_SACRIFICE_ILLEGAL_COST
         }
+
     /**
      * Whether to remove repair cost limit
      */
@@ -218,7 +246,8 @@ object ConfigOptions {
      * Get an array of key of basic config options
      */
     fun getBasicConfigKeys(): Array<String> {
-        return arrayOf(DEFAULT_LIMIT_PATH,
+        return arrayOf(
+            DEFAULT_LIMIT_PATH,
             LIMIT_REPAIR_COST,
             LIMIT_REPAIR_VALUE,
             ITEM_REPAIR_COST,

src/main/kotlin/io/delilaheve/util/EnchantmentUtil.kt

@@ -34,9 +34,11 @@ object EnchantmentUtil {
             if (!containsKey(enchantment)) {
                 // Add the enchantment if it doesn't have conflicts, or if player is allowed to bypass enchantment restrictions
                 this[enchantment] = level
-                val conflictType = ConfigHolder.CONFLICT_HOLDER.conflictManager.isConflicting(this.keys,mat,enchantment);
+                val conflictType =
+                    ConfigHolder.CONFLICT_HOLDER.conflictManager.isConflicting(this.keys, mat, enchantment)
                 if (!player.hasPermission(CustomAnvil.bypassFusePermission) &&
-                    (conflictType != ConflictType.NO_CONFLICT)){
+                    (conflictType != ConflictType.NO_CONFLICT)
+                ) {
                     CustomAnvil.verboseLog("Enchantment not yet in result list, but there is conflict (${enchantment.key}, conflict: $conflictType)")
                     this.remove(enchantment)
                 }
@@ -45,9 +47,11 @@ object EnchantmentUtil {
             // Enchantment already in result list
             else {
                 // ... and they are conflicting
-                val conflictType = ConfigHolder.CONFLICT_HOLDER.conflictManager.isConflicting(this.keys,mat,enchantment)
+                val conflictType =
+                    ConfigHolder.CONFLICT_HOLDER.conflictManager.isConflicting(this.keys, mat, enchantment)
                 if ((conflictType != ConflictType.NO_CONFLICT)
-                    && !player.hasPermission(CustomAnvil.bypassFusePermission)){
+                    && !player.hasPermission(CustomAnvil.bypassFusePermission)
+                ) {
                     CustomAnvil.verboseLog("Enchantment already in result list, and they are conflicting (${enchantment.key}, conflict: $conflictType)")
                     return@forEach
                 }
@@ -57,7 +61,9 @@ object EnchantmentUtil {
                     val newLevel = max(this[enchantment] ?: 0, other[enchantment] ?: 0)
                     // apply the greater of the two if non-zero
 
-                    if (newLevel > 0) { this[enchantment] = newLevel }
+                    if (newLevel > 0) {
+                        this[enchantment] = newLevel
+                    }
                 }
                 // ... and they're the same level
                 else {
@@ -70,7 +76,9 @@ object EnchantmentUtil {
                         ConfigOptions.enchantLimit(enchantment)
                     }
                     newLevel = min(newLevel, maxLevel)
-                    if (newLevel > 0) { this[enchantment] = newLevel }
+                    if (newLevel > 0) {
+                        this[enchantment] = newLevel
+                    }
                 }
             }
         }

src/main/kotlin/xyz/alexcrea/cuanvil/group/AbstractMaterialGroup.kt

@@ -44,6 +44,7 @@ abstract class AbstractMaterialGroup(private val name: String) {
     open fun getNonGroupInheritedMaterials(): EnumSet<Material> {
         return includedMaterial
     }
+
     /**
      * Get the group non-inherited material as a set
      */

src/main/kotlin/xyz/alexcrea/cuanvil/group/EnchantConflictGroup.kt

@@ -7,7 +7,8 @@ import org.bukkit.enchantments.Enchantment
 class EnchantConflictGroup(
     val name: String,
     private val cantConflict: AbstractMaterialGroup,
-    val minBeforeBlock: Int){
+    val minBeforeBlock: Int
+) {
 
     private val enchantments = HashSet<Enchantment>()
 
@@ -38,7 +39,7 @@ class EnchantConflictGroup(
     }
 
     fun getCantConflictGroup(): AbstractMaterialGroup {
-        return this.cantConflict;
+        return this.cantConflict
     }
 
     fun getEnchants(): HashSet<Enchantment> {

src/main/kotlin/xyz/alexcrea/cuanvil/group/EnchantConflictManager.kt

@@ -11,13 +11,17 @@ class EnchantConflictManager {
     companion object {
         // Path for the enchantments list
         const val ENCH_LIST_PATH = "enchantments"
+
         // Path for group list related to the conflict
         const val CONFLICT_GROUP_PATH = "notAffectedGroups"
+
         // Path for the maximum number of enchantment before validating the conflict
         const val ENCH_MAX_PATH = "maxEnchantmentBeforeConflict"
+
         // Path for a flag: if the enchantment will be used in the last supported version
         // TODO maybe replace this system by a list of "future" enchantment.
         private const val FUTURE_USE_PATH = "useInFuture"
+
         // Default name for a joining group
         private const val DEFAULT_GROUP_NAME = "joinedGroup"
     }
@@ -34,19 +38,17 @@ class EnchantConflictManager {
         for (key in keys) {
             val section = config.getConfigurationSection(key)!!
             val conflict = createConflict(section, itemManager, key)
-            if(conflict != null){
+
             addToMap(conflict)
             conflictList.add(conflict)
         }
 
     }
 
-    }
-
     // Add the conflict to the map
     private fun addToMap(conflict: EnchantConflictGroup) {
         conflict.getEnchants().forEach { enchant ->
-            addConflictToConflictMap(enchant, conflict);
+            addConflictToConflictMap(enchant, conflict)
         }
     }
 
@@ -62,9 +64,11 @@ class EnchantConflictManager {
     }
 
     // create and read a conflict from a yaml section
-    private fun createConflict(section: ConfigurationSection,
+    private fun createConflict(
+        section: ConfigurationSection,
         itemManager: ItemGroupManager,
-                               conflictName: String): EnchantConflictGroup? {
+        conflictName: String
+    ): EnchantConflictGroup {
         // Is it planed for the future
         val futureUse = section.getBoolean(FUTURE_USE_PATH, false)
         // Create conflict
@@ -91,9 +95,11 @@ class EnchantConflictManager {
         return conflict
     }
 
-    private fun createConflictObject(section: ConfigurationSection,
+    private fun createConflictObject(
+        section: ConfigurationSection,
         itemManager: ItemGroupManager,
-                                     conflictName: String): EnchantConflictGroup {
+        conflictName: String
+    ): EnchantConflictGroup {
         // Get the maximum number of enchantment before validating the conflict
         var minBeforeBlock = section.getInt(ENCH_MAX_PATH, 0)
         if (minBeforeBlock < 0) {
@@ -112,7 +118,11 @@ class EnchantConflictManager {
         return EnchantConflictGroup(conflictName, finalGroup, minBeforeBlock)
     }
 
-    private fun findGroup(groupName: String, itemManager: ItemGroupManager, conflictName: String): AbstractMaterialGroup {
+    private fun findGroup(
+        groupName: String,
+        itemManager: ItemGroupManager,
+        conflictName: String
+    ): AbstractMaterialGroup {
         val group = itemManager.get(groupName)
         if (group == null) {
             CustomAnvil.instance.logger.warning("Group $groupName do not exist but is ask by conflict $conflictName")

src/main/kotlin/xyz/alexcrea/cuanvil/group/ItemGroupManager.kt

@@ -4,17 +4,19 @@ import io.delilaheve.CustomAnvil
 import org.bukkit.Material
 import org.bukkit.configuration.ConfigurationSection
 import java.util.*
-import kotlin.collections.LinkedHashMap
 
 class ItemGroupManager {
 
     companion object {
         // Path for group type
         private const val GROUP_TYPE_PATH = "type"
+
         // Path for included items list
         private const val MATERIAL_LIST_PATH = "items"
+
         // Path for included groups list
         private const val GROUP_LIST_PATH = "groups"
+
         // Temporary list of elements in default config that are use in future
         private val FUTURE_MATERIAL = setOf("PIGLIN_HEAD", "BRUSH")
     }
@@ -34,9 +36,11 @@ class ItemGroupManager {
     }
 
     // Create group by key
-    private fun createGroup(config: ConfigurationSection,
+    private fun createGroup(
+        config: ConfigurationSection,
         keys: Set<String>,
-                            key: String): AbstractMaterialGroup {
+        key: String
+    ): AbstractMaterialGroup {
         val groupSection = config.getConfigurationSection(key)!!
         val groupType = groupSection.getString(GROUP_TYPE_PATH, null)
 
@@ -57,10 +61,12 @@ class ItemGroupManager {
     }
 
     // Read Group elements
-    private fun readGroup(group: AbstractMaterialGroup,
+    private fun readGroup(
+        group: AbstractMaterialGroup,
         groupSection: ConfigurationSection,
         config: ConfigurationSection,
-                          keys: Set<String>){
+        keys: Set<String>
+    ) {
         // Read material to include in this group policy
         val materialList = groupSection.getStringList(MATERIAL_LIST_PATH)
         for (materialTemp in materialList) {
@@ -70,7 +76,8 @@ class ItemGroupManager {
                 // Check if we should warn the user
                 if (materialName !in FUTURE_MATERIAL) {
                     CustomAnvil.instance.logger.warning(
-                        "Unknown material $materialTemp on group ${group.getName()}")
+                        "Unknown material $materialTemp on group ${group.getName()}"
+                    )
 
                 }
                 continue
@@ -84,7 +91,8 @@ class ItemGroupManager {
         for (groupName in groupList) {
             if (groupName !in keys) {
                 CustomAnvil.instance.logger.warning(
-                    "Group $groupName do not exist but is included in group ${group.getName()}")
+                    "Group $groupName do not exist but is included in group ${group.getName()}"
+                )
                 continue
             }
             // Get other group or create it if not yet created
@@ -96,9 +104,11 @@ class ItemGroupManager {
             // Avoid self reference or it will create an infinite loop
             if (otherGroup.isReferencing(group)) {
                 CustomAnvil.instance.logger.warning(
-                    "Group $groupName is on a reference loop with group ${group.getName()} !")
+                    "Group $groupName is on a reference loop with group ${group.getName()} !"
+                )
                 CustomAnvil.instance.logger.warning(
-                    "Please fix it in your item_groups config or the plugin will probably not work as expected.")
+                    "Please fix it in your item_groups config or the plugin will probably not work as expected."
+                )
                 continue
             }
 

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

@@ -31,12 +31,11 @@ object MetricsUtil {
      */
     private fun getHashFromKey(section: ConfigurationSection, key: String): Int {
         // Key is assumend to exist
-        val resultHash: Int
+        val resultHash = if (section.isConfigurationSection(key)) {
-        if(section.isConfigurationSection(key)){
             val sectionResult = getConfigurationHash(section.getConfigurationSection(key)!!)
-            resultHash = key.hashCode() xor sectionResult
+            key.hashCode() xor sectionResult
         } else {
-            resultHash = key.hashCode() xor section.getString(key).hashCode()
+            key.hashCode() xor section.getString(key).hashCode()
         }
         return resultHash.hashCode()
     }
@@ -83,12 +82,24 @@ object MetricsUtil {
         isDefaultUnitRepairItemConfig = unitRepairItemConfigHash == unitRepairConfig
         // If not default and debug flag active, print the hash.
         if (ConfigOptions.debugLog) {
-            if(!isDefaultBaseConfig){CustomAnvil.log("baseConfig: $baseConfig")}
+            if (!isDefaultBaseConfig) {
-            if(!isDefaultEnchantLimitsConfig){CustomAnvil.log("limitEnchantConfig: $limitEnchantConfig")}
+                CustomAnvil.log("baseConfig: $baseConfig")
-            if(!isDefaultEnchantValuesConfig){CustomAnvil.log("enchantValueConfig: $enchantValueConfig")}
+            }
-            if(!isDefaultEnchantConflictConfig){CustomAnvil.log("enchantConflictConfig: $enchantConflictConfig")}
+            if (!isDefaultEnchantLimitsConfig) {
-            if(!isDefaultItemGroupsConfig){CustomAnvil.log("itemGroupConfig: $itemGroupConfig")}
+                CustomAnvil.log("limitEnchantConfig: $limitEnchantConfig")
-            if(!isDefaultUnitRepairItemConfig){CustomAnvil.log("unitRepairConfig: $unitRepairConfig")}
+            }
+            if (!isDefaultEnchantValuesConfig) {
+                CustomAnvil.log("enchantValueConfig: $enchantValueConfig")
+            }
+            if (!isDefaultEnchantConflictConfig) {
+                CustomAnvil.log("enchantConflictConfig: $enchantConflictConfig")
+            }
+            if (!isDefaultItemGroupsConfig) {
+                CustomAnvil.log("itemGroupConfig: $itemGroupConfig")
+            }
+            if (!isDefaultUnitRepairItemConfig) {
+                CustomAnvil.log("unitRepairConfig: $unitRepairConfig")
+            }
         }
 
     }
@@ -96,18 +107,18 @@ object MetricsUtil {
     fun notifyChange(holder: ConfigHolder, path: String) {
         if (ConfigHolder.DEFAULT_CONFIG.equals(holder)) {
             if (path.startsWith(ConfigOptions.ENCHANT_LIMIT_ROOT + ".")) {
-                isDefaultEnchantLimitsConfig = false;
+                isDefaultEnchantLimitsConfig = false
             } else if (path.startsWith(ConfigOptions.ENCHANT_VALUES_ROOT + ".")) {
-                isDefaultEnchantValuesConfig = false;
+                isDefaultEnchantValuesConfig = false
             } else {
-                isDefaultBaseConfig = false;
+                isDefaultBaseConfig = false
             }
         } else if (ConfigHolder.CONFLICT_HOLDER.equals(holder)) {
-            isDefaultEnchantConflictConfig = false;
+            isDefaultEnchantConflictConfig = false
         } else if (ConfigHolder.ITEM_GROUP_HOLDER.equals(holder)) {
-            isDefaultItemGroupsConfig = false;
+            isDefaultItemGroupsConfig = false
         } else if (ConfigHolder.UNIT_REPAIR_HOLDER.equals(holder)) {
-            isDefaultUnitRepairItemConfig = false;
+            isDefaultUnitRepairItemConfig = false
         }
     }
 

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

@@ -8,6 +8,7 @@ object UnitRepairUtil {
 
     // Default value for user set default unit repair %
     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"
 

src/main/resources/plugin.yml

@@ -1,7 +1,7 @@
 main: io.delilaheve.CustomAnvil
 name: CustomAnvil
 prefix: "Custom Anvil"
-version: 1.3.2-A1
+version: 1.3.2-A2
 description: Allow to customise anvil mechanics
 api-version: 1.18
 load: POSTWORLD