I am not sure what is wrong, but when a player does /bank it says you You need 15 cookies to buy this! To earn cookies, go get some kills! When I kill a mooshroom cow, it doesn't put 2 cookies in my bank. Whats wrong with this? Code:java public boolean onCommand(CommandSender sender, Command command, String commandLabel, String[] args) { Player player = (Player) sender; if(commandLabel.equalsIgnoreCase("help")){ player.sendMessage(ChatColor.GRAY + "-----" + ChatColor.GOLD + "GGKits Help" + ChatColor.GRAY + "-----"); player.sendMessage(ChatColor.GRAY + "/sharp " + ChatColor.GREEN + "Sharpens your sword for 20 cookies." ); player.sendMessage(ChatColor.GRAY + "/spawn " + ChatColor.GREEN + "Teleports you to spawn." ); player.sendMessage(ChatColor.GRAY + "/shop " + ChatColor.GREEN + "Shop help." ); player.sendMessage(ChatColor.GRAY + "/bank " + ChatColor.GREEN + "Your bank." ); } if(commandLabel.equalsIgnoreCase("bank")){ player.openInventory(player.getEnderChest()); } if(commandLabel.equalsIgnoreCase("Shop")) { if(args.length == 0){ player.sendMessage(ChatColor.GRAY + "-----" + ChatColor.GOLD + "Shop Help" + ChatColor.GRAY + "-----"); player.sendMessage(ChatColor.GRAY + "/sharp" + ChatColor.GREEN + "Sharpens your sword, costs 15 cookies. "); player.sendMessage(ChatColor.GRAY + "/diamond" + ChatColor.GREEN + "Gives diamond armor, costs 50 cookies."); } } if(commandLabel.equalsIgnoreCase("Sharp")) { Map<Enchantment, Integer> Enchants = player.getInventory().getItemInHand().getEnchantments(); Integer level = Enchants.get(Enchantment.DAMAGE_ALL); if(player.getItemInHand().getType() == Material.DIAMOND_SWORD){ if(player.getEnderChest().contains(Material.COOKIE, 15)); if(player.getItemInHand().containsEnchantment(Enchantment.DAMAGE_ALL)){ if(level == 1){ player.getItemInHand().addEnchantment(Enchantment.DAMAGE_ALL, 2); }else if(level == 2){ player.getItemInHand().addEnchantment(Enchantment.DAMAGE_ALL, 3); }else if (level == 0){ player.getItemInHand().addEnchantment(Enchantment.DAMAGE_ALL, 1); }else{ player.sendMessage(ChatColor.GOLD + "This Sword Has Maximized Its Enchantments!"); } } }else{ player.sendMessage(ChatColor.GOLD + "Please Hold A Diamond Sword"); } }else{ player.sendMessage(ChatColor.GOLD + "You need 15 cookies to buy this! To earn cookies, go get some kills!"); }
Code:java @EventHandler public void onDeath(PlayerDeathEvent event) { Player killer = event.getEntity().getKiller(); Player dead = event.getEntity(); if(dead instanceof MushroomCow){ killer.getEnderChest().addItem(new ItemStack(Material.COOKIE, 2)); event.getDrops().clear(); event.getDrops().add(new ItemStack(Material.MUSHROOM_SOUP, 1)); event.getDrops().add(new ItemStack(Material.MUSHROOM_SOUP, 1)); event.getDrops().add(new ItemStack(Material.MUSHROOM_SOUP, 1)); event.getDrops().add(new ItemStack(Material.MUSHROOM_SOUP, 1)); } }
lcpvp event.getEntity() will return the player who died, event.getEntity().getKiller() will also return the player who got killed afaik, but it may be null. So you flipped your variable names.
Thats what I have right now though? Can you reply to my pm? EDIT by Moderator: merged posts, please use the edit button instead of double posting.