Claude/item worth display bugs 1hrf2y

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>com.yourname</groupId>
     <artifactId>SellPlugin</artifactId>
-    <version>1.0-SNAPSHOT</version>
+    <version>1.2-SNAPSHOT</version>
     <packaging>jar</packaging>
 
     <properties>
@@ -45,6 +45,13 @@
             <scope>provided</scope>
         </dependency>
 
+        <dependency>
+            <groupId>net.dmulloy2</groupId>
+            <artifactId>ProtocolLib</artifactId>
+            <version>5.4.0</version>
+            <scope>provided</scope>
+        </dependency>
+
         <dependency>
             <groupId>su.nightexpress.nightcore</groupId>
             <artifactId>NightCore</artifactId>

src/main/java/com/yourname/sellplugin/SellPlugin.java

@@ -1,11 +1,22 @@
 package com.yourname.sellplugin;
 
+import com.yourname.sellplugin.command.SellAllCommand;
 import com.yourname.sellplugin.command.SellCommand;
+import com.yourname.sellplugin.command.SellMultiCommand;
+import com.yourname.sellplugin.command.FastSellAllCommand;
+import com.yourname.sellplugin.command.ShowWorthCommand;
+import com.yourname.sellplugin.command.TopSellCommand;
+import com.yourname.sellplugin.command.WorthCommand;
 import com.yourname.sellplugin.economy.EconomyManager;
 import com.yourname.sellplugin.gui.GUIListener;
+import com.yourname.sellplugin.listener.WorthPacketListener;
+import com.yourname.sellplugin.listener.WorthRefreshListener;
 import com.yourname.sellplugin.manager.ConfigManager;
+import com.yourname.sellplugin.manager.ConfigMigrator;
 import com.yourname.sellplugin.manager.MultiplierManager;
 import com.yourname.sellplugin.manager.PriceManager;
+import com.yourname.sellplugin.manager.SellManager;
+import com.yourname.sellplugin.manager.WorthVisibilityManager;
 import org.bukkit.plugin.java.JavaPlugin;
 
 public class SellPlugin extends JavaPlugin {
@@ -14,18 +25,22 @@ public class SellPlugin extends JavaPlugin {
     private ConfigManager configManager;
     private PriceManager priceManager;
     private MultiplierManager multiplierManager;
+    private SellManager sellManager;
+    private WorthVisibilityManager worthVisibilityManager;
+    private WorthPacketListener worthPacketListener;
 
     @Override
     public void onEnable() {
-        // Initialize Config
         saveDefaultConfig();
+        new ConfigMigrator(this).migrate();
         configManager = new ConfigManager(this);
 
-        // Initialize Managers
         priceManager = new PriceManager(this);
         priceManager.loadPrices();
-        
+
         multiplierManager = new MultiplierManager(this);
+        sellManager = new SellManager(this);
+        worthVisibilityManager = new WorthVisibilityManager(this);
 
         economyManager = new EconomyManager(this);
         if (!economyManager.setupEconomy()) {
@@ -34,9 +49,20 @@ public void onEnable() {
             return;
         }
 
-        // Register Commands & Events
         getCommand("sell").setExecutor(new SellCommand(this));
+        getCommand("sellall").setExecutor(new SellAllCommand(this));
+        getCommand("fastsellall").setExecutor(new FastSellAllCommand(this));
+        getCommand("topsell").setExecutor(new TopSellCommand(this));
+        getCommand("sellmulti").setExecutor(new SellMultiCommand(this));
+        getCommand("sellworth").setExecutor(new WorthCommand(this));
+        ShowWorthCommand showWorthCommand = new ShowWorthCommand(this);
+        getCommand("showworth").setExecutor(showWorthCommand);
+        getCommand("showworth").setTabCompleter(showWorthCommand);
         getServer().getPluginManager().registerEvents(new GUIListener(this), this);
+        getServer().getPluginManager().registerEvents(new WorthRefreshListener(this), this);
+
+        worthPacketListener = new WorthPacketListener(this);
+        worthPacketListener.register();
 
         getLogger().info("SellPlugin has been enabled successfully.");
     }
@@ -46,11 +72,19 @@ public void onDisable() {
         if (multiplierManager != null) {
             multiplierManager.saveAll();
         }
+        if (worthVisibilityManager != null) {
+            worthVisibilityManager.saveNow();
+        }
+        if (worthPacketListener != null) {
+            worthPacketListener.unregister();
+        }
         getLogger().info("SellPlugin has been disabled.");
     }
 
     public EconomyManager getEconomyManager() { return economyManager; }
-    public ConfigManager getConfigManager() { return configManager; }
-    public PriceManager getPriceManager() { return priceManager; }
+    public ConfigManager getConfigManager()    { return configManager; }
+    public PriceManager getPriceManager()      { return priceManager; }
     public MultiplierManager getMultiplierManager() { return multiplierManager; }
+    public SellManager getSellManager()        { return sellManager; }
+    public WorthVisibilityManager getWorthVisibilityManager() { return worthVisibilityManager; }
 }

src/main/java/com/yourname/sellplugin/command/FastSellAllCommand.java

@@ -0,0 +1,32 @@
+package com.yourname.sellplugin.command;
+
+import com.yourname.sellplugin.SellPlugin;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandExecutor;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+
+public class FastSellAllCommand implements CommandExecutor {
+
+    private final SellPlugin plugin;
+
+    public FastSellAllCommand(SellPlugin plugin) {
+        this.plugin = plugin;
+    }
+
+    @Override
+    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
+        if (!(sender instanceof Player player)) {
+            sender.sendMessage(plugin.getConfigManager().getText("player-only-command", "&cOnly players can use this command."));
+            return true;
+        }
+
+        if (!player.hasPermission("sellplugin.use")) {
+            player.sendMessage(plugin.getConfigManager().getMessage("no-permission"));
+            return true;
+        }
+
+        plugin.getSellManager().sellAll(player);
+        return true;
+    }
+}

src/main/java/com/yourname/sellplugin/command/SellAllCommand.java

@@ -0,0 +1,32 @@
+package com.yourname.sellplugin.command;
+
+import com.yourname.sellplugin.SellPlugin;
+import com.yourname.sellplugin.gui.SellAllGUI;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandExecutor;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+
+public class SellAllCommand implements CommandExecutor {
+    private final SellPlugin plugin;
+
+    public SellAllCommand(SellPlugin plugin) {
+        this.plugin = plugin;
+    }
+
+    @Override
+    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
+        if (!(sender instanceof Player player)) {
+            sender.sendMessage(plugin.getConfigManager().getText("player-only-command", "&cOnly players can use this command."));
+            return true;
+        }
+
+        if (!player.hasPermission("sellplugin.use")) {
+            player.sendMessage(plugin.getConfigManager().getMessage("no-permission"));
+            return true;
+        }
+
+        new SellAllGUI(plugin, player).open(player);
+        return true;
+    }
+}

src/main/java/com/yourname/sellplugin/command/SellCommand.java

@@ -1,7 +1,7 @@
 package com.yourname.sellplugin.command;
 
 import com.yourname.sellplugin.SellPlugin;
-import com.yourname.sellplugin.gui.SellGUI;
+import com.yourname.sellplugin.gui.ShopMainGUI;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandExecutor;
 import org.bukkit.command.CommandSender;
@@ -16,21 +16,28 @@ public SellCommand(SellPlugin plugin) {
 
     @Override
     public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
-        if (!(sender instanceof Player)) {
-            sender.sendMessage("Only players can use this command.");
+        // Handle /sell reload
+        if (args.length > 0 && args[0].equalsIgnoreCase("reload")) {
+            if (!sender.hasPermission("sellplugin.reload")) {
+                sender.sendMessage(plugin.getConfigManager().getText("reload-no-permission", "&cYou do not have permission to reload the config."));
+                return true;
+            }
+            plugin.getConfigManager().reload();
+            sender.sendMessage(plugin.getConfigManager().getText("reload-success", "&aSellPlugin configuration reloaded."));
             return true;
         }
 
-        Player player = (Player) sender;
+        if (!(sender instanceof Player player)) {
+            sender.sendMessage(plugin.getConfigManager().getText("player-only-command", "&cOnly players can use this command."));
+            return true;
+        }
 
         if (!player.hasPermission("sellplugin.use")) {
             player.sendMessage(plugin.getConfigManager().getMessage("no-permission"));
             return true;
         }
 
-        SellGUI gui = new SellGUI(plugin, player);
-        gui.open(player);
-
+        new ShopMainGUI(plugin, player).open(player);
         return true;
     }
 }

src/main/java/com/yourname/sellplugin/command/SellMultiCommand.java

@@ -0,0 +1,37 @@
+package com.yourname.sellplugin.command;
+
+import com.yourname.sellplugin.SellPlugin;
+import com.yourname.sellplugin.gui.SellMultiGUI;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandExecutor;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+
+public class SellMultiCommand implements CommandExecutor {
+    private final SellPlugin plugin;
+
+    public SellMultiCommand(SellPlugin plugin) {
+        this.plugin = plugin;
+    }
+
+    @Override
+    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
+        if (!(sender instanceof Player player)) {
+            sender.sendMessage(plugin.getConfigManager().getText("player-only-command", "&cOnly players can use this command."));
+            return true;
+        }
+
+        if (!plugin.getConfigManager().isProgressGuiEnabled()) {
+            player.sendMessage(plugin.getConfigManager().getText("feature-disabled", "&cThis feature is currently disabled."));
+            return true;
+        }
+
+        if (!player.hasPermission("sellplugin.use")) {
+            player.sendMessage(plugin.getConfigManager().getMessage("no-permission"));
+            return true;
+        }
+
+        new SellMultiGUI(plugin, player).open(player);
+        return true;
+    }
+}

src/main/java/com/yourname/sellplugin/command/ShowWorthCommand.java

@@ -0,0 +1,88 @@
+package com.yourname.sellplugin.command;
+
+import com.yourname.sellplugin.SellPlugin;
+import com.yourname.sellplugin.util.Scheduler;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandExecutor;
+import org.bukkit.command.CommandSender;
+import org.bukkit.command.TabCompleter;
+import org.bukkit.entity.Player;
+
+import java.util.List;
+import java.util.stream.Collectors;
+import java.util.stream.Stream;
+
+/**
+ * {@code /showworth [true|false]} — lets a player show or hide the worth
+ * tooltip for themselves. With no argument it toggles the current state.
+ */
+public class ShowWorthCommand implements CommandExecutor, TabCompleter {
+
+    private final SellPlugin plugin;
+
+    public ShowWorthCommand(SellPlugin plugin) {
+        this.plugin = plugin;
+    }
+
+    @Override
+    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
+        if (!(sender instanceof Player player)) {
+            sender.sendMessage(plugin.getConfigManager().getText(
+                    "player-only-command", "&cOnly players can use this command."));
+            return true;
+        }
+
+        if (!player.hasPermission("sellplugin.use")) {
+            player.sendMessage(plugin.getConfigManager().getMessage("no-permission"));
+            return true;
+        }
+
+        boolean nowVisible;
+        if (args.length == 0) {
+            nowVisible = plugin.getWorthVisibilityManager().toggle(player.getUniqueId());
+        } else {
+            Boolean parsed = parseBoolean(args[0]);
+            if (parsed == null) {
+                player.sendMessage(plugin.getConfigManager().getText(
+                        "showworth-usage", "&cUsage: /showworth [true|false]"));
+                return true;
+            }
+            nowVisible = parsed;
+            plugin.getWorthVisibilityManager().setVisible(player.getUniqueId(), nowVisible);
+        }
+
+        String key = nowVisible ? "showworth-enabled" : "showworth-disabled";
+        String def = nowVisible
+                ? "&aItem worth is now &lshown&r&a in your inventory."
+                : "&eItem worth is now &lhidden&r&e in your inventory.";
+        player.sendMessage(plugin.getConfigManager().getText(key, def));
+
+        // Resend the inventory so the change is reflected immediately. Runs on
+        // the player's own region thread for Folia compatibility.
+        Scheduler.runEntityLater(plugin, player, () -> {
+            if (player.isOnline()) {
+                player.updateInventory();
+            }
+        }, 1L);
+        return true;
+    }
+
+    private Boolean parseBoolean(String arg) {
+        String a = arg.toLowerCase();
+        return switch (a) {
+            case "true", "on", "show", "yes", "enable", "enabled" -> Boolean.TRUE;
+            case "false", "off", "hide", "no", "disable", "disabled" -> Boolean.FALSE;
+            default -> null;
+        };
+    }
+
+    @Override
+    public List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args) {
+        if (args.length == 1) {
+            return Stream.of("true", "false")
+                    .filter(s -> s.startsWith(args[0].toLowerCase()))
+                    .collect(Collectors.toList());
+        }
+        return List.of();
+    }
+}

src/main/java/com/yourname/sellplugin/command/TopSellCommand.java

@@ -0,0 +1,38 @@
+package com.yourname.sellplugin.command;
+
+import com.yourname.sellplugin.SellPlugin;
+import com.yourname.sellplugin.gui.TopSellGUI;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandExecutor;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+
+public class TopSellCommand implements CommandExecutor {
+
+    private final SellPlugin plugin;
+
+    public TopSellCommand(SellPlugin plugin) {
+        this.plugin = plugin;
+    }
+
+    @Override
+    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
+        if (!(sender instanceof Player player)) {
+            sender.sendMessage(plugin.getConfigManager().getText("player-only-command", "&cOnly players can use this command."));
+            return true;
+        }
+
+        if (!plugin.getConfigManager().isTopSellEnabled()) {
+            player.sendMessage(plugin.getConfigManager().getText("feature-disabled", "&cThis feature is currently disabled."));
+            return true;
+        }
+
+        if (!player.hasPermission("sellplugin.topsell")) {
+            player.sendMessage(plugin.getConfigManager().getMessage("no-permission"));
+            return true;
+        }
+
+        new TopSellGUI(plugin, player, 0).open(player);
+        return true;
+    }
+}