Help needed for simple chest event code.

Hi!
So I have been working on this plugin for ages, and this one, silly bug, over an hour. It seems ridiculous that I can make other much more advanced things work, yet this doesn't. I am making a chest protection plugin, and no matter what I code, nothing seems to fire! I originally made this with the newest 1.7.8 craftbukkit, but then I realised that may be causing the issue, so I ported back to both 1.7.5 and 1.7.2.

I have tried the main two ways of seeing if a chest was opened, the player interact event, which, for some strange reason doesn't fire at all, even before it goes through check if you right clicked a chest. Really odd! Nothing comes up with the below code.

Code:java

@EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        event.getPlayer().sendMessage("Interacted");
 
        if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
            Block blk = event.getClickedBlock();
            event.getPlayer().sendMessage("Right clicked a block");
 
            if(blk.getType().equals(Material.CHEST)) {
                event.getPlayer().sendMessage("that was a chest");
 
            }
        }
    }

I even tried the way of doing it with the onInventoryOpenEvent, same thing. The event isn't fired.

Obviosly this makes you think. "Gee, Mixer doesn't know how to setup events " but all my other events work!?

Any help I would love. I am sick of trying the same thing over and over
Thanks Mixer.

 

Mixerman123
since you have other events you probably did, but did you register the event?

 

IkBenHarm
Sorry, I am kind of new to this. I thought you just did @ EventHandler and that function would run. Sorry, still getting my head around event programming Here's the code I have so far. Abit messy, but works as usual so far.

Code:java

package com.mixerman123.eagleeconomylock;
 
import java.io.File;
import java.io.IOException;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.block.Block;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.plugin.java.JavaPlugin;
 
 
 
public final class EagleEconomyLock extends JavaPlugin implements Listener{
 
    FileConfiguration ChestDataFile;
    File dfile;
 
 
 
    @Override
    public void onEnable() {
        // TODO Insert logic to be performed when the plugin is enabled
        getLogger().info("EagleEconomyLock is up and running.");
 
        //Creates a folder for the chest data file
        if (!this.getDataFolder().exists()) {
            getLogger().info(ChatColor.AQUA+"It looks like this is the first time EEL has been run. Setting up some files now...");
            this.getDataFolder().mkdir();
        }
 
        //Then make the file
        dfile = new File(this.getDataFolder(), "chestData.yml");
        if (!dfile.exists()) {
            try {
                    dfile.createNewFile();
            }
            catch (IOException e) {
                    Bukkit.getServer().getLogger().severe(ChatColor.RED + "Could not create the chest save file! Contact Mixerman123"+ChatColor.RED+" ASAP!!");
            }
        }
 
        //Now the file exists in the folder!
 
 
        ChestDataFile = YamlConfiguration.loadConfiguration(dfile);
    }
 
    public FileConfiguration getData() {
        return ChestDataFile;
    }
 
    public void saveData() {
            try {
                    ChestDataFile.save(dfile);
            }
            catch (IOException e) {
                    Bukkit.getServer().getLogger().severe(ChatColor.RED + "Could not save data.yml!");
            }
    }
 
    public void reloadData() {
            ChestDataFile = YamlConfiguration.loadConfiguration(dfile);
    }
 
    @Override
    public void onDisable() {
        // TODO Insert logic to be performed when the plugin is disabled
        getLogger().info("EagleEconomyLock is disabled! If this isn't because of a server shutdown/reload, contact Mixerman123");
    }
 
 
 
 
//    @EventHandler //ALWAYS use this before events
//    public void onInventoryOpenEvent(InventoryOpenEvent e){
//        Bukkit.broadcastMessage("Inv opeded");
//        if (e.getInventory().getHolder() instanceof Chest){
//            Bukkit.broadcastMessage("Inv opeded and was chest");
//            Chest c = (Chest) e.getInventory().getHolder();
//           
//            Chest lockedChest = c;
//            Player p = (Player) e.getPlayer();
//            p.sendMessage("The players who are aloud to open this chest are: "+getData().getInt("ChestDataForChestAt."+lockedChest.getX()+","+lockedChest.getY()+","+lockedChest.getZ()+".GroupPermitted:"));
//        }
//    }
 
 
 
 
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (cmd.getName().equalsIgnoreCase("lock")) { // If the player typed /basic then do the following...
            if (!(sender instanceof Player)) {
                sender.sendMessage("This command locks the chest you're standing on, but you're "+ChatColor.RED+"not a player!");
            } else {
                Player player = (Player) sender;
 
                if(player.getLocation().add(0, 0, 0).getBlock().getType().name()=="CHEST"){
                    //We now know the block is a chest so we can turn that block into a chest for further use in code
                    Block lockedChest = player.getLocation().add(0, 0, 0).getBlock();
                    //A silly message to the locker :P
                    player.sendMessage("Locking this chest with "+ChatColor.BOLD+"[E]agle [E]conomy [L]ock");
 
                    //Create an id for this chest that is based of its location
                    String ThisChestsId = "ChestDataForChestAt."+lockedChest.getX()+","+lockedChest.getY()+","+lockedChest.getZ()+".";
                    ChestDataFile.set(ThisChestsId+"X:", lockedChest.getX());
                    ChestDataFile.set(ThisChestsId+"Y:", lockedChest.getY());
                    ChestDataFile.set(ThisChestsId+"Z:", lockedChest.getZ());
                    ChestDataFile.set(ThisChestsId+"GroupPermitted:", player.getName());//Players allowed
                    saveData();
 
                }else{
                    player.sendMessage("To lock a chest, you need to stand on top of one.");
                }
 
            }
 
            return true;
        }
        return false;
    }
}
 

Indentation got screwed.

 

Mixerman123
use in your onEnable():

Bukkit.getServer().getPluginManager().registerEvents(*class where events are*, this);

 

Oh dang. Super Obvious. Thank you so, so, very much. You are a life saver. Why hadn't I thought of this?! *Clap clap* Now go build a time machine so you could have told me this 2 hours ago Thanks so much man. (I really need to look at the obvious more) THANKS!

 

Mixerman123
haha no problem :3