Solved How do you properly use onPlayerJoin() and onPlayerInteract()?

So i'm pretty new to the Bukkit API and I wanted to make a plugin for a hub. In this I wanted it to give a player a block, and If they click with it they go to the server. Here is my code...

Code:java

public class HubMain extends JavaPlugin implements Listener{
 
    Player p;
 
    public Inventory creative = Bukkit.createInventory(null, 9, ChatColor.DARK_BLUE + "Teleport To Creative!");
 
 
    public void onEnable() {
        getLogger().info("Plugin Enabled!");
 
                          }
 
    public void onDisable() {
        getLogger().info("Plugin Disabled!");
 
                            }
 
    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent e) { //called when player joins
        Inventory inv = p.getInventory();
        ItemStack creative = new ItemStack(Material.GRASS);
        ItemMeta creativeMeta = creative.getItemMeta();
        creativeMeta.setDisplayName(ChatColor.AQUA + "Creative!");
        creativeMeta.setLore(Arrays.asList(ChatColor.RED + "Join Creative!"));
        creative.setItemMeta(creativeMeta);
        inv.setItem(0, creative); //(inv being player.getInventory();)
 
        }
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
 
 
        if(e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK || e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_BLOCK){
            Player p = e.getPlayer();
            final ItemStack currentItem = e.getPlayer().getItemInHand();
            if(currentItem != null && currentItem.getType() != Material.AIR) {
                if(currentItem.getItemMeta().getDisplayName() == ChatColor.AQUA + "Creative!"){
            p.chat("/server Creative");
                    e.setCancelled(true);
      }
          }
                }
    }
}
 

I'm not sure where its going wrong, but when I go onto the server it doesn't give any grass block. I've looked around and couldn't find an answer. Any help is great! Thanks!

 

ryryanator You haven't registered the events.
Put "getServer().getPluginManager().registerEvents(this, this);" into your onEnable() method.
It should work after that.

Edit:As a rule of thumb, it's best not to store player variables outside of events, because they can cause memory leaks. (In reference to the Player variables on line 3)

 

Thanks!