Spawn mob with command

I'm having trouble with this :/ It shouldn't be this difficult.. Everything works, I have tried using debug messages and everything. It's just the part that spawns the zombie. When I use '/zombie create test', test being the display name, it just says "/zombie" in white in the chat.

Command portion of CommandExecutor Class:

Code:java

 public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
if(sender instanceof Player) {
Player p = (Player) sender;
if(p.isOp()) {
if(cmd.getName().equalsIgnoreCase("zombie")) {
if(args.length == 0) {
p.sendMessage(ChatColor.GOLD + "Zombie GUI Commands:");
p.sendMessage(ChatColor.YELLOW + "/zombie create <name> " + ChatColor.GRAY + "Create a zombie with the given name.");
p.sendMessage(ChatColor.YELLOW + "/zombie remove <name> " + ChatColor.GRAY + "Remove a zombie with the given name.");
p.sendMessage(ChatColor.YELLOW + "/zombie removeall " + ChatColor.GRAY + "Removed all zombies in the current world.");
return true;
}
else if(args.length == 1) {
if(args[0].equalsIgnoreCase("create")) {
p.sendMessage(ChatColor.RED + "Error: " + ChatColor.GOLD + "Usage: /zombie [subcmd] [arg]");
return true;
}
else if(args.length == 2) {
if(args[0].equalsIgnoreCase("create")) {
String zname = args[1];
LivingEntity e = (LivingEntity)p.getWorld().spawnEntity(p.getLocation(), EntityType.ZOMBIE);
 
e.setCanPickupItems(false);
e.setCustomName(ChatColor.translateAlternateColorCodes('&', zname));
e.setCustomNameVisible(true);
e.setRemoveWhenFarAway(false);
 
e.addPotionEffect(new PotionEffect(PotionEffectType.SLOW, 99999, 6));
e.addPotionEffect(new PotionEffect(PotionEffectType.JUMP, 99999, 6));
 
ItemStack bhat = new ItemStack(Material.LEATHER_HELMET);
LeatherArmorMeta bmeta = (LeatherArmorMeta)bhat.getItemMeta();
bmeta.setColor(Color.BLUE);
bhat.setItemMeta(bmeta);
ItemStack bchest = new ItemStack(Material.LEATHER_CHESTPLATE);
LeatherArmorMeta bmeta2 = (LeatherArmorMeta)bchest.getItemMeta();
bmeta2.setColor(Color.BLUE);
bchest.setItemMeta(bmeta2);
ItemStack blegs = new ItemStack(Material.LEATHER_CHESTPLATE);
LeatherArmorMeta bmeta3 = (LeatherArmorMeta)blegs.getItemMeta();
bmeta3.setColor(Color.BLUE);
blegs.setItemMeta(bmeta3);
ItemStack bboots = new ItemStack(Material.LEATHER_CHESTPLATE);
LeatherArmorMeta bmeta4 = (LeatherArmorMeta)bboots.getItemMeta();
bmeta4.setColor(Color.BLUE);
bboots.setItemMeta(bmeta4);
 
e.getEquipment().setHelmet(new ItemStack(bhat));
e.getEquipment().setChestplate(new ItemStack(bchest));
e.getEquipment().setLeggings(new ItemStack(blegs));
e.getEquipment().setBoots(new ItemStack(bboots));
 
String name = e.getCustomName();
p.sendMessage(ChatColor.GREEN + "Zombie created at your location with display name: " + ChatColor.YELLOW + name);
return true;
}
}
}
}
}
else {
p.sendMessage(ChatColor.RED + "You must be OP to use this command!");
return true;
}
}
return false;
}

onEnable() :

Code:java

    @Override
    public void onEnable() {
        log("Enabled.");
        getCommand("zombie").setExecutor(new CmdCreate());

Any help will be great. Thanks in advance.

 

Are you executing the command from the console?

 

CraftedShack Nope.. In game.

CraftedShack '/zombie create <name>' is suppose to create a zombie at my location with the name "<name>.
It just returns '/zombie' in chat, like in this screenshot.

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

 

You need to register your command in your main class. You may have it in your plugin.yml but you need to add this to your onEnable:

Code:java

 
PluginManager manager = getServer().getPluginManager();
manager.registerEvents(new ListenerClass(this), this);
 

 

ThunderWaffeMC It's not an event though, it's in the command executor class.. I'm spawning the zombie with a command, /zombie create <name>. Giving it a custom display name.

 

Sorry. I didn't mean to register an event. Register the command*

Code:java

 
getCommand("mycommand").setExecutor(new CommandClass(this));
 

 

ThunderWaffeMC Yes, I did that :/ Can you add me on skype? My skype is tylerthecreeper1

 

Bingo. You messed up the indentation. I've got it fixed for you here:

Code:java

publicboolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
if (sender instanceof Player) {
Player p = (Player) sender;
if (p.isOp()) {
if (cmd.getName().equalsIgnoreCase("zombie")) {
if (args.length == 0) {
p.sendMessage(ChatColor.GOLD + "Zombie GUI Commands:");
p.sendMessage(ChatColor.YELLOW + "/zombie create <name> " + ChatColor.GRAY + "Create a zombie with the given name.");
p.sendMessage(ChatColor.YELLOW + "/zombie remove <name> " + ChatColor.GRAY + "Remove a zombie with the given name.");
p.sendMessage(ChatColor.YELLOW + "/zombie removeall " + ChatColor.GRAY + "Removed all zombies in the current world.");
returntrue;
} else if (args.length == 1) {
if (args[0].equalsIgnoreCase("create")) {
p.sendMessage(ChatColor.RED + "Error: " + ChatColor.GOLD + "Usage: /zombie [subcmd] [arg]");
returntrue;
}
} else if (args.length == 2) {
if (args[0].equalsIgnoreCase("create")) {
String zname = args[1];
LivingEntity e = (LivingEntity) p.getWorld().spawnEntity(p.getLocation(), EntityType.ZOMBIE);
 
e.setCanPickupItems(false);
e.setCustomName(ChatColor.translateAlternateColorCodes('&', zname));
e.setCustomNameVisible(true);
e.setRemoveWhenFarAway(false);
 
e.addPotionEffect(new PotionEffect(PotionEffectType.SLOW, 99999, 6));
e.addPotionEffect(new PotionEffect(PotionEffectType.JUMP, 99999, 6));
 
ItemStack bhat = new ItemStack(Material.LEATHER_HELMET);
LeatherArmorMeta bmeta = (LeatherArmorMeta) bhat.getItemMeta();
bmeta.setColor(Color.BLUE);
bhat.setItemMeta(bmeta);
ItemStack bchest = new ItemStack(Material.LEATHER_CHESTPLATE);
LeatherArmorMeta bmeta2 = (LeatherArmorMeta) bchest.getItemMeta();
bmeta2.setColor(Color.BLUE);
bchest.setItemMeta(bmeta2);
ItemStack blegs = new ItemStack(Material.LEATHER_CHESTPLATE);
LeatherArmorMeta bmeta3 = (LeatherArmorMeta) blegs.getItemMeta();
bmeta3.setColor(Color.BLUE);
blegs.setItemMeta(bmeta3);
ItemStack bboots = new ItemStack(Material.LEATHER_CHESTPLATE);
LeatherArmorMeta bmeta4 = (LeatherArmorMeta) bboots.getItemMeta();
bmeta4.setColor(Color.BLUE);
bboots.setItemMeta(bmeta4);
 
e.getEquipment().setHelmet(new ItemStack(bhat));
e.getEquipment().setChestplate(new ItemStack(bchest));
e.getEquipment().setLeggings(new ItemStack(blegs));
e.getEquipment().setBoots(new ItemStack(bboots));
 
String name = e.getCustomName();
p.sendMessage(ChatColor.GREEN + "Zombie created at your location with display name: " + ChatColor.YELLOW + name);
returntrue;
}
}
}
} else {
p.sendMessage(ChatColor.RED + "You must be OP to use this command!");
returntrue;
}
}
returnfalse;
}

(There were too many indentation mistakes to point them all out)

 

DJSkepter Thanks a lot, I'll try it and let you know

DJSkepter Works, thanks !

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

 

No problem - indentation isn't a simple topic. In the end, your previous indentation forgot about the isOp part and went straight to return false, where it sends the default command message. Happy coding