Solved Won't let me take anything out of my Inventory

I'm making a plugin and i made a compass inventory thing
out of my liking and the compass part works great, but
it wont let me take anything out of my creative Inventory!
Please Help!

Code:java

package me.CraigTheMailman.ProjectX;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Effect;
import org.bukkit.GameMode;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.block.Block;
import org.bukkit.block.BlockFace;
import org.bukkit.block.BlockState;
import org.bukkit.block.Chest;
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.entity.EntityDamageEvent;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.event.player.PlayerToggleFlightEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.java.JavaPlugin;
 
public class ProjectX extends JavaPlugin implements Listener {
 
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, this);
        getLogger().info("Inventory GUI Enabled");
    }
 
    @EventHandler
    public void onPlayerToggleFlight(PlayerToggleFlightEvent event) {
        Player player = event.getPlayer();
        if(player.getGameMode()==GameMode.CREATIVE)
            return;
        event.setCancelled(true);
        player.setAllowFlight(false);
        player.setFlying(false);
        player.setVelocity(player.getLocation().getDirection().multiply(1.5).setY(1));
    }
 
 
    @EventHandler
    public void onPlayerMove(PlayerMoveEvent event) {
        Player player = event.getPlayer();
        if((player.getGameMode()!=GameMode.CREATIVE)
            && (player.getLocation().subtract(0, 1, 0).getBlock().getType()!=Material.AIR)
            && (!player.isFlying()));
        player.setAllowFlight(true);
 
        }
 
    @EventHandler
    public void EntityBlooc(EntityDamageEvent e) {
        e.getEntity().getLocation().getWorld().playEffect(e.getEntity().getLocation().add(0, 1, 0), Effect.STEP_SOUND, 152);
    }
 
    @EventHandler
    public void playerDeath(PlayerDeathEvent e){
        e.getDrops().clear();
 
        ItemStack[] inv = e.getEntity().getInventory().getContents();
        Location loc = e.getEntity().getLocation();
        loc.getBlock().setType(Material.CHEST);
        Block chest = loc.getBlock();
        BlockState state = chest.getState();
        Chest c = (Chest) state;
        if(inv.length >= 27){
            chest.getRelative(BlockFace.NORTH).setType(Material.CHEST);   
        }
 
        c.getInventory().clear();
        c.getInventory().setContents(inv);
    }
 
    public 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 inv1 = Bukkit.createInventory(null, 9, ChatColor.GOLD
                + "Server Selector");
 
        ItemStack SurvSelect = new ItemStack (Material.SAPLING);
        ItemMeta SurvMeta = SurvSelect.getItemMeta();
 
        SurvMeta.setDisplayName(ChatColor.GREEN + "Wilderness");
        SurvSelect.setItemMeta(SurvMeta);
 
        inv1.setItem(1, SurvSelect);
 
        player.openInventory(inv1);
    }
 
    @EventHandler
    public void onInventoryClick(InventoryClickEvent event) {
        if(!ChatColor.stripColor(event.getInventory().getName())
            .equalsIgnoreCase("Server Selector"))
            return;
        Player player = (Player) event.getWhoClicked();
        event.setCancelled(true);
 
        if(event.getCurrentItem()==null || event.getCurrentItem().getType() == Material.AIR) {
            player.closeInventory();
            return;
        }
 
        switch(event.getCurrentItem().getType()) {
        case SAPLING:
            teleportInWorld(player, 0, 0, 0);
            player.closeInventory();
            player.sendMessage(String.format("%s Teleported to %sWilderness%s!",
                    ChatColor.GOLD, ChatColor.DARK_GREEN, ChatColor.GOLD));
 
            break;
        default:
            player.closeInventory();
            break;   
        }
 
    }
 
 
 
    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event) {
        event.getPlayer().getInventory()
            .addItem(new ItemStack(Material.COMPASS));
    }
 
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        Action action = event.getAction();
 
        ItemStack is = event.getItem();
 
        if(action == Action.PHYSICAL || is == null || is.getType() == Material.AIR)
            return;
 
        if(is.getType() == Material.COMPASS)
            openGUI(event.getPlayer());
    }
}
 

No Errors were presented either!

 

CraigTheMman

Hi, it looks like in your Inventory Click listener on line 81, you are cancelling the event every time it happens. Maybe you meant something like this which would only cancel the event if the player isn't in creative mode:

Code:java

 Player player = (Player) event.getWhoClicked();
if (!(player.getGameMode() == GameMode.CREATIVE)) {
event.setCancelled(true);
}

Edit: Nevermind, please refer to the post below me, I misinterpreted what you were doing.

 

Code:java

if(ChatColor.stripColor(event.getInventory().getName())
.equalsIgnoreCase("Server Selector"))
return;

I think you just got this little line wrong, now you are only cancelling the event (and doing the server sending stuff) when the player is NOT in the server selector. So you just need to put an exclamation point to invert the condition of this if-statement.

 

Like this?

Code:java

@EventHandler
    public void onInventoryClick(InventoryClickEvent event) {
        if(!ChatColor.stripColor(event.getInventory().getName())
            .equalsIgnoreCase("Server Selector"))
            return;

 

Exactly