Shop

I have a weird bug for my shop plugin, when you buy something it wont take away your coins, and it won't give you items, and I've tried to change the slot that the TNT class is in, it wont change. Please HELP! this is my main class

Code:java

package me.JoeyLangston.RevisedKits;
 
import java.util.ArrayList;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.java.JavaPlugin;
 
public class KitsMain extends JavaPlugin implements Listener {
 
public static FileConfiguration config;
 
public static KitsMain plugin = null;
 
public void onEnable() {
getServer().getPluginManager().registerEvents(new EventHandlers(), this);
 
config = getConfig();
 
plugin = this;
}
 
public static void saveFile(){
plugin.saveConfig();
}
 
public ItemStack createItem(Material material, int amount, short shrt,
String displayname, String lore) {
ItemStack item = new ItemStack(material, amount, (short) shrt);
ItemMeta meta = item.getItemMeta();
meta.setDisplayName(displayname);
ArrayList<String> Lore = new ArrayList<String>();
Lore.add(lore);
meta.setLore(Lore);
 
item.setItemMeta(meta);
return item;
}
 
static Inventory shop;
{
shop = Bukkit.createInventory(null, 9, ChatColor.RED.toString() + ChatColor.BOLD + "SelectibleKits");
 
shop.setItem(1, createItem(Material.IRON_SWORD, 1, (short) 0, "§3Warrior","§fPrice §6200 §fSilver"));
shop.setItem(3, createItem(Material.BOW, 1, (short) 0, "§4Archer","§fPrice §2200 §fSilver"));
shop.setItem(6, createItem(Material.TNT, 1, (short) 0, "§5DemoMan","§fPrice §5200 §fSilver"));
 
}
public boolean onCommand(CommandSender sender, Command cmd, String label,String[] a) {
Player player = (Player) sender;
if (cmd.getName().equalsIgnoreCase("mkit") && player.hasPermission("kits.show")) {
player.openInventory(shop);
 
} else {
player.sendMessage(ChatColor.AQUA + "Im sorry, but your current permission group does not allow you to use our plugin(mkit) if for some reason you belive this is an error please contact a server administrator or the " + ChatColor.GREEN + "Plugin Developer :D");
}
return false;
}
}
 

This is my currency class :

Code:java

package me.JoeyLangston.RevisedKits;
 
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
 
public class myAPI implements Listener {
 
    public static void giveSilver(Player p, int i) {
        KitsMain.config.set(p.getName() + ".Silver",
                KitsMain.config.getInt(p.getName() + ".Silver", 0) + i);
        KitsMain.saveFile();
        p.sendMessage("§2§l$" + i + " silver received!");
}
 
public static void takeSilver(Player p, int i) {
    KitsMain.config.set(p.getName() + ".Silver",
                    KitsMain.config.getInt(p.getName() + ".Silver", 0) - i);
    KitsMain.saveFile();
        p.sendMessage("§c§l$" + i + " silver taken!");
}
 
public static boolean hasEnough(Player p, int i) {
        if (KitsMain.config.getInt(p.getName() + ".Silver") >= i)
                return true;
        return false;
}
}
 

This is my event handler class:

Code:java

package me.JoeyLangston.RevisedKits;
 
import org.bukkit.Material;
import org.bukkit.entity.Monster;
import org.bukkit.entity.Player;
import org.bukkit.entity.Villager;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDeathEvent;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.inventory.ItemStack;
 
public class EventHandlers implements Listener {
 
    @EventHandler
    public void onJoin(PlayerJoinEvent e) {
            Player p = e.getPlayer();
 
            if (!KitsMain.config.contains(p.getName())) {
                    KitsMain.config.set(p.getName() + ".Silver", 0);
            }
    }
 
    @EventHandler
    public void onKill(EntityDeathEvent e) {
 
            if (e.getEntity() instanceof Monster) {
                    Monster m = (Monster) e.getEntity();
                    if (m.getKiller() instanceof Player) {
                            Player p = m.getKiller();
 
                            myAPI.giveSilver(p, 200);
                    }
 
 
            if (e.getEntity() instanceof Villager) {
                    Villager v = (Villager) e.getEntity();
                    if (v.getKiller() instanceof Player) {
                            Player p = v.getKiller();
                            myAPI.takeSilver(p, 200);
                    }
            }
        }
    }
    @EventHandler
    public void onClick(InventoryClickEvent event) {
            Player p = (Player) event.getWhoClicked();
 
            if (event.getInventory().getName().equals(KitsMain.shop.getName())) {
                    event.setCancelled(true);
 
                    if (event.getCurrentItem() == null) {
                            return;
                    }
 
                    if (!(event.getCurrentItem().hasItemMeta())) {
                            return;
                    }
 
                    if (event.getCurrentItem().getItemMeta().getDisplayName().equals("§3Warrior")) {
                        event.getWhoClicked().getInventory().addItem(new ItemStack(Material.IRON_SWORD, 1));
                        event.getWhoClicked().getInventory().addItem(new ItemStack(Material.COOKED_BEEF, 64));
                        event.getWhoClicked().getInventory().setChestplate(new ItemStack(Material.CHAINMAIL_CHESTPLATE, 1));
                        event.getWhoClicked().getInventory().setHelmet(new ItemStack(Material.TNT, 1));
                        event.getWhoClicked().getInventory().setLeggings(new ItemStack(Material.CHAINMAIL_LEGGINGS, 1));
                        event.getWhoClicked().getInventory().setBoots(new ItemStack(Material.CHAINMAIL_BOOTS, 1));
                        event.getWhoClicked().closeInventory();
                    if (event.getCurrentItem().getItemMeta().getDisplayName().equals("§3Archer")) {
                        event.getWhoClicked().getInventory().addItem(new ItemStack(Material.BOW, 1));
                        event.getWhoClicked().getInventory().addItem(new ItemStack(Material.ARROW, 64));
                        event.getWhoClicked().getInventory().setChestplate(new ItemStack(Material.CHAINMAIL_CHESTPLATE, 1));
                        event.getWhoClicked().getInventory().setHelmet(new ItemStack(Material.CHAINMAIL_HELMET, 1));
                        event.getWhoClicked().getInventory().setLeggings(new ItemStack(Material.CHAINMAIL_LEGGINGS, 1));
                        event.getWhoClicked().getInventory().setBoots(new ItemStack(Material.CHAINMAIL_CHESTPLATE, 1));
                        event.getWhoClicked().closeInventory();
                    if (event.getCurrentItem().getItemMeta().getDisplayName().equals("§3DemoMan")) {
                        event.getWhoClicked().getInventory().addItem(new ItemStack(Material.STONE_SWORD, 1));
                            event.getWhoClicked().getInventory().addItem(new ItemStack(Material.COOKED_BEEF, 64));
                          event.getWhoClicked().getInventory().setChestplate(new ItemStack(Material.LEATHER_CHESTPLATE, 1));
                            event.getWhoClicked().getInventory().setHelmet(new ItemStack(Material.LEATHER_HELMET, 1));
                            event.getWhoClicked().getInventory().setLeggings(new ItemStack(Material.LEATHER_LEGGINGS, 1));
                            event.getWhoClicked().getInventory().setBoots(new ItemStack(Material.LEATHER_CHESTPLATE, 1));
                            event.getWhoClicked().getInventory().addItem(new ItemStack(Material.REDSTONE_TORCH_ON, 5));
                          event.getWhoClicked().getInventory().addItem(new ItemStack(Material.TNT, 15));
                            event.getWhoClicked().closeInventory();
 
 
                        }
                            if (myAPI.hasEnough(p, 25)) {
                                    myAPI.takeSilver(p, 25);
                            } else {
                                    p.sendMessage("§cYou're too poor, go kill stuff!");
                            }
                    }
            }
        }
    }
}
 

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

 

Try casting the thing who clicked into a player.

Code:java

Player p = (Player) e.getWhoClicked();

 

ok im trying now!

no work

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