Crate Plugin Help!

Hi,
I need some help with a crate plugin I am trying to make! This is my code so far

Code:java

package me.lambo.craft.crates;
 
import java.util.logging.Logger;
 
import org.bukkit.Material;
import org.bukkit.block.Chest;
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.plugin.java.JavaPlugin;
 
public class Crates extends JavaPlugin implements Listener {
 
Logger log;
 
public void onEnable() {
log = this.getLogger();
log.info("LamboCraft-Crates v1.0 Has Been Enabled!");
getServer().getPluginManager().registerEvents(this, this);
}
 
public void onDisable() {
log.info("LamboCraft-Crates v1.0 Has Been Disabled!");
}
 
public void blockChestInterract(PlayerInteractEvent event) {
Chest chest = (Chest) event.getClickedBlock().getState();
if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
if (chest.getInventory().getName().equals("Crate") && event.getItem().getType().equals(Material.TRIPWIRE_HOOK));
event.getPlayer().getInventory().addItem(new ItemStack(Material.DIAMOND, 1));
}
}
}
 

I was wondering how I would be able to cancel the event of a chest opening if the name of the chest is "Crate". I understand how to cancel the event of a chest opening but not really sure how to if the name of the chest is "Crate". Second of all, if the name of the chest is "Crate" and the player is holding a tripwire hook with a certain color code and name "&6&l&nLambo-Key" when they right click the chest with the name "Crate" they will recieve a diamond. I am pretty stumped on what to do at the moment. If you could help that would be great!

 

Forgot to add @EventHandler above the event

 

Just forgot about that! Thanks Would you mind helping me with the other things JustinsCool15

 

1337Pluginz
For the trip hook thing you would use

Code:java

if(e.getItem().hasItemMeta() && e.getPlayer().getInventory().getItemInHand().getItemMeta().getDisplayName().equals(ChatColor.BOLD + "" + ChatColor.UNDERLINE + "Labo-Key")){
  //code to give them diamond here.
}

Idk about the Crate name thing.

Also if you don't know how to give a player an item I can help with that too.

 

JustinsCool15 Thanks!

Just stuck on the Chest as it is giving me a NullPointerException at line 30 in the code above. It gives you the exception when you sneak right click, you break ANY block, place ANY block! I am really stumped now

 

1337Pluginz How would you be naming the chest? Do you mean a GUI? If so get the inventory name, ex.

Code:Java

inventory.getName().equalsIgnoreCase("Chest"){ /*Code*/ }

 

So basically, if the name of this chest is "Crate" the chest will not open but you will recieve a diamond and the tripwire hook (the key) will be removed from the inventory if you right click on the chest. Niknea Thanks for your help, just need some further understanding to make this work!

 

1337Pluginz Correct, use my code in an if statement, if it passes give the player the items and cancel the event.

 

Code:java

@EventHandler(priority = EventPriority.HIGHEST)
public void blockChestInterract(PlayerInteractEvent event) {
Chest chest = (Chest) event.getClickedBlock().getState();
if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
if (((Logger) chest).getName().equalsIgnoreCase("Chest") && event.getItem().getType().equals(Material.TRIPWIRE_HOOK)) {
if (event.getItem().hasItemMeta() && event.getPlayer().getInventory().getItemInHand().getItemMeta().getDisplayName().equals(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.UNDERLINE + "Lambo-Key")) {
event.getPlayer().getInventory().addItem(new ItemStack(Material.DIAMOND, 1));
event.setCancelled(true);
}
}
}
}

Only added the Cast since Eclipse told me to.

 

1337Pluginz Niknea's method is for getting the name of the inventory, not the block itself.

 

1337Pluginz
Little tip, in Eclipse if you press Ctrl + A then Ctrl + I it will organize all your code's spacings.

Also are you having any more issues?

 

I really cant get this to work Its giving me NullPointerExceptions here and there! Would anyone mind reworking my code to get it to work please?

 

1337Pluginz
Having one of us just re-write your code to make it work isn't going to help you learn.
Paste the stack trace with the NPE's and we will help you fix them.
If you just want someone to code a plugin for you, you should of have made a request in the Plugin Requests forum.

 

1337Pluginz

Post the updated code and what line is making the Exception (In the updated code.)?

 

Code:java

package me.lambo.craft.crates;
 
import java.util.logging.Logger;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.block.Chest;
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.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Crates extends JavaPlugin implements Listener {
 
Logger log;
 
public void onEnable() {
log = this.getLogger();
log.info("LamboCraft-Crates v1.0 Has Been Enabled!");
getServer().getPluginManager().registerEvents(this, this);
}
 
public void onDisable() {
log.info("LamboCraft-Crates v1.0 Has Been Disabled!");
}
 
@EventHandler(priority = EventPriority.HIGHEST)
public void blockChestInterract(PlayerInteractEvent event) {
Chest chest = (Chest) event.getClickedBlock().getState();
if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
if (((Logger) chest).getName().equalsIgnoreCase("Chest") && event.getItem().getType().equals(Material.TRIPWIRE_HOOK)) {
if (event.getItem().hasItemMeta() && event.getPlayer().getInventory().getItemInHand().getItemMeta().getDisplayName().equals(ChatColor.GOLD + "" + ChatColor.BOLD + "" + ChatColor.UNDERLINE + "Lambo-Key")) {
event.getPlayer().getInventory().addItem(new ItemStack(Material.DIAMOND, 1));
event.setCancelled(true);
}
}
}
}
}
 

Code:

[Server thread/ERROR]: Could not pass event PlayerInteractEvent to LamboCraft-Crates v1.0
org.bukkit.event.EventException
    at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:294) ~[bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62) ~[bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:501) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:486) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at org.bukkit.craftbukkit.v1_7_R3.event.CraftEventFactory.callPlayerInteractEvent(CraftEventFactory.java:216) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.PlayerInteractManager.interact(PlayerInteractManager.java:374) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.PlayerConnection.a(PlayerConnection.java:629) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.PacketPlayInBlockPlace.a(SourceFile:60) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.PacketPlayInBlockPlace.handle(SourceFile:9) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.NetworkManager.a(NetworkManager.java:157) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.ServerConnection.c(SourceFile:134) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.v(MinecraftServer.java:667) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.DedicatedServer.v(DedicatedServer.java:260) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.u(MinecraftServer.java:558) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.run(MinecraftServer.java:469) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    at net.minecraft.server.v1_7_R3.ThreadServerApplication.run(SourceFile:628) [bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
Caused by: java.lang.ClassCastException: org.bukkit.craftbukkit.v1_7_R3.block.CraftChest cannot be cast to java.util.logging.Logger
    at me.lambo.craft.crates.Crates.blockChestInterract(Crates.java:34) ~[?:?]
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_11]
    at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_11]
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_11]
    at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_11]
    at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:292) ~[bukkit.jar:git-Bukkit-1.7.9-R0.1-10-g8688bd4-b3092jnks]
    ... 15 more

I hope we can work together to fix my plugin guys! Thanks for all the contributions and tips so far

Anyone have a solution?

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

 

1337Pluginz
Remove the ((logger) chest) and just use chest on line 34

 

Method name is undefined for the type Chest

Code:java

if (chest.name().equalsIgnoreCase("Crate") && event.getItem().getType().equals(Material.TRIPWIRE_HOOK)) {

 

1337Pluginz
Try using (Chest) instead of ((logger) Chest)

 

Method getName() is undefined for type Chest

BEFORE

Code:java

if (((Logger) chest).getName().equalsIgnoreCase("Chest") && event.getItem().getType().equals(Material.TRIPWIRE_HOOK)) {

AFTER

Code:java

if (chest.getName().equalsIgnoreCase("Chest") && event.getItem().getType().equals(Material.TRIPWIRE_HOOK)) {

Code:java

if (((Chest) chest).name().equalsIgnoreCase("Chest") && event.getItem().getType().equals(Material.TRIPWIRE_HOOK)) {

Like this right? If yes, it still gives the Method name is undefined for the type Chest

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

 

1337Pluginz That's because there is no getName() method for the chest. Once you place a block, you lose all the ItemMeta from the ItemStack.

 

So what can I do then endoml ?

 

1337Pluginz You could make an arraylist of the chests that you want to give diamonds

 

Very confused now
Decided to make a plugin request to see if anyone would make it

http://forums.bukkit.org/threads/crates.293331/

 

That's not what hasItemMeta() does.

 

1337Pluginz
I don't see the reason for making that thread if what you want is your code to be fixed.