Right Click Compass

I am making a plugin and this is row is getting an error. (Row 58.) Please help me!

Code:java

package com.lucatoni1102.youtube;
 
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.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 InventoryGUI extends JavaPlugin implements Listener{
 
 
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this, 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, 9, ChatColor.DARK_GREEN + "Select your server!");
 
 
        ItemStack prison = new ItemStack (Material.IRON_FENCE);
        ItemMeta prisonMeta = prison.getItemMeta();
        ItemStack minigame = new ItemStack (Material.BEACON);
        ItemMeta  minigameMeta = minigame.getItemMeta();
 
 
        prisonMeta.setDisplayName(ChatColor.GRAY + "PRISON");
        prison.setItemMeta(prisonMeta);
 
        minigameMeta.setDisplayName(ChatColor.RED + "MINIGAMES");
        minigame.setItemMeta(minigameMeta);
 
        inv.setItem(0, prison);
        inv.setItem(1, minigame);
 
        player.openInventory(inv);
        }
    @EventHandler
    public void onInventoryClick(InventoryClickEvent event) {
        if(!ChatColor.stripColor(event.getInventory().getName())
            .equalsIgnoreCase("Select your server!"));
        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 IRON_FENCE:
            teleportInWorld(player, 223, 344, 6);
            player.closeInventory();
            player.sendMessage(String.format("%sTeleported To %sPrison%s!",
                    ChatColor.GOLD, ChatColor.RED, ChatColor.GOLD));
            break;
        case BEACON:
            teleportInWorld(player, 223, 344, 6);
            player.closeInventory();
            player.sendMessage(String.format("%sTeleported To %sMiniGames%s!",
                    ChatColor.BLUE, ChatColor.RED, ChatColor.BLUE));
 
            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 a = event.getAction();
        ItemStack is = event.getItem();
 
        if(a == Action.PHYSICAL || is == null || is.getType()== Material.AIR)
            return;
 
        if(is.getType() == Material.COMPASS);
            openGUI(event.getPlayer());
    }
}
 

 

Remove the Semicolon on line 56.

 

Like Glumpz said, remove the semicolon on line 56,
when using an if statement, you can do this without using brackets if you only need the condition for the next line.
Putting a semicolon there would mean it'll be executing an empty statement, thus doing nothing. After that it'll see the return; and just stop the whole process no matter what.