Command is shown after the command is performend

Not sure if the title made sense but basically what is happening is when a user types /gmc it will turn them into creative mode and also say "/gmc". The below is my code:

Code:java

 //if the command is /gmc
if (cmd.getName().equalsIgnoreCase("gmc")){
if(player.hasPermission("mortality.gmc")) {
player.setGameMode(GameMode.CREATIVE);
player.sendMessage(ChatColor.RED + "You are now in gamemode: " + ChatColor.YELLOW + "creative.");
}
} else {
player.sendMessage(ChatColor.RED + "You can't do this.");
}
}
 

The screenshot:

 

gaz1812 Return true.

 

Code:java

//if the command is /gmc
if (cmd.getName().equalsIgnoreCase("gmc")){
    if(player.hasPermission("mortality.gmc")) {
        player.setGameMode(GameMode.CREATIVE);
            player.sendMessage(ChatColor.RED + "You are now in gamemode: " + ChatColor.YELLOW + "creative.");
            return true;
        }
    } else {
        player.sendMessage(ChatColor.RED + "You can't do this.");
        return true;
    }
}

As stated above you need to return true.

 

CaptainBern Thanks! Also by any chance do you know why I keep receiving the permission error message when I have permission for the commands? The creative command works fine but then with survival and adventure mode it keeps printing out the message which I set player.sendMessage(ChatColor.RED + "You can't do this."); if you don't have permission.

Code:java

//if the command is /gmc
        if (cmd.getName().equalsIgnoreCase("gmc")){
                    if(player.hasPermission("mortality.gmc")) {
                        player.setGameMode(GameMode.CREATIVE);
                            player.sendMessage(ChatColor.RED + "You are now in gamemode: " + ChatColor.YELLOW + "CREATIVE.");
                        }
                    } else {
                        player.sendMessage(ChatColor.RED + "You can't do this.");
                        return true;
                    }
 
 
    //if the command is /gms
        if (cmd.getName().equalsIgnoreCase("gms")){
            if(player.hasPermission("mortality.gms")) {
                player.setGameMode(GameMode.SURVIVAL);
                    player.sendMessage(ChatColor.RED + "You are now in gamemode: " + ChatColor.YELLOW + "CREATIVE.");
                }
            } else {
                player.sendMessage(ChatColor.RED + "You can't do this.");
                return true;
            }

 

If the command doesn't equal gmc at the beginning of your code, then the else runs, which prints "You can't do this." in red and then returns true, exiting the method before your gms command runs. I don't think he meant for you to do it in an else statement.

 

Change this

Code:java

 else {
                        player.sendMessage(ChatColor.RED + "You can't do this.");
                        return true;
                    }

To this

Code:java

 
 
else if (!player.hasPermission("mortatility.gmc"))
{
    player.sendMessage(ChatColor.RED + "You can't do this.");
    return true;
}

 

gaz1812
Your brackets are messed up.

 

RainoBoy97 I've fixed it it's alright

 

You should set up the permissions in the plugin.yml file, really.

http://wiki.bukkit.org/Plugin_YAML

 

Garris0n Thanks will do from now on!