Why doesn't this plugin work?

I already have the correct setup, this is just 1 command out of like 5 and they all work so I know this one is weird. Any help?

Code:java

 
if(cmd.getName().equalsIgnoreCase("message")) {
if (args.length < 2) {
p.sendMessage("§cIncorect syntax. Usage: /message <player> <message>");
}else{
Player targetPlayer = p.getServer().getPlayer(args[1]);
if (targetPlayer == null) {
p.sendMessage("§c" + args[1] + "§c is offline");
}else{
targetPlayer.sendMessage("§a[§a§l" + p.getName() + " whispers§a]:§f " + args[2]);
p.sendMessage("§a[§a§lYou whisper to " + targetPlayer.getName() + "§a]:§f " + args[2]);
}

 

You should probably change .getPlayer(args[1]) to .getPlayer(args[0]) as well as change args.length < 1 to args.length <2. I would also recommend checking if targetPlayer is not null, but that is not necessary for it to work.

 

I did change both of those previously, but they didn't work so I tried changing it and thanks for the idea of checking if the player s null

edit: I changed the code but still doesn't work.

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

 

Paste edited code and explain error.

 

Did you register the command 'message'?

This is what should be working:

Code:java

@Override
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
String message = "";
 
if(sender instanceof Player) {
Player player = (Player)sender;
 
if(label.equalsIgnoreCase("message")) {
if(args.length <= 0) {
player.sendMessage(ChatColor.RED + "Incorrect syntax. Usage: /message <player> [message]");
return true;
} else if(args.length == 1) {
player.sendMessage(ChatColor.RED + "Provide a message");
return true;
}
 
if(args.length >= 2) { //Checking for more args because message contains space
Player targetPlayer = player.getServer().getPlayer(args[0]);
 
if(targetPlayer == null) {
player.sendMessage(ChatColor.RED + "The player " + ChatColor.YELLOW + targetPlayer + ChatColor.RED + " cannot be found");
return true;
}
 
for(int i = 1; i < args.length; i++) {
message = message + args[i] + " ";
}
 
player.sendMessage(ChatColor.GREEN + "[ " + ChatColor.DARK_BLUE + "To " + targetPlayer.getName() + ChatColor.GREEN + "]: " + ChatColor.GRAY + message);
targetPlayer.sendMessage(ChatColor.GREEN + "[ " + ChatColor.DARK_BLUE + "From " + player.getName() + ChatColor.GREEN + "] " + ChatColor.GRAY + message);
return true;
}
}
}
}[/i]

 

I'm not sure but I think:

Code:java

Player targetPlayer = p.getServer().getPlayer(args[1]);

Should be:

Code:java

Player targetPlayer = Bukkit.getServer().getPlayer(args[0]);

 

EgyptianKing that was already mentioned.

 

tomudding
Look closer:
p.getServer()
to
Bukkit.getServer().

 

EgyptianKing Technically wouldn't matter since they both return Server