PlayerInteractEntityEvent

I am creating a CopStick type plugin and i have it pretty much done but im having issues.

Main Class (TehDrugs.java)

Code:java

package net.imphantom.drugs;
 
import java.util.logging.Logger;
import net.imphantom.drugs.command.CommandStick;
import net.imphantom.drugs.event.PlayerListener;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class TehDrugs extends JavaPlugin{
 
    Logger log = Logger.getLogger("Minecraft");
 
    @Override
    public void onEnable() {
        log.info("TehDrugs Running.");
        PluginManager pm = getServer().getPluginManager();
        pm.registerEvents(new PlayerListener(), this);
        getCommand("copstick").setExecutor(new CommandStick());
        PlayerListener.items = getConfig().getIntegerList("items");
        saveDefaultConfig();
        getConfig().options().copyDefaults(true);
        saveConfig();
    }
 
    @Override
    public void onDisable() {
 
    }
 
}
 

PlayerListener.java:

Code:java

package net.imphantom.drugs.event;
 
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import org.bukkit.ChatColor;
import org.bukkit.Material;
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.PlayerInteractEntityEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.PlayerInventory;
 
public class PlayerListener implements Listener{
 
    public static List<Integer> items = new ArrayList();
 
    @EventHandler
    public void onPlayerInteractEntity(PlayerInteractEntityEvent event) {
        if((event.getRightClicked() instanceof Player)) {
            Player player = event.getPlayer();
            Player criminal = (Player)event.getRightClicked();
 
            if ((player.getItemInHand().getItemMeta() != null) && (player.getItemInHand().getItemMeta().getDisplayName() != null)) {
                if((player.getItemInHand().getItemMeta().getDisplayName().equals("§b§lJail Stick")) && (player.hasPermission("tehdrugs.jailstick"))) {
                    if (InventoryC(criminal, this.items).booleanValue()) {
                        player.performCommand("/jail " + criminal.getName() + " prison 5m");
                    } else
                    {
                        player.sendMessage(ChatColor.RED + "This player does not have drugs.");
                    }
                }
            }
        }
    }
 
    public Boolean InventoryC(Player p, List<Integer> l) {
        PlayerInventory inv = p.getInventory();
        Iterator localIterator = l.iterator();
        if(localIterator.hasNext()) {
            Integer i = (Integer)localIterator.next();
            if(inv.contains(i.intValue())) {
                return Boolean.valueOf(true);
            }
            return Boolean.valueOf(false);
        }
        return null;
    }
}
 

But when i right click on a player with the cop stick when that player has drugs it still sends "This player does not have drugs!"

What did i do wrong? This is my config:

http://pastebin.com/6cXevp2D

 

ImPhantom
Use boolean with a lowercase b; and return true; if the inventory contains the value, and return false everywhere else. Don't return null.


In your event if check, you can remove the booleanValue() because it is arbitrary.

Code:java

@EventHandler
    public void onPlayerInteractEntity(PlayerInteractEntityEvent event) {
        if((event.getRightClicked() instanceof Player)) {
            Player player = event.getPlayer();
            Player criminal = (Player)event.getRightClicked();
 
            if ((player.getItemInHand().getItemMeta() != null) && (player.getItemInHand().getItemMeta().getDisplayName() != null)) {
                if((player.getItemInHand().getItemMeta().getDisplayName().equals("§b§lJail Stick")) && (player.hasPermission("tehdrugs.jailstick"))) {
                    if (InventoryC(criminal, this.items)) { //if true, execute the command.
                        player.performCommand("/jail " + criminal.getName() + " prison 5m");
                    } else
                    {
                        player.sendMessage(ChatColor.RED + "This player does not have drugs.");
                    }
                }
            }
        }
    }
 
    public boolean InventoryC(Player p, List<Integer> l) {
        PlayerInventory inv = p.getInventory();
        Iterator localIterator = l.iterator();
        if(localIterator.hasNext()) {
            int i = (int) localIterator.next();
            if(inv.contains(i.intValue())) {
                return true; // return true;
            } // fall through if not in if-statement
        } //if it falls through or doesn't pass first if check, return false.
        return false;
    }

 

AoH_Ruthless


There is an error in one place... The "i.intValue()" is returning "Cannot invoke intValue() on the primitive type int"

 

ImPhantom
Well remove the intValue() method then... why get an intValue() of an already integer? Note: The advice I gave you in my previous post might not help the issue. If it doesn't, debug your code.