Solved Help with item interaction

So, I have a plugin made and what it does is when you right click with a book in your hand it shows you your /money or /balance.

Code:

Code:

package me.Bill4788.Wallet;
 
import java.util.logging.Logger;
 
import net.milkbowl.vault.chat.Chat;
import net.milkbowl.vault.economy.Economy;
import net.milkbowl.vault.permission.Permission;
 
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.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.PlayerInventory;
import org.bukkit.plugin.RegisteredServiceProvider;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener {
public void onEnable() {
if (!setupEconomy() ) {
            log.severe(String.format("[%s] - Disabled due to no Vault dependency found!", getDescription().getName()));
            getServer().getPluginManager().disablePlugin(this);
            return;
        }
        setupPermissions();
        setupChat();
 
Bukkit.getServer().getPluginManager().registerEvents(this, this);
}
 
private static final Logger log = Logger.getLogger("Minecraft");
public static Economy econ = null;
public static Permission perms = null;
public static Chat chat = null;
 
    private boolean setupEconomy() {
        if (getServer().getPluginManager().getPlugin("Vault") == null) {
            return false;
        }
        RegisteredServiceProvider<Economy> rsp = getServer().getServicesManager().getRegistration(Economy.class);
        if (rsp == null) {
            return false;
        }
        econ = rsp.getProvider();
        return econ != null;
    }
 
    private boolean setupChat() {
        RegisteredServiceProvider<Chat> rsp = getServer().getServicesManager().getRegistration(Chat.class);
        chat = rsp.getProvider();
        return chat != null;
    }
 
    private boolean setupPermissions() {
        RegisteredServiceProvider<Permission> rsp = getServer().getServicesManager().getRegistration(Permission.class);
        perms = rsp.getProvider();
        return perms != null;
    }
 
 
 
 
 
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
if (cmd.getName().equalsIgnoreCase("wallet")) { // If the player typed /basic then do the following...
Player p = (Player) sender;
PlayerInventory pi = p.getInventory();
 
// Inv Stufferino
ItemStack wallet = new ItemStack(Material.BOOK, 1);
pi.addItem(wallet);
 
return true;
}
 
return false; 
}
 
@EventHandler
public void onPlayerInteract(PlayerInteractEvent e) {
Player p = e.getPlayer();
 
if (e.getAction() == Action.RIGHT_CLICK_AIR) {
if (e.getPlayer().getItemInHand().getType() == Material.BOOK) {
p.sendMessage(ChatColor.GOLD + "[" + ChatColor.GREEN + "Wallet" + ChatColor.GOLD + "] You have: " + econ.getBalance(e.getPlayer()));
}
}
 
}
}

Stack: http://pastebin.com/2kqJfVDc

 

It's seems like the exception occurs in your setupChat(). The RegisteredServiveProvider rsp is null, which is why you can't do rsp.getProvider(); Check if (rsp == null) first.

 

Here are some changes I've made and now your code works:

From:

Code:java

public static Economy econ = null;
public static Permission perms = null;
public static Chat chat = null;

To:

Code:java

public static Economy econ;
public static Permission perms;
public static Chat chat;

From:

Code:java

private boolean setupChat() {
        RegisteredServiceProvider<Chat> rsp = getServer().getServicesManager().getRegistration(Chat.class);
        chat = rsp.getProvider();
        return chat != null;
    }

To:

Code:java

private boolean setupChat() {
        RegisteredServiceProvider<Chat> chatProvider = getServer().getServicesManager().getRegistration(net.milkbowl.vault.chat.Chat.class);
        if (chatProvider != null) {
            chat = chatProvider.getProvider();
        }
 
        return (chat != null);
    }

From:

Code:java

@EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
        Player p = e.getPlayer();
 
        if (e.getAction() == Action.RIGHT_CLICK_AIR) {
            if (e.getPlayer().getItemInHand().getType() == Material.BOOK) {
                p.sendMessage(ChatColor.GOLD + "[" + ChatColor.GREEN + "Wallet" + ChatColor.GOLD + "] You have: " + econ.getBalance(e.getPlayer()));
            }
        }
 
    }

To:

Code:java

@EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
 
        Player p = e.getPlayer();
        ItemStack i = p.getItemInHand();
        ItemStack book = new ItemStack(Material.BOOK);
        if (e.getAction() == Action.RIGHT_CLICK_AIR) {
            if (i.equals(book)) {
                p.sendMessage(ChatColor.GOLD + "[" + ChatColor.GREEN + "Wallet" + ChatColor.GOLD + "] You have: " + econ.getBalance(p));
            }
        }
 
    }

Hope this helps.

 

EgyptianKing
You're probably going to get hate for spoonfeeding

 

hankered

Yea you might be right, but I did not fix the issue with any knowledge on the error, rather I tried and experimented until it worked. Since I cannot give him a nice explanation which would be useful for him, I pasted the code. But I hope the code I provided him would help him figure out what he did wrong.

 

EgyptianKing

Thanks man!