It's giving the player 2 items instead of 1.

I'm making a plugin for my server that gives that player an item a compass in this case and then they right click and select bla bla bla but for reason it gives me 2 items heres the code

Code:java

@EventHandler(priority=EventPriority.MONITOR)
    public void onPlayerJoinEvent(PlayerJoinEvent event)
    {
        Player p = event.getPlayer();
        if (event.getPlayer().getLocation().getWorld().getName().equals("Spawn"))
        {
            ItemStack spawncompass = new ItemStack(Material.COMPASS, 0);
            ItemMeta stackMeta = spawncompass.getItemMeta();
            stackMeta.setDisplayName(ChatColor.RED + "Quick Join!");
            ArrayList<String> lore = new ArrayList<String>();
            lore.add(ChatColor.GRAY + "If you don't feel like ");
            lore.add(ChatColor.GRAY + "running and jumping in ");
            lore.add(ChatColor.GRAY + "the portals this is the ");
            lore.add(ChatColor.GRAY + "perfect thing for you, All ");
            lore.add(ChatColor.GRAY + "you got to do is right click ");
            lore.add(ChatColor.GRAY + "me and choose the game you");
            lore.add(ChatColor.GRAY + "wanna play!");
            stackMeta.setLore(lore);
            spawncompass.setItemMeta(stackMeta);
            p.getInventory().addItem(spawncompass);
        }
  } 

 

ninja2003

Check to make sure the code isn't run twice, also use Player#updateInventory() to ensure the inventory is consistent with what the serber recognizes it as, finally make sure to remove the item when the player leaves.

 

ninja2003

I'm not 100% sure on this but you might want to try this:

Code:java

ItemStack spawncompass = new ItemStack(Material.COMPASS, 1);

Instead of this:

Code:java

ItemStack spawncompass = new ItemStack(Material.COMPASS, 0);

Sometimes the simplest of things can make all the different.

Regards,
Alex Harris