Unexpected exception while parsing console command

Okay, so I just made a ban command, and it works in-game, but when I try it in console I get this:

Code:

>ban
[13:55:02 WARN]: Unexpected exception while parsing console command "ban"
org.bukkit.command.CommandException: Unhandled exception executing command 'ban'
in plugin tgMain v0.1
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46) ~[cra
ftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:17
5) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchCommand(CraftServe
r.java:683) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchServerCommand(Craf
tServer.java:670) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        at net.minecraft.server.v1_7_R1.DedicatedServer.aw(DedicatedServer.java:
286) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:2
51) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:5
45) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java
:457) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:6
17) [craftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
Caused by: java.lang.ClassCastException: org.bukkit.craftbukkit.v1_7_R1.command.
ColouredConsoleSender cannot be cast to org.bukkit.entity.Player
        at me.ehArcher.tgMain.MainCommandExecutor.onCommand(MainCommandExecutor.
java:20) ~[?:?]
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44) ~[cra
ftbukkit.jar:git-Bukkit-1.7.2-R0.3-b3020jnks]
        ... 8 more

Here's my ban command:

Code:java

    @SuppressWarnings("deprecation")
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) {
 
        Player player = (Player) sender;
 
        if(cmd.getName().equalsIgnoreCase("ban")) {
 
            if(args.length == 0) {
                player.sendMessage(ChatColor.RED + "Usage: /ban (player)");
            }
 
            if(args.length == 1) {
                Player targetPlayer = Bukkit.getServer().getPlayer(args[0]);
                targetPlayer.setBanned(true);
                targetPlayer.kickPlayer(ChatColor.RED + "You have been banned by " + player.getName());
                Bukkit.broadcastMessage(ChatColor.RED + targetPlayer.getName() + " has been banned by " + player.getName());
            }
            if(args.length == 2) {
                Player targetPlayer = Bukkit.getServer().getPlayer(args[0]);
            }
        }

Please help!

 

Can we see your plugin.yml?
Also next time I suggest using this forum for coding help: http://forums.bukkit.org/forums/plugin-development.5/

 

Code:

name: tgMain
description: tgMain - A core plugin for TinyGames
version: 0.1
main: me.ehArcher.tgMain.tgMain
 
commands:
  kick:
    description: /kick
  ban:
    description: /ban
  unban:
    description: /unban

There.

And my full CommandExecutor

Code:java

package me.ehArcher.tgMain;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
 
public class MainCommandExecutor implements CommandExecutor{
 
    public MainCommandExecutor(tgMain tgMain) {
 
    }
 
    @SuppressWarnings("deprecation")
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) {
 
        Player player = (Player) sender;
 
        if(cmd.getName().equalsIgnoreCase("ban")) {
 
            if(args.length == 0) {
                player.sendMessage(ChatColor.RED + "Usage: /ban (player)");
            }
 
            if(args.length == 1) {
                Player targetPlayer = Bukkit.getServer().getPlayer(args[0]);
                targetPlayer.setBanned(true);
                targetPlayer.kickPlayer(ChatColor.RED + "You have been banned by " + player.getName());
                Bukkit.broadcastMessage(ChatColor.RED + targetPlayer.getName() + " has been banned by " + player.getName());
            }
            if(args.length == 2) {
                Player targetPlayer = Bukkit.getServer().getPlayer(args[0]);
            }
        }
 
        if(cmd.getName().equalsIgnoreCase("silentban")) {
 
            if(args.length == 0) {
                player.sendMessage(ChatColor.RED + "Usage: /ban (player)");
            }
 
            if(args.length == 1) {
                Player targetPlayer = Bukkit.getServer().getPlayer(args[0]);
                targetPlayer.setBanned(true);
                targetPlayer.kickPlayer(ChatColor.RED + "You have been banned by " + player.getName());
            }
 
        }
 
        if(cmd.getName().equalsIgnoreCase("kick")) {
 
            if(args.length == 0) {
                player.sendMessage(ChatColor.RED + "Usage: /kick (player)");
            }
            if(args.length == 1) {
                Player targetPlayer = Bukkit.getServer().getPlayer(args[0]);
                targetPlayer.kickPlayer(ChatColor.RED + "You have been kicked by " + player.getName());
                Bukkit.broadcastMessage(ChatColor.RED + targetPlayer.getName() + " has been kicked by " + player.getName());
            }
 
        }
 
        return false;
    }
}

Code:java

package me.ehArcher.tgMain;
 
import java.util.logging.Logger;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
public class tgMain extends JavaPlugin {
 
    public final Logger logger = Logger.getLogger("Minecraft");
    public PluginDescriptionFile pdfFile = getDescription();
 
    @Override
    public void onEnable() {
        getCommand("ban").setExecutor(new MainCommandExecutor(this));
        getCommand("kick").setExecutor(new MainCommandExecutor(this));
    }
 
    @Override
    public void onDisable() {
    }
 
}
 

 

Moved to correct section.

 

ehArcher
You're casting sender to player before even checking if it's a player who runs the command...
Player player = (Player) sender; disables all console usage.

You should check if the command executor is a player before casting the sender to a player.

 

Code:java

do if (!(sender instanceof Player)){
sender.sendMessage("ChatColor.RED + "Only Players can Execute This Command!");
return false;
}

Use this check in order to get no error in console buddy!
hope it helped!
Heirteir

 

That doesn't really fix the issue. The console should be able to perform the command. You need to remove the line which sets sender to player, and then replace everywhere that uses player to sender.

On the line which broadcasts who banned who, perhaps do a check to see if it is a player who sent the command, and if it was the console, say 'player has been banned by console' instead of just not letting the console ban.