Update pom.xml

pom.xml

@@ -23,7 +23,7 @@
 
     <groupId>lol.hyper</groupId>
     <artifactId>toolstats</artifactId>
-    <version>1.6.1</version>
+    <version>1.6.2</version>
     <packaging>jar</packaging>
 
     <name>ToolStats</name>

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

@@ -66,19 +66,20 @@ public class VillagerTrade implements Listener {
             if (event.getSlotType() == InventoryType.SlotType.RESULT) {
                 ItemStack item = event.getCurrentItem();
                 // only check items we want
-                if (ItemChecker.isValidItem(item.getType())) {
+                if (!ItemChecker.isValidItem(item.getType())) {
+                    return;
+                }
                 // if the player shift clicks, show the warning
                 if (event.isShiftClick()) {
                     String configMessage = toolStats.config.getString("messages.shift-click-warning.trading");
                     if (configMessage != null) {
-                            event.getWhoClicked().sendMessage(ChatColor.translateAlternateColorCodes('&', configMessage));
+                        player.sendMessage(ChatColor.translateAlternateColorCodes('&', configMessage));
                     }
                 }
                 ItemStack newItem = addLore(item, player);
                 if (newItem != null) {
-                        // this gets delayed since villager inventories suck for no reason
-                        Bukkit.getScheduler().runTaskLater(toolStats, () -> event.setCurrentItem(newItem), 5);
-                    }
+                    // set the new item
+                    inventory.setItem(event.getSlot(), newItem);
                 }
             }
         }