InventoryGUI Issue. :/

I am making a GUI for my KitPvP plugin. It loads up the GUI, but I can move things in it around freely and it does not execute the command or close the inventory when I do.
Here is the code for it.

Code:java

public class SimpleKitPvP  extends JavaPlugin implements Listener
{
  public void onEnable()
  {
    getServer().getPluginManager().registerEvents(this, this);
    getLogger().info("SimpleKitPvP has been enabled. - created by coolguy4744");
  }
 
  ArrayList<String> kits = new ArrayList();
  ArrayList<String> NinjaCooldown = new ArrayList();
  ArrayList<String> Ninja = new ArrayList();
 
  public void openGUI(Player player) {
      Inventory inv = Bukkit.createInventory(null, 9,  ChatColor.GREEN
              + "Member Kits" );
 
      ItemStack swordsman = new ItemStack(Material.DIAMOND_SWORD);
      ItemMeta swordsmanMeta = swordsman.getItemMeta();
      ItemStack archer = new ItemStack(Material.BOW);
      ItemMeta archerMeta = archer.getItemMeta();
 
      swordsmanMeta.setDisplayName(ChatColor.BLUE
              + "Swordsman");
      swordsman.setItemMeta(swordsmanMeta);
 
      archerMeta.setDisplayName(ChatColor.BLUE
              + "Archer");
      archer.setItemMeta(archerMeta);
 
      inv.setItem(1, swordsman);
      inv.setItem(2, archer);
 
      player.openInventory(inv);
  }
 
 
@EventHandler
  public void onPlayerJoin(PlayerJoinEvent event) {
      event.getPlayer().getInventory()
              .addItem(new ItemStack(Material.BEDROCK));
  }
@EventHandler
public void onInventoryClick(InventoryClickEvent event) {
      if(ChatColor.stripColor(event.getInventory().getName())
              .equalsIgnoreCase("Member Kits"))
          return;
      Player player = (Player) event.getWhoClicked();
      event.setCancelled(true);
 
      if (event.getCurrentItem() == null
              || event.getCurrentItem().getType() == Material.AIR
              || !event.getCurrentItem().hasItemMeta()) {
          player.closeInventory();
          return;
      }
      switch (event.getCurrentItem().getType()) {
      case DIAMOND_SWORD:
          player.performCommand("Swordsman");
          player.closeInventory();
          break;
      case BOW:
          player.performCommand("Archer");
          player.closeInventory();
          break;
      default:
          player.closeInventory();
          break;
      }
}
 
  @EventHandler
  public void onPlayerInteract(PlayerInteractEvent event) {
      Action a = event.getAction();
      ItemStack is = event.getItem();
 
      if (a == Action.PHYSICAL || is == null || is.getType() == Material.AIR)
          return;
 
      if (is.getType() == Material.BEDROCK)
          openGUI(event.getPlayer());
  }

Please help, also I get errors and it does not work when I add 2 ChatColors. I would like to do

Code:java

ChatColor.BLUE + ChatColor.BOLD + "Text");

 

Sorry on phone but in inventory click event you wrote if the inventory name was the name you wanted to return... Remove that portion of the code

 

For that chatcolor thing I would do:

Code:java

p.sendmessage(Chatcolor.BOLD + "" + Chatcolor.BLUE + "Text")

 

It didn't do anything WD_MUFFINFLUFFER. :/

 

@michael566
@coolguy4744


Also

Code:java

p.sendmessage("§l§9Text");

Source: http://minecraft.gamepedia.com/Color_codes

 

Now remove event.setCanceled(true);
And place it before each break;
Because you are canceling the event before the code checks what item you clicked

Never do chat color like that...

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

WD_MUFFINFLUFFER
It didn't do anything. :/

 

Paste your class here because from what I see you didn't even make the commands yet so obviously it won't do anything, although it should close the inventory. Paste class please

 

WD_MUFFINFLUFFER
I did make the commands, I only showed the GUI. And even I didn't then it would say unknown command but this one doesn't even close the inventory after.

Hmmm, after testing it is not reading this.

Code:java

      }
      switch (event.getCurrentItem().getType()) {
      case Swordsman:
          player.performCommand("Swordsman");
          player.closeInventory();
          event.setCancelled(true);
          break;
      case BOW:
          player.performCommand("Archer");
          player.closeInventory();
          event.setCancelled(true);
          break;
      default:
          player.closeInventory();
          event.setCancelled(true);
          break;
      }
}

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Please paste your class

 

WD_MUFFINFLUFFER
Oh right sorry.

Code:java

package com.coolguy4744.SimpleKitPvP;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
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.event.player.PlayerJoinEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.java.JavaPlugin;
 
public class InvGUI  extends JavaPlugin implements Listener
{
  public void onEnable()
  {
    getServer().getPluginManager().registerEvents(this, this);
  }
 
  public void openGUI(Player player) {
      Inventory inv = Bukkit.createInventory(null, 9,  ChatColor.GREEN + "" + ChatColor.BOLD
              + "Member Kits" );
 
      ItemStack swordsman = new ItemStack(Material.DIAMOND_SWORD);
      ItemMeta swordsmanMeta = swordsman.getItemMeta();
      ItemStack archer = new ItemStack(Material.BOW);
      ItemMeta archerMeta = archer.getItemMeta();
 
      swordsmanMeta.setDisplayName(ChatColor.BLUE
              + "Swordsman");
      swordsman.setItemMeta(swordsmanMeta);
 
      archerMeta.setDisplayName(ChatColor.BLUE
              + "Archer");
      archer.setItemMeta(archerMeta);
 
      inv.setItem(1, swordsman);
      inv.setItem(2, archer);
 
      player.openInventory(inv);
  }
 
 
@EventHandler
  public void onPlayerJoin(PlayerJoinEvent event) {
      event.getPlayer().getInventory()
              .addItem(new ItemStack(Material.BEDROCK));
  }
@EventHandler
public void onInventoryClick(InventoryClickEvent event) {
      if(ChatColor.stripColor(event.getInventory().getName())
              .equalsIgnoreCase("Member Kits"))
          return;
      Player player = (Player) event.getWhoClicked();
 
 
      if (event.getCurrentItem() == null
              || event.getCurrentItem().getType() == Material.AIR
              || !event.getCurrentItem().hasItemMeta()) {
          player.closeInventory();
          return;
      }
      switch (event.getCurrentItem().getType()) {
      case DIAMOND_SWORD:
          player.performCommand("Swordsman");
          player.closeInventory();
          event.setCancelled(true);
          break;
      case BOW:
          player.performCommand("Archer");
          player.closeInventory();
          event.setCancelled(true);
          break;
      default:
          player.closeInventory();
          event.setCancelled(true);
          break;
      }
}
 
  @EventHandler
  public void onPlayerInteract(PlayerInteractEvent event) {
      Action a = event.getAction();
      ItemStack is = event.getItem();
 
      if (a == Action.PHYSICAL || is == null || is.getType() == Material.AIR)
          return;
 
      if (is.getType() == Material.BEDROCK)
          openGUI(event.getPlayer());
  }
}
 
 

 

@WD_MUFFINFLUFFER
You've never used?, Or should I not use the?, Work! and reduces your code much

 

chikenlitle99
I did not understand any of that. XD

 

coolguy4744

Code:

if(ChatColor.stripColor(event.getInventory().getName())
              .equalsIgnoreCase("Member Kits"))
          return;

You are returning if the inventory is "Member Kits", You should be returning if it is not "Member Kits"

 

I mean about the color code

 

Oh right.

xTigerRebornx
What do you mean? Like should I just remove it?

xTigerRebornx
Yes it worked! Thank you so much. ^.^ <3

Thanks everyone!

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Dude I told you that like 5 posts ago..

coolguy4744

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

WD_MUFFINFLUFFER
Ah sorry, I got a bit confused. I know that I am very slow so I thank you for your patience. The person who knows this the most is, WeeziMonkey. XD Thanks guys!