Makes sure only intended items are re-forge-able, and changes version to 1.0.0

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>net.knarcraft</groupId>
     <artifactId>blacksmith</artifactId>
-    <version>1.19.1-SNAPSHOT</version>
+    <version>1.0.0-SNAPSHOT</version>
     <name>Blacksmith</name>
     <description>Blacksmith Character for the CitizensAPI</description>
 

src/main/java/net/knarcraft/blacksmith/config/NPCSettings.java

@@ -1,12 +1,16 @@
 package net.knarcraft.blacksmith.config;
 
 import net.citizensnpcs.api.util.DataKey;
+import net.knarcraft.blacksmith.BlacksmithPlugin;
+import net.knarcraft.blacksmith.trait.BlacksmithTrait;
 import org.bukkit.Material;
+import org.bukkit.inventory.ItemStack;
 
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.logging.Level;
 
 /**
  * A class which keeps track of all Blacksmith settings/config values for one NPC
@@ -333,8 +337,11 @@ public class NPCSettings {
                 continue;
             }
             Material material = Material.matchMaterial(String.valueOf(item));
-            if (material != null) {
+            if (material != null && BlacksmithTrait.isRepairable(new ItemStack(material, 1))) {
                 this.reforgeAbleItems.add(material);
+            } else {
+                BlacksmithPlugin.getInstance().getLogger().log(Level.WARNING, "Unable to verify " + material +
+                        " as a valid reforge-able item");
             }
         }
     }

src/main/java/net/knarcraft/blacksmith/trait/BlacksmithTrait.java

@@ -8,6 +8,7 @@ import net.knarcraft.blacksmith.config.NPCSettings;
 import net.knarcraft.blacksmith.manager.EconomyManager;
 import org.bukkit.Bukkit;
 import org.bukkit.Material;
+import org.bukkit.enchantments.EnchantmentTarget;
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 import org.bukkit.inventory.ItemStack;
@@ -208,8 +209,8 @@ public class BlacksmithTrait extends Trait {
      * @param item <p>The item to check</p>
      * @return <p>True if the item is repairable</p>
      */
-    private static boolean isRepairable(ItemStack item) {
+    public static boolean isRepairable(ItemStack item) {
-        return item.getItemMeta() instanceof Damageable;
+        return item.getItemMeta() instanceof Damageable && EnchantmentTarget.BREAKABLE.includes(item);
     }
 
 }

src/main/resources/config.yml

@@ -47,11 +47,14 @@ defaults:
 
   # All settable delays
   delays-in-seconds:
-    maximum: 30 # The maximum time for a re-forge to finish
+    # The maximum time for a re-forge to finish
+    maximum: 30
 
-    minimum: 5 # The minimum time for a re-forge to finish
+    # The minimum time for a re-forge to finish
+    minimum: 5
 
-    reforge-cool-down: 60 # The cool-down period between each re-forge
+    # The cool-down period between each re-forge
+    reforge-cool-down: 60
 
   # All messages used by the NPC
   messages:

src/main/resources/plugin.yml

@@ -1,7 +1,7 @@
 name: Blacksmith
 author: EpicKnarvik97, aPunch, jrbudda, HurricanKai
 authors: [ EpicKnarvik97, aPunch, jrbudda, HurricanKai ]
-version: 1.18.1
+version: 1.0.0
 main: net.knarcraft.blacksmith.BlacksmithPlugin
 depend: [ Citizens, Vault ]