Solved Guard Confiscator

Hey i made this plugin so guards cant take stuff away from prisoners. But when they use blaze rod they can take anything. And i know i have to use a array list for that but how would i set that up. And also they dont need to use blaze rod to take stuff from prisoners. They can use there hands and i dont want that. Can anyone help?

Main Class:

Code:java

package me.cruz2000;
 
import org.bukkit.plugin.java.JavaPlugin;
 
public class ConfiscatorMain
extends JavaPlugin
{
RightClickListener rightclick;
static ConfiscatorMain plugin;
 
public void onEnable()
{
plugin = this;
this.rightclick = new RightClickListener();
}
}
 

Listener Class:

Code:java

package me.cruz2000;
 
import org.bukkit.Bukkit;
import org.bukkit.entity.Entity;
import org.bukkit.entity.EntityType;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerInteractEntityEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.PlayerInventory;
import org.bukkit.plugin.PluginManager;
 
public class RightClickListener
implements Listener
{
public RightClickListener()
{
Bukkit.getPluginManager().registerEvents(this, ConfiscatorMain.plugin);
}
 
@EventHandler
public void onRightClick(PlayerInteractEntityEvent e)
{
if (((e.getPlayer().hasPermission("prisonconfiscator.confiscate") & e.getRightClicked().getType() == EntityType.PLAYER)) && (!((Player)e.getRightClicked()).hasPermission("prisonconfiscator.noconfiscate")))
{
Player c = e.getPlayer();
Player p = (Player)e.getRightClicked();
ItemStack item = p.getItemInHand();
p.setItemInHand(null);
p.updateInventory();
c.getInventory().addItem(new ItemStack[] { item });
}
}
}
 

 

SeniorCluckers
register your events in the Main class, not the listener class.

 

I did but not really answering my question..

 

drpk both work fine
SeniorCluckers just check the Material of the item in hand. Also, don't use static instances of your main class, use the constructor to pass around an instance of the main class.

 

Can you help me i haven't really read up on java im on the api docs but i have no idea what to do. I know i have to check the item the guard is holding to use that item to take away what the prisoner is holding.

 

SeniorCluckers its the same as getting the item the prisoner is holding (but you call the method on the guard)

 

Code:java

 Player c = e.getPlayer();
Player p = (Player)e.getRightClicked();
ItemStack item = p.getItemInHand();
p.setItemInHand(null);
p.updateInventory();
c.getInventory().addItem(new ItemStack[] { item });
c.updateInventory();

Umm still confused. Is it " e.getItemInHand(Material.BLAZE_ROD); " ??

 

SeniorCluckers no you get the ItemStack (and that method takes no parameters), then get the type of that ItemStack, and check if it == Material.BLAZE_ROD

 

Im confused. Im sorry.

 

SeniorCluckers getItemInHand().getType()

 

I cant get it to work. Am i over thinking it?

 

SeniorCluckers most likely

 

In your point of view rate in 1-10 how easy is this.

And im not sure where where that line goes.

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

 

SeniorCluckers just after you define Player c, check if getItemInHand().getType() == Material.BLAZE_ROD

 

Code:java

      Player c = e.getPlayer().getItemInHand().getType() == Material.BLAZE_ROD));

Like this??

fireblast709
Found out what i was doing wrong thanks so much!

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