Solved "Bug" with soup chests!

So this code works but, if i take soups at the same time with multiple players i can see when they take soup. I think its because i used this

Code:java

soupInv = Bukkit.getServer().createInventory(null, 54, ChatColor.RED + "RefillChest");

it should be player instead of Bukkit but idk how to make it work that way! D:

Here's my code:

Code:java

public class SoupChest implements Listener {
 
    public static Inventory soupInv;
 
    public SoupChest(mainclass plugin)
    {
            soupInv = Bukkit.getServer().createInventory(null, 54, ChatColor.RED + "RefillChest");
            ItemStack SOUP = new ItemStack(Material.MUSHROOM_SOUP);
            ItemMeta isM = SOUP.getItemMeta();
            isM.setDisplayName(ChatColor.GREEN.toString() + ChatColor.BOLD + "Soppa");
            SOUP.setItemMeta(isM);
 
            for(int i = 0; i < 54; i++)
            {
            soupInv.addItem(SOUP);
            }
    }
 
            @EventHandler
            public void PlayerCloseInventory(InventoryCloseEvent e)
            {
                ItemStack SOUP = new ItemStack(Material.MUSHROOM_SOUP);
                ItemMeta isM = SOUP.getItemMeta();
                isM.setDisplayName(ChatColor.GREEN.toString() + ChatColor.BOLD + "Soppa");
                SOUP.setItemMeta(isM);
                    soupInv.clear();
                    for(int i = 0; i < 54; i++)
                    {
                            soupInv.addItem(SOUP);
                    }
            }   
 
            @EventHandler
            public void ChestClicker(PlayerInteractEvent e)
            {
                    if(e.getAction() == Action.RIGHT_CLICK_BLOCK)
                    {
                            if(e.getClickedBlock().getType() == Material.TRAPPED_CHEST)
                            {
                                    Player player = (Player) e.getPlayer();
                                    player.getServer().getPlayer(player.getName()).openInventory(soupInv);
                                    e.setCancelled(true);
                            }
                    }
            }
 
            @EventHandler
            public void onPlayerInteract(PlayerInteractEvent event) {
                Player player = event.getPlayer();           
                if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {
                    if(event.getClickedBlock().getType() == Material.SIGN || event.getClickedBlock().getType () == Material.WALL_SIGN) {
                        Block sign = event.getClickedBlock();
                        Sign s = (Sign)sign.getState();
                        if(s.getLine(1).equalsIgnoreCase(ChatColor.stripColor("Soup"))) {
                            event.setCancelled(true);
                            player.getServer().getPlayer(player.getName()).openInventory(soupInv);
                        }
                    }
                }
            }
 
}
 

 

APPLEisROP Put this

Code:java

soupInv = Bukkit.getServer().createInventory(null, 54, ChatColor.RED + "RefillChest");

Each time they interact/ when they open it.

APPLEisROP
For example

Code:java

 @EventHandler
public void onPlayerInteract(PlayerInteractEvent event) {
Player player = event.getPlayer();
if(event.getAction() == Action.RIGHT_CLICK_BLOCK) {
if(event.getClickedBlock().getType() == Material.SIGN || event.getClickedBlock().getType () == Material.WALL_SIGN) {
//Here
Block sign = event.getClickedBlock();
Sign s = (Sign)sign.getState();
if(s.getLine(1).equalsIgnoreCase(ChatColor.stripColor("Soup"))) {
event.setCancelled(true);
player.getServer().getPlayer(player.getName()).openInventory(soupInv);
}
}
}
}

EDIT by Moderator: merged posts, please use the edit button instead of double posting.