Cancel Event

Sort of new to coding. If you look, you'll realise my plugins are pretty simple.

Here I need some help for the GUI to have an inventory but no one can take out anything.
Someone help?
It's a kit plugin where you preview what's in it.
I have two classes: One called KitPreview and one called KitPreviews xD

Here is KitPreview:

Code:java

package me.dillz.preview;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.block.Sign;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.block.SignChangeEvent;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
 
public class KitPreview
implements Listener
{
@EventHandler(priority=EventPriority.MONITOR)
public void onCreateSign(SignChangeEvent e)
{
if ((e.isCancelled()) && (!e.getPlayer().isOp())) {
return;
}
String[] lines = e.getLines();
if (lines[0] == null) {
return;
}
if (!lines[0].equalsIgnoreCase("[PreviewKit]")) {
return;
}
if (!e.getPlayer().isOp())
{
e.getPlayer().sendMessage(ChatColor.BOLD + "You do not have permission to do this");
e.setCancelled(true);
return;
}
e.setCancelled(false);
e.setLine(0, ChatColor.BOLD + "[PreviewKit]");
e.setLine(1, e.getLine(1));
e.setLine(2, ChatColor.BLUE + "Right Click");
e.setLine(3, ChatColor.BLUE + "To See Kit!");
}
 
@EventHandler
public void onClickSign(PlayerInteractEvent e)
{
if ((e.getAction() != Action.RIGHT_CLICK_BLOCK) && (e.getAction() != Action.RIGHT_CLICK_BLOCK)) {
return;
}
if ((e.getClickedBlock().getState() instanceof Sign))
{
Sign s = (Sign)e.getClickedBlock().getState();
if (s.getLine(0).contentEquals(ChatColor.BOLD + "[PreviewKit]"))
{
String kit = s.getLine(1);
Player p = e.getPlayer();
Inventory vInv = Bukkit.createInventory(null, 36, ChatColor.BLUE + "Preview Kit: " + ChatColor.DARK_AQUA + kit);
ItemStack[] i = p.getInventory().getContents();
p.getInventory().clear();
Bukkit.dispatchCommand(Bukkit.getConsoleSender(), "kit " + kit + " " + e.getPlayer().getName());
try
{
Thread.sleep(50L);
}
catch (InterruptedException ex) {}
ItemStack[] ii = p.getInventory().getContents();
p.getInventory().clear();
if (i != null) {
for (int iii = 0; iii < i.length; iii++) {
if (i[iii] != null) {
p.getInventory().setItem(iii, i[iii]);
}
}
}
if (ii != null) {
for (int iii = 0; iii < ii.length; iii++) {
if (ii[iii] != null) {
vInv.addItem(new ItemStack[] { ii[iii] });
}
}
}
p.openInventory(vInv);
e.setCancelled(true);
}
}
}
 
@SuppressWarnings("deprecation")
@EventHandler
public void onClickItem(InventoryClickEvent e)
{
if (e.getInventory().getTitle().startsWith(ChatColor.DARK_PURPLE + "PreviewKit:"))
{
e.setCursor(null);
e.getWhoClicked().closeInventory();
e.getWhoClicked().openInventory(e.getInventory());
e.setCancelled(true);
}
}
 
public void previewKit(String kitName, Player player)
{
String kit = kitName;
Player p = player;
Inventory vInv = Bukkit.createInventory(null, 36, ChatColor.DARK_PURPLE + "Preview Kit: " + ChatColor.BOLD + kit);
ItemStack[] i = p.getInventory().getContents();
p.getInventory().clear();
Bukkit.dispatchCommand(Bukkit.getConsoleSender(), "kit " + kit + " " + p.getName());
try
{
Thread.sleep(50L);
}
catch (InterruptedException ex) {}
ItemStack[] ii = p.getInventory().getContents();
p.getInventory().clear();
if (i != null) {
for (int iii = 0; iii < i.length; iii++) {
if (i[iii] != null) {
p.getInventory().setItem(iii, i[iii]);
}
}
}
if (ii != null) {
for (int iii = 0; iii < ii.length; iii++) {
if (ii[iii] != null) {
vInv.addItem(new ItemStack[] { ii[iii] });
}
}
}
p.openInventory(vInv);
}
}
 

Here is KitPreviews:

Code:java

package me.dillz.preview;
 
import java.util.Random;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.HandlerList;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerCommandPreprocessEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class KitPreviews
extends JavaPlugin
implements Listener
{
KitPreview tp;
public static final String TAG = ChatColor.BOLD + "[PreviewKit]" + ChatColor.GOLD;
@SuppressWarnings("unused")
private static final Random RANDOM_GENERATOR = new Random();
 
public void onEnable()
{
this.tp = new KitPreview();
Bukkit.getPluginManager().registerEvents(this.tp, this);
Bukkit.getPluginManager().registerEvents(this, this);
}
 
public void onDisable()
{
HandlerList.unregisterAll(this.tp);
}
 
public boolean onCommand(CommandSender s, Command c, String a, String[] args)
{
if (c.getName().equalsIgnoreCase("PreviewKit"))
{
if (args.length < 1)
{
s.sendMessage(TAG + " Usage: /previewkit (kit)");
return true;
}
String kit = args[0];
if (!(s instanceof Player))
{
s.sendMessage(TAG + " Console cannot do this");
return true;
}
Player p = (Player)s;
this.tp.previewKit(kit, p);
}
return true;
}
 
@EventHandler
public void onCommandPreprocess(PlayerCommandPreprocessEvent e)
{
if (e.getMessage().toLowerCase().startsWith("/previewkit"))
{
e.getPlayer().sendMessage(TAG + " Dillz's PreviewKit Plugin!");
e.setCancelled(true);
}
}
 
@EventHandler(priority=EventPriority.HIGHEST)
public void onPlayerJoin(PlayerJoinEvent e)
{
e.getPlayer().sendMessage(TAG + " This server uses PreviewKit!");
}
}
 

Any help appreciated. Just edit the code instead of telling me if you can

 

Just use event.setCancelled(true); to PlayerInteractEvent in the inventory.
Not going to spoon feed :l

 

It didn't work :c
Btw did you mean e.setCancelled(true);

 

17xDillz1997

He meant Cancellable#setCancelled(true), but that's besides the point. I think you should really reconsider what you're doing, you sleep the thread (something which, in Bukkit programming, you should almost never do), you modify/cancel an event when you set the prioirty to Monitor, which violates the contract of EventHandlers, you opened an inventory in a handler for an inventory event, which is specifically recommended against in the JavaDocs, and you use horrible names like "i" and "ii" for variables that aren't throwaways. Learn Java, then try again.

 

Awesome, on my team of using e, not event ;D

 

Event. or e.set...
Great going for the one who helped
and now pls switch for 'Solved'

ChipDev, You will see "Prefix" on the title, Click on it and look for Solved.
Cause people will come for nothing here, And also when the people see that this is solved they will come over here and take some info and help from people

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