"An internal error occurred..."

Ok, I am making a command that you can do do /clearkit to your self, and someone else. It works fine, except for the fact when you clear your own kit it says this:
An internal error occurred while attempting to perform this command

Code:

Code:java

package com.regablith.klegacykits.commands;
 
import org.bukkit.ChatColor;
 
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.potion.PotionEffect;
 
import com.regablith.klegacykits.kLegacyKits;
 
public class CommandCK implements CommandExecutor {
 
kLegacyKits plugin;
 
public CommandCK(kLegacyKits klegacykits) {
this.plugin = klegacykits;
}
 
public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
Player player = (Player) sender;
if(commandLabel.equalsIgnoreCase("clearkit") || commandLabel.equalsIgnoreCase("ck")){
player.sendMessage(ChatColor.GOLD + "[" + ChatColor.AQUA + "LegacyKits" + ChatColor.GOLD + "] " + ChatColor.GRAY + "You removed your kit, you may now use another one");
kLegacyKits.HasKit.remove(player);
kLegacyKits.JumperKit.remove(player);
kLegacyKits.SpartanKit.remove(player);
kLegacyKits.VampireKit.remove(player);
kLegacyKits.WizardKit.remove(player);
kLegacyKits.HyperKit.remove(player);
kLegacyKits.CrusaderKit.remove(player);
for (PotionEffect effect : player.getActivePotionEffects())
player.removePotionEffect(effect.getType());
player.getInventory().clear();
player.getInventory().setHelmet(null);
player.getInventory().setChestplate(null);
player.getInventory().setLeggings(null);
player.getInventory().setBoots(null);
}
if(player.hasPermission("lk.admin.clearkit.others")){
for (Player target : plugin.server.getOnlinePlayers())
{
if (target.getName().equalsIgnoreCase(args[0]))
{
target.sendMessage(ChatColor.GOLD + "[" + ChatColor.AQUA + "LegacyKits" + ChatColor.GOLD + "] " + ChatColor.GRAY + "You removed your kit, you may now use another one");
player.sendMessage(ChatColor.GOLD + "[" + ChatColor.AQUA + "LegacyKits" + ChatColor.GOLD + "] " + ChatColor.GREEN + "You removed " + target.getName() + "'s kit.");
kLegacyKits.HasKit.remove(target);
kLegacyKits.JumperKit.remove(target);
kLegacyKits.SpartanKit.remove(target);
kLegacyKits.VampireKit.remove(target);
kLegacyKits.WizardKit.remove(target);
kLegacyKits.HyperKit.remove(target);
kLegacyKits.CrusaderKit.remove(target);
for (PotionEffect effect : target.getActivePotionEffects())
target.removePotionEffect(effect.getType());
target.getInventory().clear();
target.getInventory().setHelmet(null);
target.getInventory().setChestplate(null);
target.getInventory().setLeggings(null);
target.getInventory().setBoots(null);
}
}
}
return false;
}
}

 

Let's start by taking a look at the console shall we...

 

Ivan
http://gyazo.com/1028b680426e58e86e4f33960c635df4

 

You're assuming there is an argument at line 43, but there isn't. You're asking for the first argument, but you didn't specify any when you did the command. That's why Java throws the error. Check if there is an argument by using args.length(). That will return the amount of arguments.

 

Ivan

I added
if(args.length == 0){

}
above the first part, still doesn't work though

Code:

    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
        Player player = (Player) sender;
        if(commandLabel.equalsIgnoreCase("clearkit") || commandLabel.equalsIgnoreCase("ck")){
            if(args.length == 0){             
                player.sendMessage(ChatColor.GOLD + "[" + ChatColor.AQUA + "LegacyKits" + ChatColor.GOLD + "] " + ChatColor.GRAY + "You removed your kit, you may now use another one");
                kLegacyKits.HasKit.remove(player);
                kLegacyKits.JumperKit.remove(player);
                kLegacyKits.SpartanKit.remove(player);
                kLegacyKits.VampireKit.remove(player);
                kLegacyKits.WizardKit.remove(player);
                kLegacyKits.HyperKit.remove(player);
                kLegacyKits.CrusaderKit.remove(player);
                for (PotionEffect effect : player.getActivePotionEffects())
                    player.removePotionEffect(effect.getType());
                player.getInventory().clear();
                player.getInventory().setHelmet(null);
                player.getInventory().setChestplate(null);
                player.getInventory().setLeggings(null);
                player.getInventory().setBoots(null);
            }
        }
        if(player.hasPermission("lk.admin.clearkit.others")){
            for (Player target : plugin.server.getOnlinePlayers())
            {
                if (target.getName().equalsIgnoreCase(args[0]))
                {
                    target.sendMessage(ChatColor.GOLD + "[" + ChatColor.AQUA + "LegacyKits" + ChatColor.GOLD + "] " + ChatColor.GRAY + "You removed your kit, you may now use another one");
                    player.sendMessage(ChatColor.GOLD + "[" + ChatColor.AQUA + "LegacyKits" + ChatColor.GOLD + "] " + ChatColor.GREEN + "You removed " + target.getName() + "'s kit.");
                    kLegacyKits.HasKit.remove(target);
                    kLegacyKits.JumperKit.remove(target);
                    kLegacyKits.SpartanKit.remove(target);
                    kLegacyKits.VampireKit.remove(target);
                    kLegacyKits.WizardKit.remove(target);
                    kLegacyKits.HyperKit.remove(target);
                    kLegacyKits.CrusaderKit.remove(target);
                    for (PotionEffect effect : target.getActivePotionEffects())
                        target.removePotionEffect(effect.getType());
                    target.getInventory().clear();
                    target.getInventory().setHelmet(null);
                    target.getInventory().setChestplate(null);
                    target.getInventory().setLeggings(null);
                    target.getInventory().setBoots(null);
                }
            }
        }
        return false;
    }

 

Regablith
Your plugin only has one command, right? If so, then you do not need to check the label.
Move this block

Code:

if(player.hasPermission("lk.admin.clearkit.others")){

into your args check.