Trouble Creating An Inventory

I am attempting to make a plugin that opens a virtual inventory when you right click a cauldron.
What am I doing wrong?

Here is my code:

Code:java

package me.Rocketboy901.Trashcans;
 
import org.bukkit.entity.Player;
import org.bukkit.event.*;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
 
 
public class Main extends JavaPlugin implements Listener {
 
    @Override
    public void onEnable() {
        Bukkit.getServer().getPluginManager().registerEvents(this, this);
    }
 
    @EventHandler(priority = EventPriority.HIGH)
    public void onInteract(PlayerInteractEvent event) {
        Player player = event.getPlayer();
 
        if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
            if (event.getClickedBlock().equals(Material.CAULDRON)){
                Bukkit.createInventory(player, 18, ChatColor.RED + "" + ChatColor.BOLD + "Trashcan");
 
            }
 
        }
    }
 
}
 

 

Rocketboy901
have the inventory be a variable, and have the player open that inventory variable.

 

Still doesn't work! EDIT: It works if I remove the if (getClickedBlock().equals(Material.Caldren) statment so it has something to do with that...

 

I think you should define the type of the inventory, InventoryType.CHEST, or whatever type you want.

 

It opens an inventory if I remove the if (getClickedBlock().equals(Material.<block>); What should I do?

 

Here's a hint: You are comparing a Block to a Material... you should be comparing a Material to a Material.

 

Rocketboy901

What I told you will work for the Inventory. Just make sure the rest of your code is all good.

 

Create the inventory on your onEnable method, then in your event handler just get the player to open it. At the moment you are just creating the inventory when the block is right clicked and not doing anything else with it.

Hope I helped.

 

Rocketboy901 I would create the inventory, globally, and staticly like so:

Code:java

public static Inventory customInventory = Bukkit.createInventory(player, 18, ChatColor.RED + "" + ChatColor.BOLD + "Trashcan");

Then, you can do (from the Main class)

Code:java

player.openInventory(customInventory);

And from a diffirent class:

Code:java

player.openInventory(Main.customInventory);

 

Here you go:

Code:java

package me.Rocketboy901.Trashcans;
 
import org.bukkit.block.Block;
import org.bukkit.entity.Player;
import org.bukkit.event.*;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
 
public class Trashcans extends JavaPlugin implements Listener {
public static Inventory trashcan;
@Override
public void onEnable() {
Bukkit.getServer().getPluginManager().registerEvents(this, this);
}
 
@EventHandler(priority = EventPriority.HIGH)
public void onInteract(PlayerInteractEvent event) {
Player player = event.getPlayer();
 
if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    Block block = event.getClickedBlock();
if(block.getType() == Material.CAULDRON){
trashcan = Bukkit.createInventory(player, 18, ChatColor.RED + "" + ChatColor.BOLD + "Trashcan");
player.openInventory(Trashcans.trashcan);
 
}
 
}
}
 
}

 

Guys, I fixed this myself a while back! Thanks though.