bump config

* fix logic with the edit command
* improved command suggestions

pom.xml

@@ -23,7 +23,7 @@
 
     <groupId>lol.hyper</groupId>
     <artifactId>toolstats</artifactId>
-    <version>2.0</version>
+    <version>2.0.2</version>
     <packaging>jar</packaging>
 
     <name>ToolStats</name>
@@ -51,36 +51,12 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
-                <version>3.14.1</version>
+                <version>3.15.0</version>
                 <configuration>
                     <source>${java.version}</source>
                     <target>${java.version}</target>
                 </configuration>
             </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-shade-plugin</artifactId>
-                <version>3.6.1</version>
-                <configuration>
-                    <relocations>
-                        <relocation>
-                            <pattern>lol.hyper.hyperlib</pattern>
-                            <shadedPattern>lol.hyper.toolstats.hyperlib</shadedPattern>
-                        </relocation>
-                    </relocations>
-                </configuration>
-                <executions>
-                    <execution>
-                        <phase>package</phase>
-                        <goals>
-                            <goal>shade</goal>
-                        </goals>
-                        <configuration>
-                            <createDependencyReducedPom>false</createDependencyReducedPom>
-                        </configuration>
-                    </execution>
-                </executions>
-            </plugin>
         </plugins>
         <resources>
             <resource>
@@ -112,7 +88,7 @@
             <groupId>com.github.hyperdefined</groupId>
             <artifactId>hyperlib</artifactId>
             <version>1.0.9</version>
-            <scope>compile</scope>
+            <scope>provided</scope>
         </dependency>
     </dependencies>
 </project>

src/main/java/lol/hyper/toolstats/ToolStats.java

@@ -36,7 +36,7 @@ import java.io.File;
 
 public final class ToolStats extends JavaPlugin {
 
-    public final int CONFIG_VERSION = 15;
+    public final int CONFIG_VERSION = 16;
     public final ComponentLogger logger = this.getComponentLogger();
     public final File configFile = new File(this.getDataFolder(), "config.yml");
     public boolean tokens = false;
@@ -152,7 +152,7 @@ public final class ToolStats extends JavaPlugin {
         Bukkit.getServer().getPluginManager().registerEvents(inventoryClose, this);
         Bukkit.getServer().getPluginManager().registerEvents(playerDrop, this);
 
-        this.getCommand("toolstats").setExecutor(commandToolStats);
+        registerCommand("toolstats", commandToolStats);
 
         HyperUpdater updater = new HyperUpdater(hyperLib);
         updater.setGitHub("hyperdefined", "ToolStats");

src/main/java/lol/hyper/toolstats/ToolStatsLoader.java

@@ -0,0 +1,37 @@
+/*
+ * This file is part of ToolStats.
+ *
+ * ToolStats is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * ToolStats is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with ToolStats.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+package lol.hyper.toolstats;
+
+import io.papermc.paper.plugin.loader.PluginClasspathBuilder;
+import io.papermc.paper.plugin.loader.PluginLoader;
+import io.papermc.paper.plugin.loader.library.impl.MavenLibraryResolver;
+import org.eclipse.aether.artifact.DefaultArtifact;
+import org.eclipse.aether.graph.Dependency;
+import org.eclipse.aether.repository.RemoteRepository;
+
+public class ToolStatsLoader implements PluginLoader {
+    @Override
+    public void classloader(PluginClasspathBuilder classpathBuilder) {
+        MavenLibraryResolver resolver = new MavenLibraryResolver();
+
+        resolver.addRepository(new RemoteRepository.Builder("jitpack", "default", "https://jitpack.io").build());
+        resolver.addDependency(new Dependency(new DefaultArtifact("com.github.hyperdefined:hyperlib:1.0.9"), null));
+
+        classpathBuilder.addLibrary(resolver);
+    }
+}

src/main/java/lol/hyper/toolstats/commands/CommandToolStats.java

@@ -17,29 +17,29 @@
 
 package lol.hyper.toolstats.commands;
 
+import io.papermc.paper.command.brigadier.BasicCommand;
+import io.papermc.paper.command.brigadier.CommandSourceStack;
 import lol.hyper.hyperlib.datatypes.UUIDDataType;
 import lol.hyper.toolstats.ToolStats;
 import net.kyori.adventure.text.Component;
 import net.kyori.adventure.text.format.NamedTextColor;
 import org.bukkit.Bukkit;
 import org.bukkit.OfflinePlayer;
-import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.command.ConsoleCommandSender;
-import org.bukkit.command.TabExecutor;
 import org.bukkit.entity.Player;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.ShapedRecipe;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.bukkit.persistence.PersistentDataContainer;
 import org.bukkit.persistence.PersistentDataType;
-import org.jetbrains.annotations.NotNull;
+import org.jspecify.annotations.NonNull;
-import org.jetbrains.annotations.Nullable;
+import org.jspecify.annotations.Nullable;
 
 import java.util.*;
 import java.util.stream.Collectors;
 
-public class CommandToolStats implements TabExecutor {
+public class CommandToolStats implements BasicCommand {
 
     private final ToolStats toolStats;
 
@@ -48,14 +48,11 @@ public class CommandToolStats implements TabExecutor {
     }
 
     @Override
-    public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, @NotNull String @NotNull [] args) {
+    public void execute(@NonNull CommandSourceStack source, String @NonNull [] args) {
-        if (!sender.hasPermission("toolstats.command")) {
+        CommandSender sender = source.getSender();
-            sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
-            return true;
-        }
         if (args.length == 0) {
             sender.sendMessage(Component.text("ToolStats version " + toolStats.getPluginMeta().getVersion() + ". Created by hyperdefined.", NamedTextColor.GREEN));
-            return true;
+            return;
         }
         switch (args[0]) {
             case "reload": {
@@ -87,128 +84,128 @@ public class CommandToolStats implements TabExecutor {
                 } else {
                     sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
                 }
-                return true;
+                return;
             }
             // /toolstats edit stat value
             case "edit": {
                 if (!sender.hasPermission("toolstats.edit")) {
                     sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 if (sender instanceof ConsoleCommandSender) {
                     sender.sendMessage(Component.text("You must be a player for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 if (args.length < 3) {
                     sender.sendMessage(Component.text("Invalid syntax. Usage: /toolstats edit <stat> <value>", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 handleEdit(args[1], args[2], (Player) sender);
-                return true;
+                return;
             }
             // /toolstats remove stat
             case "remove": {
                 if (!sender.hasPermission("toolstats.remove")) {
                     sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 if (sender instanceof ConsoleCommandSender) {
                     sender.sendMessage(Component.text("You must be a player for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 if (args.length < 2) {
                     sender.sendMessage(Component.text("Invalid syntax. Usage: /toolstats remove <stat>", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 handleRemove(args[1], (Player) sender);
-                return true;
+                return;
 
             }
             case "reset": {
                 if (!sender.hasPermission("toolstats.reset")) {
                     sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 if (sender instanceof ConsoleCommandSender) {
                     sender.sendMessage(Component.text("You must be a player for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 if (args.length == 2 && args[1].equalsIgnoreCase("confirm")) {
                     if (!sender.hasPermission("toolstats.reset.confirm")) {
                         sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
-                        return true;
+                        return;
                     }
                     Player player = (Player) sender;
                     ItemStack heldItem = player.getInventory().getItemInMainHand();
                     if (!toolStats.itemChecker.isValidItem(heldItem.getType())) {
                         sender.sendMessage(Component.text("You must hold a valid item.", NamedTextColor.RED));
-                        return true;
+                        return;
                     }
                     fixItemLore(heldItem, player);
                     sender.sendMessage(Component.text("The lore was reset!", NamedTextColor.GREEN));
-                    return true;
+                    return;
                 }
                 sender.sendMessage(Component.text("This will remove ALL current lore from the held item and replace it with the correct lore.", NamedTextColor.GREEN));
                 sender.sendMessage(Component.text("If the owner of the item is broken, it will reset to the person holding it.", NamedTextColor.GREEN));
                 sender.sendMessage(Component.text("Only use this if the tags on the tool are incorrect.", NamedTextColor.GREEN));
                 sender.sendMessage(Component.text("Type /toolstats reset confirm to confirm this.", NamedTextColor.GREEN));
-                return true;
+                return;
             }
             case "purge": {
                 if (!sender.hasPermission("toolstats.purge")) {
                     sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 if (sender instanceof ConsoleCommandSender) {
                     sender.sendMessage(Component.text("You must be a player for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 if (args.length == 2 && args[1].equalsIgnoreCase("confirm")) {
                     if (!sender.hasPermission("toolstats.purge.confirm")) {
                         sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
-                        return true;
+                        return;
                     }
                     Player player = (Player) sender;
                     ItemStack heldItem = player.getInventory().getItemInMainHand();
                     if (!toolStats.itemChecker.isValidItem(heldItem.getType())) {
                         sender.sendMessage(Component.text("You must hold a valid item.", NamedTextColor.RED));
-                        return true;
+                        return;
                     }
                     ItemStack purgedItem = toolStats.itemLore.removeAll(heldItem, true);
                     player.getInventory().setItemInMainHand(purgedItem);
                     sender.sendMessage(Component.text("The item was purged!", NamedTextColor.GREEN));
-                    return true;
+                    return;
                 }
                 sender.sendMessage(Component.text("This will purge ALL ToolStats data from this item.", NamedTextColor.GREEN));
                 sender.sendMessage(Component.text("This includes all stats, ownership, and creation time.", NamedTextColor.GREEN));
                 sender.sendMessage(Component.text("THIS CANNOT BE UNDONE!", NamedTextColor.GREEN));
                 sender.sendMessage(Component.text("Type /toolstats purge confirm to confirm this.", NamedTextColor.GREEN));
-                return true;
+                return;
             }
             case "givetokens": {
                 if (!sender.hasPermission("toolstats.givetokens")) {
                     sender.sendMessage(Component.text("You do not have permission for this command.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 // Make sure /toolstats givetoken <player> <token> is present
                 if (args.length < 3) {
                     sender.sendMessage(Component.text("Invalid syntax. Usage: /toolstats givetokens <player> <token>", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 Player target = Bukkit.getPlayerExact(args[1]);
                 if (target == null) {
                     sender.sendMessage(Component.text("Player not found.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 String tokenType = args[2];
                 if (!toolStats.tokenData.getTokenTypes().contains(tokenType)) {
                     sender.sendMessage(Component.text("Invalid token type.", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 // make sure tokens are enabled before giving
                 if (!toolStats.config.getBoolean("tokens.enabled")) {
                     sender.sendMessage(Component.text("Unable to give tokens. Tokens are disabled", NamedTextColor.RED));
-                    return true;
+                    return;
                 }
                 // if the user does not send in a number, default to 1
                 int amount = 1;
@@ -217,24 +214,29 @@ public class CommandToolStats implements TabExecutor {
                         amount = Integer.parseInt(args[3]);
                         if (amount <= 0) { // Optional: Prevent negative or zero values
                             sender.sendMessage(Component.text("Token quantity must be above or 1.", NamedTextColor.RED));
-                            return true;
+                            return;
                         }
                     } catch (NumberFormatException exception) {
                         sender.sendMessage(Component.text("Invalid token quantity.", NamedTextColor.RED));
-                        return true;
+                        return;
                     }
                 }
                 giveToken(target, tokenType, amount);
                 if (sender instanceof Player) {
                     sender.sendMessage(Component.text("Gave " + target.getName() + " " + amount + " " + tokenType + " tokens.", NamedTextColor.GREEN));
                 }
-                return true;
+                return;
             }
             default: {
                 sender.sendMessage(Component.text("Invalid sub-command.", NamedTextColor.RED));
             }
         }
-        return true;
+        return;
+    }
+
+    @Override
+    public @Nullable String permission() {
+        return "toolstats.command";
     }
 
     /**
@@ -448,6 +450,7 @@ public class CommandToolStats implements TabExecutor {
      * @param player    The player using the command.
      */
     private void handleEdit(String stat, Object userValue, Player player) {
+        boolean updated = false;
         ItemStack editedItem = player.getInventory().getItemInMainHand().clone();
         if (!toolStats.itemChecker.isValidItem(editedItem.getType())) {
             player.sendMessage(Component.text("This is not a valid item.", NamedTextColor.RED));
@@ -480,6 +483,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateCropsMined(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -509,6 +513,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateBlocksMined(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -538,6 +543,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     double difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateArmorDamage(editedItem, difference, false);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -567,6 +573,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     double difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateWeaponDamage(editedItem, difference, false);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -596,6 +603,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateMobKills(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -625,6 +633,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updatePlayerKills(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -654,6 +663,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateArrowsShot(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -683,6 +693,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateSheepSheared(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -712,6 +723,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     long difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateFlightTime(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -741,6 +753,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateFishCaught(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -770,6 +783,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateBossesKilled(editedItem, difference, "wither");
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -799,6 +813,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateBossesKilled(editedItem, difference, "enderdragon");
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -828,6 +843,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateCriticalStrikes(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -857,6 +873,7 @@ public class CommandToolStats implements TabExecutor {
                     }
                     int difference = value - statValue;
                     editedItemMeta = toolStats.itemLore.updateTridentThrows(editedItem, difference);
+                    updated = true;
                 } else {
                     player.sendMessage(Component.text("This item does not have that stat.", NamedTextColor.RED));
                 }
@@ -867,9 +884,12 @@ public class CommandToolStats implements TabExecutor {
                 return;
             }
         }
-        editedItem.setItemMeta(editedItemMeta);
+
-        player.getInventory().setItemInMainHand(editedItem);
+        if (updated) {
-        player.sendMessage(Component.text("Updated stat " + stat + " for held item!", NamedTextColor.GREEN));
+            editedItem.setItemMeta(editedItemMeta);
+            player.getInventory().setItemInMainHand(editedItem);
+            player.sendMessage(Component.text("Updated stat " + stat + " for held item!", NamedTextColor.GREEN));
+        }
     }
 
     /**
@@ -1112,7 +1132,7 @@ public class CommandToolStats implements TabExecutor {
                 }
                 break;
             }
-            case "fight-caught": {
+            case "fish-caught": {
                 if (container.has(toolStats.toolStatsKeys.getFishCaught())) {
                     Integer statValue = container.get(toolStats.toolStatsKeys.getFishCaught(), PersistentDataType.INTEGER);
                     if (statValue == null) {
@@ -1125,7 +1145,7 @@ public class CommandToolStats implements TabExecutor {
                         return;
                     }
                     container.remove(toolStats.toolStatsKeys.getFishCaught());
-                    List<String> newTokens = toolStats.itemChecker.removeToken(tokens, "fight-caught");
+                    List<String> newTokens = toolStats.itemChecker.removeToken(tokens, "fish-caught");
                     if (newTokens.isEmpty()) {
                         container.remove(toolStats.toolStatsKeys.getTokenApplied());
                     } else {
@@ -1262,10 +1282,11 @@ public class CommandToolStats implements TabExecutor {
         player.sendMessage(Component.text("Removed stat " + stat + " for held item!", NamedTextColor.GREEN));
     }
 
-    @Nullable
     @Override
-    public List<String> onTabComplete(@NotNull CommandSender sender, @NotNull Command command, @NotNull String alias, @NotNull String @NotNull [] args) {
+    public @NonNull Collection<String> suggest(@NonNull CommandSourceStack source, String[] args) {
-        if (args.length == 1) {
+        CommandSender sender = source.getSender();
+        // suggest basic sub commands
+        if (args.length == 0) {
             List<String> suggestions = new ArrayList<>();
             if (sender.hasPermission("toolstats.reload")) {
                 suggestions.add("reload");
@@ -1285,15 +1306,19 @@ public class CommandToolStats implements TabExecutor {
             if (sender.hasPermission("toolstats.purge")) {
                 suggestions.add("purge");
             }
-            return suggestions.isEmpty() ? null : suggestions;
+            return suggestions;
         }
 
+        // suggest confirm for reset
         if (args.length == 2 && args[0].equalsIgnoreCase("reset") && sender.hasPermission("toolstats.reset.confirm")) {
             return Collections.singletonList("confirm");
         }
+        // suggest confirm for purge
         if (args.length == 2 && args[0].equalsIgnoreCase("purge") && sender.hasPermission("toolstats.purge.confirm")) {
             return Collections.singletonList("confirm");
         }
+
+        // suggest keys for edit
         if (args.length == 2 && args[0].equalsIgnoreCase("edit") && sender.hasPermission("toolstats.edit")) {
             // yes I am lazy
             return toolStats.tokenData.getTokenTypes().stream()
@@ -1301,6 +1326,8 @@ public class CommandToolStats implements TabExecutor {
                     .map(s -> s.equals("crops-mined") ? "crops-harvested" : s)
                     .collect(Collectors.toList());
         }
+
+        // suggest keys for remove
         if (args.length == 2 && args[0].equalsIgnoreCase("remove") && sender.hasPermission("toolstats.remove")) {
             // yes I am lazy
             return toolStats.tokenData.getTokenTypes().stream()
@@ -1308,10 +1335,16 @@ public class CommandToolStats implements TabExecutor {
                     .map(s -> s.equals("crops-mined") ? "crops-harvested" : s)
                     .collect(Collectors.toList());
         }
+
+        // suggest players for givetokens
+        if (args.length == 2 && args[0].equalsIgnoreCase("givetokens") && sender.hasPermission("toolstats.givetokens")) {
+            return Bukkit.getOnlinePlayers().stream().map(Player::getName).toList();
+        }
+
+        // suggest token types for givetokens
         if (args.length == 3 && args[0].equalsIgnoreCase("givetokens") && sender.hasPermission("toolstats.givetokens")) {
             return toolStats.tokenData.getTokenTypes();
         }
-
+        return Collections.emptyList();
-        return null;
     }
 }

src/main/java/lol/hyper/toolstats/events/BlockDispenseEvent.java

@@ -113,6 +113,29 @@ public class BlockDispenseEvent implements Listener {
             lore = new ArrayList<>();
         }
 
+        // by request
+        if (newItem.getType() == Material.ELYTRA) {
+            if (!toolStats.config.getBoolean("enabled.elytra-tag")) {
+                return null;
+            }
+
+            Component creationDate = toolStats.itemLore.formatCreationTime(timeCreated, 4, newItem);
+            if (creationDate != null) {
+                container.set(toolStats.toolStatsKeys.getTimeCreated(), PersistentDataType.LONG, timeCreated);
+                container.set(toolStats.toolStatsKeys.getOriginType(), PersistentDataType.INTEGER, 4);
+                lore.add(creationDate);
+                meta.lore(lore);
+            }
+
+            Component itemOwner = toolStats.itemLore.formatOwner(owner.getName(), 4, newItem);
+            if (itemOwner != null) {
+                container.set(toolStats.toolStatsKeys.getItemOwner(), new UUIDDataType(), owner.getUniqueId());
+                container.set(toolStats.toolStatsKeys.getOriginType(), PersistentDataType.INTEGER, 4);
+                lore.add(itemOwner);
+                meta.lore(lore);
+            }
+        }
+
         // if creation date is enabled, add it
         Component creationDate = toolStats.itemLore.formatCreationTime(timeCreated, 2, newItem);
         if (creationDate != null) {

src/main/java/lol/hyper/toolstats/events/InventoryOpen.java

@@ -19,12 +19,14 @@ package lol.hyper.toolstats.events;
 
 import lol.hyper.hyperlib.datatypes.UUIDDataType;
 import lol.hyper.toolstats.ToolStats;
+import org.bukkit.block.DoubleChest;
 import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
 import org.bukkit.event.inventory.InventoryOpenEvent;
-import org.bukkit.event.inventory.InventoryType;
+import org.bukkit.inventory.BlockInventoryHolder;
 import org.bukkit.inventory.Inventory;
+import org.bukkit.inventory.InventoryHolder;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.bukkit.persistence.PersistentDataContainer;
@@ -47,6 +49,13 @@ public class InventoryOpen implements Listener {
         }
 
         Inventory inventory = event.getInventory();
+        InventoryHolder holder = inventory.getHolder();
+        boolean isBlockInventory = holder instanceof BlockInventoryHolder || holder instanceof DoubleChest;
+        if (!(inventory.getHolder() instanceof BlockInventoryHolder)) {
+            // ignore not real inventories
+            return;
+        }
+
         Player player = (Player) event.getPlayer();
         if (!toolStats.configTools.checkWorld(player.getWorld().getName())) {
             return;

src/main/java/lol/hyper/toolstats/tools/config/ConfigTools.java

@@ -53,7 +53,7 @@ public class ConfigTools {
         String itemName = material.toString().toLowerCase();
         String itemType = null;
         // hardcode these
-        if (material == Material.BOW || material == Material.CROSSBOW || material == Material.SHEARS || material == Material.TRIDENT || material == Material.FISHING_ROD) {
+        if (material == Material.BOW || material == Material.CROSSBOW || material == Material.SHEARS || material == Material.TRIDENT || material == Material.FISHING_ROD || material == Material.SHIELD) {
             switch (material) {
                 case CROSSBOW:
                 case BOW: {
@@ -72,6 +72,10 @@ public class ConfigTools {
                     itemType = "fishing-rod";
                     break;
                 }
+                case SHIELD: {
+                    itemType = "shield";
+                    break;
+                }
             }
         } else {
             itemType = itemName.substring(itemName.indexOf('_') + 1);
@@ -88,6 +92,7 @@ public class ConfigTools {
             case "fishing-rod" -> toolStats.config.getBoolean("enabled." + configName + ".fishing-rod");
             case "mace" -> toolStats.config.getBoolean("enabled." + configName + ".mace");
             case "spear" -> toolStats.config.getBoolean("enabled." + configName + ".spear");
+            case "shield" -> toolStats.config.getBoolean("enabled." + configName + ".shield");
             case "helmet", "chestplate", "leggings", "boots" ->
                     toolStats.config.getBoolean("enabled." + configName + ".armor");
             default -> false;

src/main/java/lol/hyper/toolstats/tools/config/ConfigUpdater.java

@@ -42,6 +42,7 @@ public class ConfigUpdater {
             case 12 -> new Version13(toolStats).update(); // 12 to 13
             case 13 -> new Version14(toolStats).update(); // 13 to 14
             case 14 -> new Version15(toolStats).update(); // 14 to 15
+            case 15 -> new Version16(toolStats).update(); // 15 to 16
         }
     }
 }

src/main/java/lol/hyper/toolstats/tools/config/versions/Version16.java

@@ -0,0 +1,77 @@
+/*
+ * This file is part of ToolStats.
+ *
+ * ToolStats is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * ToolStats is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with ToolStats.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+package lol.hyper.toolstats.tools.config.versions;
+
+import lol.hyper.toolstats.ToolStats;
+
+import java.io.File;
+import java.io.IOException;
+
+public class Version16 {
+
+    private final ToolStats toolStats;
+
+    /**
+     * Used for updating from version 15 to 16.
+     *
+     * @param toolStats ToolStats instance.
+     */
+    public Version16(ToolStats toolStats) {
+        this.toolStats = toolStats;
+    }
+
+    /**
+     * Perform the config update.
+     */
+    public void update() {
+        // save the old config first
+        try {
+            toolStats.config.save("plugins" + File.separator + "ToolStats" + File.separator + "config-15.yml");
+        } catch (IOException exception) {
+            toolStats.logger.error("Unable to save config-15.yml!", exception);
+        }
+
+        // we make this super verbose so that admins can see what's being added
+        toolStats.logger.info("Updating config.yml to version 16.");
+        toolStats.config.set("config-version", 16);
+
+        toolStats.logger.info("Adding enabled.crafted-on.shield to config.yml.");
+        toolStats.config.set("enabled.crafted-on.shield", true);
+        toolStats.logger.info("Adding enabled.crafted-by.shield to config.yml.");
+        toolStats.config.set("enabled.crafted-by.shield", true);
+
+        toolStats.logger.info("Adding enabled.traded-on.shield to config.yml.");
+        toolStats.config.set("enabled.traded-on.shield", true);
+        toolStats.logger.info("Adding enabled.traded-by.shield to config.yml.");
+        toolStats.config.set("enabled.traded-by.shield", true);
+
+        toolStats.logger.info("Adding enabled.looted-on.shield to config.yml.");
+        toolStats.config.set("enabled.looted-on.shield", true);
+        toolStats.logger.info("Adding enabled.looted-by.shield to config.yml.");
+        toolStats.config.set("enabled.looted-by.shield", true);
+
+        // save the config and reload it
+        try {
+            toolStats.config.save("plugins" + File.separator + "ToolStats" + File.separator + "config.yml");
+        } catch (IOException exception) {
+            toolStats.logger.error("Unable to save config.yml!", exception);
+        }
+        toolStats.loadConfig();
+        toolStats.logger.info("Config has been updated to version 16. A copy of version 6 has been saved as config-15.yml");
+    }
+}

src/main/resources/config.yml

@@ -195,6 +195,7 @@ enabled:
     mace: true
     fishing-rod: true
     spear: true
+    shield: true
   # Will show "Crafted on <date>"
   crafted-on:
     pickaxe: true
@@ -208,6 +209,7 @@ enabled:
     mace: true
     fishing-rod: true
     spear: true
+    shield: true
   # Will show "Fished by <player>"
   fished-by:
     pickaxe: true
@@ -242,6 +244,7 @@ enabled:
     armor: true
     fishing-rod: true
     spear: true
+    shield: true
   # Will show "Found on <date>"
   looted-on:
     pickaxe: true
@@ -254,6 +257,7 @@ enabled:
     armor: true
     fishing-rod: true
     spear: true
+    shield: true
   # Will show "Traded by <player>"
   traded-by:
     pickaxe: true
@@ -265,6 +269,7 @@ enabled:
     bow: true
     armor: true
     fishing-rod: true
+    shield: true
   # Will show "Traded on <date>"
   traded-on:
     pickaxe: true
@@ -276,6 +281,7 @@ enabled:
     bow: true
     armor: true
     fishing-rod: true
+    shield: true
   damage-done:
     sword: true
     axe: true

src/main/resources/paper-plugin.yml

@@ -1,34 +1,26 @@
 name: ToolStats
-version: '${project.version}'
+version: ${project.version}
 main: lol.hyper.toolstats.ToolStats
 api-version: 1.21
+folia-supported: true
 author: hyperdefined
 description: Track various tool stats!
 website: https://github.com/hyperdefined/ToolStats
-folia-supported: true
+
-commands:
+loader: lol.hyper.toolstats.ToolStatsLoader
-  toolstats:
+
-    usage: /toolstats
-    permission: toolstats.command
 permissions:
   toolstats.command:
     description: Allows the usage of /toolstats.
     default: true
+    children:
+      toolstats.reset: true
+      toolstats.reset.confirm: true
+      toolstats.purge: true
+      toolstats.purge.confirm: true
   toolstats.reload:
     description: Allows the usage of /toolstats reload.
     default: op
-  toolstats.reset:
-    description: Allows the usage of /toolstats reset.
-    default: true
-  toolstats.reset.confirm:
-    description: Allows the usage of /toolstats reset confirm.
-    default: true
-  toolstats.purge:
-    description: Allows the usage of /toolstats purge.
-    default: true
-  toolstats.purge.confirm:
-    description: Allows the usage of /toolstats purge confirm.
-    default: true
   toolstats.givetokens:
     description: Allows the usage of /toolstats givetoken.
     default: op