Using switch statement on same material

I have a thought that this won't work because I am using the item meta. If this is wrong, can anyone tell me an alternative way to go?

Code:java

import java.util.ArrayList;
import java.util.List;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.Plugin;
 
public class LobbyMenuGui implements Listener{
    Visibility vsB = new Visibility();
 
    public void onEnable(){
        Bukkit.getServer().getPluginManager().registerEvents(this, (Plugin) this);
    }
 
    private void openMainGui(Player player){
        Inventory inv = Bukkit.createInventory(null, 9, "Lobby Menu");
 
        //Visibility
        ItemStack visibility = new ItemStack(Material.ENDER_PEARL);
        ItemMeta vsMeta =  visibility.getItemMeta();
        vsMeta.setDisplayName("Player Visibility");
        List<String> vsLore = new ArrayList<String>();
        String vsLLine1 = ChatColor.GRAY + "Toggle player visibility";
        vsLore.add(vsLLine1);
        vsMeta.setLore(vsLore);
        visibility.setItemMeta(vsMeta);
 
        inv.setItem(0, visibility);
    }
 
    private ItemStack vsNone = new ItemStack(Material.ENDER_PEARL);
    private ItemStack vsPremium = new ItemStack(Material.ENDER_PEARL);
    private ItemStack vsAll = new ItemStack(Material.ENDER_PEARL);
 
    private void openPVGui(Player player){
        Inventory inv = Bukkit.createInventory(null, 9, "Player Visibility");
 
        //View none
        ItemMeta vsNMeta =  vsNone.getItemMeta();
        vsNMeta.setDisplayName("View none.");
        List<String> vsNLore = new ArrayList<String>();
        String vsLNLine1 = ChatColor.GRAY + "Hides all players.";
        vsNLore.add(vsLNLine1);
        vsNMeta.setLore(vsNLore);
        vsNone.setItemMeta(vsNMeta);
 
        //View premium only
        ItemMeta vsPMeta =  vsPremium.getItemMeta();
        vsPMeta.setDisplayName("View premium only.");
        List<String> vsPLore = new ArrayList<String>();
        String vsLPLine1 = ChatColor.GRAY + "Shows only premium members.";
        vsPLore.add(vsLPLine1);
        vsPMeta.setLore(vsPLore);
        vsPremium.setItemMeta(vsPMeta);
 
        //View all
        ItemMeta vsAllMeta =  vsAll.getItemMeta();
        vsAllMeta.setDisplayName("View all.");
        List<String> vsAllLore = new ArrayList<String>();
        String vsLALine1 = ChatColor.GRAY + "Shows all players.";
        vsAllLore.add(vsLALine1);
        vsAllMeta.setLore(vsAllLore);
        vsAll.setItemMeta(vsAllMeta);
 
        inv.setItem(0, vsNone);
        inv.setItem(4, vsPremium);
        inv.setItem(8, vsAll);
    }
 
    public void onInventoryClick(InventoryClickEvent event){
        Player player = (Player) event.getWhoClicked();
        event.setCancelled(true);
 
        switch (event.getCurrentItem().getData().toItemStack().getItemMeta().getDisplayName()){
        case "Player Visibility":
            player.closeInventory();
            openPVGui(player);
        case "View none.":
            vsNone = new ItemStack(Material.EYE_OF_ENDER);
            vsPremium = new ItemStack(Material.ENDER_PEARL);
            vsAll = new ItemStack(Material.ENDER_PEARL);
 
            vsB.playerVisibility(player, 1);
            player.closeInventory();
        case "View premium only.":
            vsNone = new ItemStack(Material.ENDER_PEARL);
            vsPremium = new ItemStack(Material.EYE_OF_ENDER);
            vsAll = new ItemStack(Material.ENDER_PEARL);
 
            vsB.playerVisibility(player, 2);
            player.closeInventory();
        case "View all.":
            vsNone = new ItemStack(Material.ENDER_PEARL);
            vsPremium = new ItemStack(Material.ENDER_PEARL);
            vsAll = new ItemStack(Material.EYE_OF_ENDER);
 
            vsB.playerVisibility(player, 3);
            player.closeInventory();
        default:
            break;
 
        }
    }
 
    @SuppressWarnings("deprecation")
    public void onPlayerInteract(PlayerInteractEvent event){
        ItemStack is = event.getItem();
        Action a = event.getAction();
 
        if (a == Action.PHYSICAL || is.getType() == Material.AIR){
            return;
        }
 
        if (is.getType().getId() == 347){
            openMainGui(event.getPlayer());
        }
    }
}
 

 

Is...is that your main class?

If so, it doesn't extend JavaPlugin.

If not, there is no onEnable method to override nor can the class be cast to a Plugin.

Also, this is a perfect example of why people should use @Override.

 

Kend2000
You should be registering the events in your Main class's onEnable method. Also you are missing the @EventHandler annotation above your event methods. Refer to this: http://wiki.bukkit.org/Event_API_Reference

You should be breaking out of each case (Unless you intend to run all the cases below it or they will "fall through").

 

It is not the main class.

My classes do not need @EventHandler because it is handled by the main class.
Here's one of my classes that works:

Code:java

import java.util.ArrayList;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.Plugin;
 
public class MenuInventoryGui implements Listener{
 
public void onEnable(){
Bukkit.getServer().getPluginManager().registerEvents(this, (Plugin) this);
}
 
private void teleportInWorld(Player player, int x, int y, int z){
player.teleport(new Location(player.getWorld(), x, y, z));
}
 
private void openGui(Player player){
Inventory inv = Bukkit.createInventory(null, 45, "Game Menu");
 
Ranks rank = new Ranks();
Currency currency = new Currency();
 
//Survival Games
ItemStack sg = new ItemStack(Material.IRON_SWORD);
ItemMeta sgMeta = sg.getItemMeta();
sgMeta.setDisplayName(ChatColor.GOLD + "" + ChatColor.BOLD + "" +
ChatColor.UNDERLINE + "Survival Games");
 
sg.setItemMeta(sgMeta);
 
//Arcade
ItemStack arcade = new ItemStack(Material.SLIME_BALL);
ItemMeta arcadeMeta = arcade.getItemMeta();
arcadeMeta.setDisplayName(ChatColor.GOLD + "" + ChatColor.BOLD + "" +
ChatColor.UNDERLINE + "Arcade");
 
arcade.setItemMeta(arcadeMeta);
 
//PVP Games
ItemStack pvp = new ItemStack(Material.BOW);
ItemMeta pvpMeta = pvp.getItemMeta();
pvpMeta.setDisplayName(ChatColor.GOLD + "" + ChatColor.BOLD + "" +
ChatColor.UNDERLINE + "PVP Games");
 
pvp.setItemMeta(pvpMeta);
 
//Back to hub
ItemStack hub = new ItemStack(Material.PORTAL);
ItemMeta hubMeta = hub.getItemMeta();
hubMeta.setDisplayName(ChatColor.WHITE + "" + ChatColor.BOLD + "" + "Back to hub");
hub.setItemMeta(hubMeta);
 
//Help
ItemStack help = new ItemStack(Material.BOOKSHELF);
ItemMeta helpMeta = help.getItemMeta();
helpMeta.setDisplayName(ChatColor.WHITE + "" + ChatColor.BOLD + "" + "Help");
help.setItemMeta(helpMeta);
 
//Player Stats
ItemStack plrs = new ItemStack(Material.WRITTEN_BOOK);
ItemMeta plrsMeta = plrs.getItemMeta();
plrsMeta.setDisplayName(ChatColor.GREEN + "Player Stats");
java.util.List<String> plrslore = new ArrayList<String>();
String pllore1 = ChatColor.GOLD + "Coins: " + currency.getCoins(player);
String pllore2 = ChatColor.GREEN +"Membership: " + rank.getRank(player);
plrslore.add(pllore1);
plrslore.add(pllore2);
plrsMeta.setLore(plrslore);
plrs.setItemMeta(plrsMeta);
 
//Visit Website
ItemStack vs = new ItemStack(Material.COMMAND);
ItemMeta vsMeta = vs.getItemMeta();
vsMeta.setDisplayName(ChatColor.WHITE + "" + ChatColor.BOLD + "" + "Visit Website");
vs.setItemMeta(vsMeta);
 
//Set the items
inv.setItem(4, sg);
inv.setItem(3, arcade);
inv.setItem(5, pvp);
inv.setItem(22, hub);
inv.setItem(35, help);
inv.setItem(40, plrs);
inv.setItem(44, vs);
 
player.openInventory(inv);
}
 
public void onInventoryClick(InventoryClickEvent event){
Player player = (Player) event.getWhoClicked();
event.setCancelled(true);
 
switch(event.getCurrentItem().getType()){
case IRON_SWORD:
teleportInWorld(player, 1024, 55, 1121);
player.closeInventory();
break;
case SLIME_BALL:
teleportInWorld(player, 970, 55, 1175);
player.closeInventory();
break;
case BOW:
teleportInWorld(player, 1024, 55, 1231);
player.closeInventory();
break;
case PORTAL:
player.performCommand("spawn");
player.closeInventory();
break;
case BOOKSHELF:
player.performCommand("anhelp");
player.closeInventory();
break;
case COMMAND:
player.performCommand("website");
player.closeInventory();
break;
default:
break;
}
}
 
@EventHandler
public void onPlayerInteract(PlayerInteractEvent event){
ItemStack is = event.getItem();
Action a = event.getAction();
 
if (a == Action.PHYSICAL || is.getType() == Material.AIR){
return;
}
 
if (is.getType() == Material.COMPASS){
openGui(event.getPlayer());
}
}
}
 

 

Well post your main class, then, I obviously don't have enough context.

Either way, you can't just magically cast the listener class to a Plugin and expect it to work, the listener class isn't a Plugin.

 

Here's the main class then:

Code:java

import java.io.File;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.logging.Logger;
 
import net.milkbowl.vault.chat.Chat;
import net.milkbowl.vault.economy.Economy;
import net.milkbowl.vault.permission.Permission;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.AsyncPlayerChatEvent;
import org.bukkit.event.player.PlayerDropItemEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.event.player.PlayerQuitEvent;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.entity.EntityDamageEvent;
import org.bukkit.event.entity.ItemSpawnEvent;
import org.bukkit.event.entity.ProjectileLaunchEvent;
import org.bukkit.event.player.PlayerToggleFlightEvent;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.BookMeta;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.Plugin;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Anaminus extends JavaPlugin implements Listener{
 
    Currency currency = new Currency();
    DoubleJump dJump = new DoubleJump();
    Mail mail = new Mail();
    ManageScoreboard msb = new ManageScoreboard();
    MenuInventoryGui menuGUI = new MenuInventoryGui();
    MOTD motd = new MOTD();
    PlayerExtras pExtras = new PlayerExtras();
    Ranks rank = new Ranks();
    Stacker stacker = new Stacker();
    ShopInventoryGui shopGui = new ShopInventoryGui();
    Gadgets gadgets = new Gadgets();
    ChatFilter chatF = new ChatFilter();
    StringBuilder stringB = new StringBuilder();
    GrappleHook grappleH = new GrappleHook();
    LobbyMenuGui lobbyGUI = new LobbyMenuGui();
 
    File file = new File(getDataFolder() + File.separator + "config.yml");
    public static Plugin instance;
    public static Bukkit plugin;
    public final Logger logger = Logger.getLogger("Minecraft");
    public static Economy econ = null;
    public static Permission perms = null;
    public static Chat chat = null;
 
    @Override
    public void onDisable(){
        getLogger().info("Anaminus is now disabled!");
        this.saveConfig();
    }
 
    @Override
    public void onEnable(){   
        instance = this;
 
        getLogger().info("Anaminus Loaded.");
 
        if (!file.exists()){
            this.getLogger().info("Generating config.yml...");
 
 
            this.getConfig().addDefault("Anaminus.Options", "");
            this.getConfig().addDefault("Anaminus.Options.BarMessageColor", "Green");
            this.getConfig().addDefault("Anaminus.Options.BarMessage", "Grand Opening!");
            this.getConfig().addDefault("Anaminus.Options.BarHealth", 300.0);
            this.getConfig().addDefault("Anaminus.UserData", "");
 
            this.getConfig().options().copyDefaults(true);
            this.saveConfig();
        }
 
        this.saveConfig();
 
        this.getServer().getPluginManager().registerEvents(this, this);
    }
 
    public FileConfiguration getCustomConfig() {
        if (this.getConfig() == null) {
            reloadConfig();
        }
        return this.getConfig();
    }
 
    @SafeVarargs
    public static <E> Object[] unpack(E... objects) {
        List<Object> list = new ArrayList<Object>();
        for (Object object : objects) {
            if (object instanceof Object[]) {
                list.addAll(Arrays.asList((Object[]) object));
            }
            else{
                list.add(object);
            }
        }
 
        return list.toArray(new Object[list.size()]);
    }
 
    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event){
        //Currency
        currency.loadCoins(event);
        //Ranks
        rank.findRank(event);
        //Draw Main Scoreboard
        msb.drawMainSB(event);
        //PXtras
        pExtras.onPlayerJoin(event);
        //Mail
        mail.welcomeMail(event);
        mail.notifyMail(event);
 
        getConfig().addDefault("Anaminus.UserData." + event.getPlayer().getName() + ".BoughtGadgets", gadgets.bought);
        getConfig().addDefault("Anaminus.UserData." + event.getPlayer().getName() + ".Mail", mail.MailBox);
        getConfig().addDefault("Anaminus.UserData." + event.getPlayer().getName() + ".Membership", ChatColor.stripColor(rank.getRank(event.getPlayer()).toString()));
        if (!event.getPlayer().hasPlayedBefore()){
            getConfig().addDefault("Anaminus.UserData." + event.getPlayer().getName() + ".Coins", 0);
        }
 
        this.saveDefaultConfig();
        this.saveConfig();
    }
 
    @EventHandler
    public void onPlayerQuit(PlayerQuitEvent event){
        //Currency
        currency.saveCoins(event);
    }
 
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event){
        //Gui
        menuGUI.onPlayerInteract(event);
        gadgets.onPlayerInteract(event);
        lobbyGUI.onPlayerInteract(event);
    }
 
    @EventHandler
    public void onPlayerToggleFlightEvent(PlayerToggleFlightEvent event){
        if (event.getPlayer().hasPermission("anaminus.gold")){
            dJump.onPlayerToggleFlight(event);
        }
    }
 
    @EventHandler
    public void onPlayerMove(PlayerMoveEvent event){
        if (event.getPlayer().hasPermission("anaminus.gold")){
            dJump.onPlayerMove(event);
        }
    }
 
    @EventHandler
    public void onInventoryClick(InventoryClickEvent event){
        if (ChatColor.stripColor(event.getInventory().getName()).equalsIgnoreCase("Game Menu")){
            menuGUI.onInventoryClick(event);
        }
        if (ChatColor.stripColor(event.getInventory().getName()).equalsIgnoreCase("Gadgets")){
            gadgets.onInventoryClick(event);
        }
        if (ChatColor.stripColor(event.getInventory().getName()).equalsIgnoreCase("Lobby Menu")){
            lobbyGUI.onInventoryClick(event);
        }
    }
 
    @EventHandler
    public void onEDBE(EntityDamageByEntityEvent event){
        stacker.ridePlayers(event);
    }
 
    @EventHandler
    public void onPlayerDrop(PlayerDropItemEvent event){
        pExtras.onPlayerDrop(event);
    }
 
    @EventHandler
    public void onItemSpawn(ItemSpawnEvent event){
        pExtras.onItemSpawn(event);
    }
 
    @EventHandler
    public void onPlayerChat(AsyncPlayerChatEvent event){
        chatF.onPlayerChat(event);
    }
 
    @EventHandler
    public void onProjectileThrow(ProjectileLaunchEvent event){
        grappleH.onGrappleThrow(event);
    }
 
    @EventHandler
    public void onEntityDamage(EntityDamageEvent event){
        grappleH.onEntityDamage(event);
    }
 
    @SuppressWarnings("deprecation")
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
        //Note: There is 10 lines of text that the player can see in unfocused mode.
        String prefix = ChatColor.BLUE + "Commands> ";
 
        //Help
        if (label.equalsIgnoreCase("anhelp")){
            if (sender.hasPermission("anaminus.free")){
                sender.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                sender.sendMessage(ChatColor.BLUE + "" + ChatColor.BOLD + "Anaminus " +
                    ChatColor.GOLD + "" + ChatColor.BOLD + "Network");
                sender.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                sender.sendMessage(ChatColor.BLUE + "/about" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Info about the server");
                sender.sendMessage(ChatColor.BLUE + "/donate" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Donate to the server");
                sender.sendMessage(ChatColor.BLUE + "/buy" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Gets Anaminus' Shop");
                sender.sendMessage(ChatColor.BLUE + "/coins" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Displays your current coin balance");
                sender.sendMessage(ChatColor.BLUE + "/membership" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Displays your memebership level");
                sender.sendMessage(ChatColor.BLUE + "/restoreitems" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Lost your items? Get them back with this command");
                sender.sendMessage(ChatColor.BLUE + "/mail" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Displays your mail commands");
                sender.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
            }
        }
 
        //Website
        if (label.equalsIgnoreCase("website")){
            sender.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
            sender.sendMessage("");
            sender.sendMessage(ChatColor.BLUE + "Website: [url]http://anaminus.enjin.com/[/url]");
            sender.sendMessage("");
            sender.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
        }
 
        //Report
        if (label.equalsIgnoreCase("report")){
            if (args.length == 2){
                Player target = getServer().getPlayer(args[1]);
                if (target != null){
                    String reason = args[2];
                    if (reason != null){
                        //TODO: Send to admin's mail and owner's email
                        //Email
                    }
                }else{
                    sender.sendMessage(ChatColor.RED + "Cannot find player!");
                }
            }
        }
 
        //About
        if (label.equalsIgnoreCase("about")){
            sender.sendMessage("");
            sender.sendMessage("");
            sender.sendMessage(ChatColor.GOLD + "Anaminus build: " + String.format("%s", getDescription().getVersion()));
            sender.sendMessage(ChatColor.GOLD + "Developed by: Kend2000");
            sender.sendMessage("");
            sender.sendMessage(ChatColor.GOLD + "Anaminus plugin is the Anaminus Network's main plugin.");
        }
 
        if (sender instanceof Player){
            Player player = (Player) sender;
            //Heal
            if (label.equalsIgnoreCase("heal")){
                    if (player.hasPermission("anaminus.heal")){
                        player.setHealth(20.0);
                        player.sendMessage(ChatColor.YELLOW + "You have been healed.");
                    }
            }
            //eat
            if (label.equalsIgnoreCase("eat")){
                    if (player.hasPermission("anaminus.feed")){
                        player.setFoodLevel(20);
                        player.sendMessage(ChatColor.YELLOW + "You have been fed.");
                    }
            }
 
            //Coins
            if (label.equalsIgnoreCase("coins")){
                    player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                    player.sendMessage("");
                    player.sendMessage(ChatColor.BLUE + " Your current balance: " + ChatColor.GOLD + currency.getCoins(player) + ChatColor.BLUE + " coins.");
                    player.sendMessage(ChatColor.BLUE + " Purchase more at [url]http://anaminus.enjin.com/shop[/url]");
                    player.sendMessage("");
                    player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
            }
            //Donate
            if (label.equalsIgnoreCase("donate")){
                    if (!(player.hasPermission("anaminus.basicpremium") || player.hasPermission("anaminus.goldPremium") || player.hasPermission("anaminus.ultraPremium"))){
                    player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                    player.sendMessage("");
                    player.sendMessage(ChatColor.BLUE + "Buy a membership at [url]http://anaminus.enjin.com/shop[/url]");
                    player.sendMessage("");
                    player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                    }else{
                        player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                        player.sendMessage("");
                        player.sendMessage(ChatColor.BLUE + "You already have a membership.");
                        player.sendMessage(ChatColor.BLUE + "If you want to upgrade then go to [url]http://anaminus.enjin.com/shop[/url]");
                        player.sendMessage("");
                        player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                    }
            }
            //Membership
            if (label.equalsIgnoreCase("membership")){
                player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                player.sendMessage("");
                player.sendMessage(ChatColor.BLUE + "" + ChatColor.BOLD + "Your membership level is: " + rank.getRank(player));
                player.sendMessage("");
                player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
            }
            //Mail
            if (label.equalsIgnoreCase("mail")){
                player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                player.sendMessage(ChatColor.BLUE + "" + ChatColor.WHITE + "Mail");
                player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                player.sendMessage(ChatColor.BLUE + "/maildisplay" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Displays mail");
                player.sendMessage(ChatColor.BLUE + "/mailclear" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Clears mailbox");
                player.sendMessage(ChatColor.BLUE + "/mailsend" + ChatColor.WHITE + " - " + ChatColor.BLUE + "Sends mail to desired player");
                player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
            }
            //MailDisplay
            if (label.equalsIgnoreCase("maildisplay")){
                Mail mail = new Mail();
 
                player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                player.sendMessage(ChatColor.BLUE + "" + ChatColor.WHITE + "Mail");
                player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.STRIKETHROUGH + "------------------------------------------");
                player.sendMessage(mail.getMail(player).toString());
            }
            //MailClear
            if (label.equalsIgnoreCase("mailclear")){
                mail.clearMail(player);
                player.sendMessage(ChatColor.GOLD + "Your mailbox has been cleared");
            }
            //MailSend
            if (label.equalsIgnoreCase("mailsend")){
                if (args.length == 2){
                    Player target = (Player) getServer().getOfflinePlayer(args[1]);
                    if (target != null){
                        String msg = args[2];
                        if (msg != null){
                            mail.sendMail(player, target, msg);
                        }
                    }else{
                        player.sendMessage(ChatColor.RED + "Player cannot be found!");
                    }
                }
            }
            //restoreitems
            if (label.equalsIgnoreCase("restoreitems")){
                if (!(player.getInventory().contains(Material.COMPASS))){
                    ItemStack gamemenu = new ItemStack(Material.COMPASS);
                    ItemMeta gamemenuMeta = gamemenu.getItemMeta();
                    gamemenuMeta.setDisplayName(ChatColor.GOLD + "" + "" +
                    "- " + ChatColor.WHITE + "Game Menu" +
                            ChatColor.GOLD + "" +  " -");
 
                    gamemenu.setItemMeta(gamemenuMeta);
                    player.getInventory().setItem(0, gamemenu);
                }
                if (!(player.getInventory().contains(347))){
                    ItemStack lobbyMenu = new ItemStack(347);
                    ItemMeta lobbyMenuMeta = lobbyMenu.getItemMeta();
                    lobbyMenuMeta.setDisplayName(ChatColor.GOLD + "" + "" +
                            "- " + ChatColor.WHITE + "Lobby Menu" +
                                ChatColor.GOLD + "" +  " -");
 
                    lobbyMenu.setItemMeta(lobbyMenuMeta);
                    player.getInventory().setItem(1, lobbyMenu);
                }
                if (!(player.getInventory().contains(Material.BONE))){
                    ItemStack petShop = new ItemStack(Material.BONE);
                    ItemMeta petShopMeta = petShop.getItemMeta();
                    petShopMeta.setDisplayName(ChatColor.GOLD + "" + "" +
                            "- " + ChatColor.WHITE + "Pet Menu" +
                                ChatColor.GOLD + "" +  " -");
 
                    petShop.setItemMeta(petShopMeta);
                    player.getInventory().setItem(3, petShop);
                }
                if (!(player.getInventory().contains(Material.EMERALD))){
                    ItemStack anShop = new ItemStack(Material.EMERALD);
                    ItemMeta anShopMeta = anShop.getItemMeta();
                    anShopMeta.setDisplayName(ChatColor.GOLD + "" + "" +
                            "- " + ChatColor.WHITE + "Shop" +
                                ChatColor.GOLD + "" +  " -");
 
                    anShop.setItemMeta(anShopMeta);
                    player.getInventory().setItem(8, anShop);
                }
                if (!(player.getInventory().contains(Material.WORKBENCH))){
                    ItemStack wardrobe = new ItemStack(Material.WORKBENCH);
                    ItemMeta wardrobeMeta = wardrobe.getItemMeta();
                    wardrobeMeta.setDisplayName(ChatColor.GOLD + "" + "" +
                            "- " + ChatColor.WHITE + "Wardrobe" +
                                ChatColor.GOLD + "" +  " -");
                    wardrobe.setItemMeta(wardrobeMeta);
                    player.getInventory().setItem(7, wardrobe);
                }
                if (!(player.getInventory().contains(Material.WRITTEN_BOOK))){
                    ItemStack achievements = new ItemStack(Material.WRITTEN_BOOK);
                    BookMeta achieveBookMeta = (BookMeta) achievements.getItemMeta();
 
                    achieveBookMeta.setPages(Arrays.asList(ChatColor.AQUA + "" + ChatColor.BOLD + "Achievements\n" + "\n" +
                            ChatColor.GRAY + "" + ChatColor.BOLD + "Hub Achievements\n" +
                            ChatColor.GRAY + ">Join the network!: " + pExtras.wTN + "\n" +
                            ChatColor.GRAY + ""));
                    achieveBookMeta.setAuthor(ChatColor.DARK_RED + "" + ChatColor.BOLD + "Owner");
                    achieveBookMeta.setTitle(ChatColor.GOLD + "" + "" +
                            "- " + ChatColor.WHITE + "Achievements" +
                                ChatColor.GOLD + "" +  " -");
 
                    achievements.setItemMeta(achieveBookMeta);
                    player.getInventory().setItem(6, achievements);
                }
                if (!(player.getInventory().contains(Material.BLAZE_POWDER))){
                    ItemStack gadget = new ItemStack(Material.BLAZE_POWDER);
                    ItemMeta gMeta = gadget.getItemMeta();
                    gMeta.setDisplayName(ChatColor.GOLD + "" + "" +
                            "- " + ChatColor.WHITE + "Gadgets" +
                                ChatColor.GOLD + "" +  " -");
                    gadget.setItemMeta(gMeta);
                    player.getInventory().setItem(5, gadget);
                }
                if (!(player.getInventory().contains(Material.COMMAND))){
                    ItemStack options = new ItemStack(Material.COMMAND);
                    ItemMeta optMeta = options.getItemMeta();
                    optMeta.setDisplayName(ChatColor.GOLD + "" + "" +
                            "- " + ChatColor.WHITE + "Player Options" +
                                ChatColor.GOLD + "" +  " -");
                    options.setItemMeta(optMeta);
                    player.getInventory().setItem(2, options);
                }
                if (!(player.getInventory().contains(102))){
                    ItemStack blankSlot = new ItemStack(102);
                    ItemMeta bSlotMeta = blankSlot.getItemMeta();
                    bSlotMeta.setDisplayName(ChatColor.GOLD + "" + "" +
                            "- " + ChatColor.GRAY + "Empty Gadget Slot" +
                                ChatColor.GOLD + "" +  " -");
                    blankSlot.setItemMeta(bSlotMeta);
                    player.getInventory().setItem(4, blankSlot);
                }
            }
            //register
            if (label.equalsIgnoreCase("register")){
                if (args.length == 1){
                    String playerName = player.getName();
                    String email = args[1];
 
                }
            }
            //nick
            if (label.equalsIgnoreCase("nick")){
                if (args.length == 1){
                    if (args[1].contains("&")){
                        args[1].replace("&", "§");
                    }
                    player.setDisplayName(args[1]);
                    player.sendMessage(prefix + ChatColor.GRAY + "Your name is now: " + args[1]);
                }
            }
        }else if (!(sender instanceof Player)){
 
            //AddCoins
            if (label.equalsIgnoreCase("addcoins")){
                int amount;
                if (args.length == 2){
                    if (Integer.parseInt(args[1]) > 0){
                        amount = Integer.parseInt(args[1]);
                        Player target = Bukkit.getPlayer(args[2]);
                        if (target != null){
                            currency.increaseAmountBy(target, amount);
                            sender.sendMessage(ChatColor.GREEN + "Successfully added " + amount + " coins to " + target.getName() + "'s account.");
                        }else{
                            sender.sendMessage(ChatColor.RED + "Cannot find player");
                        }
                    }
                }
            }
            //removecoins
            if (label.equalsIgnoreCase("addcoins")){
                int amount;
                if (args.length == 2){
                    if (Integer.parseInt(args[1]) > 0){
                        amount = Integer.parseInt(args[1]);
                        Player target = Bukkit.getPlayer(args[2]);
                        if (target != null){
                            currency.decreaseAmountBy(target, amount);
                            sender.sendMessage(ChatColor.GREEN + "Successfully removed " + amount + " coins to " + target.getName() + "'s account.");
                        }else{
                            sender.sendMessage(ChatColor.RED + "Cannot find player");
                        }
                    }
                }
            }
        }
        return false;
    }
}
 

 

I'd honestly switch upon ChatColor.stripColor(event.getCurrentItem().getData().toItemStack().getItemMeta().getDisplayName()).toLowerCase(), that way that it's a built-in equalsIgnoreCase and removal of color codes.

Edit: Of course, all your cases would be lowercased, so you'd have to modify that.

 

Thanks!

 

No problem bud, and I'm a fan of the modularization! Keep up the good work!

 

Okay, first of all, stop implementing listener and get rid of all those onEnable methods (they are never called by anything, and if they were called by anything, they would throw ClassCastExceptions because, as I said, you can not cast the classes to Plugins). This is of course unless you actually want to switch to using the event system, in which case you could register them in the onEnable in your main class. You should also pass the main class instance through the constructors of the other classes instead of using that static instance, but whatever. Anyway, I would recommend actually using the event system as it was designed. Along with a few other changes I'd say you should make:

1. Instead of using Logger.getLogger("Minecraft"), JavaPlugin contains a handy getLogger() method that will also prefix any logged messages with your plugin name. Use that instead.
2. What on earth is "public static Bukkit plugin"? I have no idea why that's there, but you should probably remove it.
3. As I said, switch to the proper event system, it's cleaner.
4. Don't print enable/disable messages, the server does that automatically.
5. I'm not even going to ask about that unpack method.

There are probably more things but I don't really have time to read through all of that code.

Oh, and for that original issue, uh...you might want to try actually opening the inventory