Solved Opening Player Inventory (Kit Selector)

Hey guys,
This is my code:

Code:java

@EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
        Player player = e.getPlayer();
 
        Inventory inv = Bukkit.createInventory(null, InventoryType.CHEST, ChatColor.GOLD + "Select your kit!");
 
        ItemStack sword = new ItemStack(Material.DIAMOND_SWORD);
        ItemMeta smeta = sword.getItemMeta();
        smeta.setDisplayName(ChatColor.GREEN + "PVP");
        smeta.setLore(Arrays.asList("Choose the PVP kit!"));
 
        inv.setItem(0, sword);
 
        if (e.getAction() != Action.RIGHT_CLICK_AIR || e.getAction() != Action.RIGHT_CLICK_BLOCK) return;
        if (player.getInventory().getItemInHand().getType() != Material.PAPER) return;
 
        if (e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK && player.getInventory().getItemInHand().getType() == Material.PAPER) {
 
            player.openInventory(inv);
        }
    }

And it does not work in game, well, I can't open the inventory when I click on a paper.
I tried checking for if the paper has meta data (which it has) but it still didn't work.
Please help me, I'd appreciate it!
Thanks,
Wiesel.

 

Try putting in some debug messages and seeing where it makes it and, did you register the events?

 

Have you registered your events in your main class ?

 

I would recommend you create a method called invGUI and accept a player input:

Code:java

public void invGUI(Player player){
//Do your GUI here
}

Then just do:
invGUI(player);
in your interact event.

On the other hand; for Checking clicking in the inventory do;

Code:java

public void invClick(InventoryClickEvent e){
Player p = e#getPlayer
switch(e#getCurrentItem#getType){
case ITEM:
// DO STUFF
break;
default:
break;
}
 
}

Make sure to add checks to make sure it's the correct Inventory.

 

Oh, i totally forgot to register the event in the main class silly me thanks for your help tho

 

I tried using your invGUI(player); method, it didn't work.
This is what i did:

Code:java

public void showKitSelector(Player player) {
 
        kitselector = Bukkit.createInventory(null, 9*6, ChatColor.GOLD + "Select your kit!");

........
and in the listener for clicking an item, i did (plugin is a variable of the Main.class) : plugin.KitSelector.showKitSelector(player);

As I said, didnt work out for me. Thanks for your reply though.

 

Erm. Don't tell me you did switch(e#getCurrentItem#getType())..

 

Remember to do:

Code:java

p#openInventory([inv]);

 

Create a field public Inventory Selectkit;
then create it in you onEnable, Selectkit = bukkit.createinventory(null, 54, name);

And in your code, you forgot to set the item meta.
sword.setitemmeta(smeta)

and... You're checking two time the same thing with your e.getaction()

 

Code:java

public Inventory getCustomInventory(){
    Inventory inv = Bukkit.createInventory(null, 9, "lol");
    inv.setItem(0, someItemStack());
    return inv;
}

If you don't need the player object in the inventory, u could do this.
But you could also just do

Code:java

public void openInv(Player p){
    Inventory inv = blabla;
    p.openInventory(inv); //Make sure this is the last line
}

 

Wiesel000 line 14 seems to be always evaluating to true (thus always returning)

And the issue with that is...?

 

If he copy Pasted the #'s.

 

ChipDev ah like that. I thought you were criticising the enum switch itself