Block Break Event not working

hey guys i made a chest lock plugin and it worked great until i realized that you must listen for block break event(see if someone is trying to break it AND see if the user is breaking it therefore removing it from the config) but it wont work when someone tries to delete their own chest. if a stranger tries it gives him the you cant break message, but when an owner breaks it the config wont save. I did config.save() and im getting no console errors. What is happening!

Code:java

@EventHandler
    public void onBreak(BlockBreakEvent e) throws IOException {
        Player p = e.getPlayer();
        Block b = e.getBlock();
        if (b.getType().equals(Material.CHEST)) {
            BlockFace face = BlockFace.NORTH_EAST;
            for (BlockFace c : new BlockFace[]{BlockFace.NORTH,BlockFace.SOUTH,BlockFace.EAST,BlockFace.WEST}) {
                if (b.getRelative(c,1).getType().equals(Material.CHEST)) {
                    face = c;
                    break;
                }
            }
            if (face == BlockFace.NORTH_EAST) {
                if (config.isSet(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ() + ".owner")) {
                    if (!config.getString(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ() + ".owner").equals(UUIDS.getPlayerUUID(p.getName()))) {
                        e.setCancelled(true);
                        p.sendMessage(msg + " This chest has been locked by " + UUIDS.getPlayerName(config.getString(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ() + ".owner")));
                    }else{
                        config.set(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ(), null);
                        config.save(file);
                    }
                }
            }else{
                if (config.isSet(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ() + ".owner")) {
                    if (!config.getString(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ() + ".owner").equals(UUIDS.getPlayerUUID(p.getName()))) {
                        p.sendMessage(msg + " This chest has been locked by " + UUIDS.getPlayerName(config.getString(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ() + ".owner")));
                        e.setCancelled(true);
                    }else{
                        config.set(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ(), null);
                        config.save(file);
                    }
                }
                else if (config.isSet(b.getWorld().getName() + "." + b.getRelative(face,1).getX() + b.getRelative(face,1).getY() + b.getRelative(face,1).getZ() + ".owner")) {
                    if (!config.getString(b.getWorld().getName() + "." + b.getRelative(face,1).getX() + b.getRelative(face,1).getY() + b.getRelative(face,1).getZ() + ".owner").equals(UUIDS.getPlayerUUID(p.getName()))) {
                        e.setCancelled(true);
                        p.sendMessage(msg + " This chest has been locked by " + UUIDS.getPlayerName(config.getString(b.getWorld().getName() + "." + b.getRelative(face,1).getX() + b.getRelative(face,1).getY() + b.getRelative(face,1).getZ() + ".owner")));
                    }else{
                        config.set(b.getWorld().getName() + "." + b.getRelative(face,1).getX() + b.getRelative(face,1).getY() + b.getRelative(face,1).getZ(), null);
                        config.save(file);
                    }
                }
            }
            }
            else if (b.getType().equals(Material.FURNACE)) {
                if (config.isSet(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ() + ".owner")) {
                }
            }
    }

 

Gamerdude157 Phew, dirty code. I'd do the following:

1 - Use UUIDs, compatible between users.
2 - Instead of all the config.isSet's, just do something like this:
2.a - Use this class I created to store Locations: http://pastebin.com/YJLEbPYS
2.b - Use this config example to be able to support more than chest locking

Code:java

 
Config Example:
 
<uuid>
    world,0,10,123: CHEST
 

3 - As far as your issue, use my above examples and check if the config has <playerUUID>.<location>. If he does, let him break it. If not, cancel the event and tell him the block is protected by Bukkit.getOfflinePlayer(UUID);

 

So what exactly is the problem Dudemister1999 becuase i am using uuids so whats happening

 

Gamerdude157 So like you said, owners of blocks cannot break them. So if you store locations in configs like I showed above, you can just check if the config has the key <UUID>.<LOCATION>, then allow the block to be broken.

 

no Dudemister1999 I said that NON-OWNERS can not break them. But the config is set up correctly and the path is right but for some reason when the owner breaks the block the config doesn't save like i set it to. Read my code once more

hello?

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

 

Gamerdude157 I was agreeing with your statement. "So like you said, owners of blocks cannot break them." > "So, you had said that owners of blocks cannot break the owned blocks" (Translation)

 

ok whatever Dudemister1999 but why isn't the block being removed from the config? That is the actual problem. No errors at ALL just nothing

 

Gamerdude157 I just noticed something:

Code:java

config.set(b.getWorld().getName() + "." + b.getX() + b.getY() + b.getZ(), null);

That will create something like world.1051854, how would you figure out where the positions are?

 

what do you mean Dudemister1999 figure out where the positions are? I save them like that on purpose

hello Dudemister1999 ?

hello?

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

 

Gamerdude157 You can only bump in 12+ hours. (Unless it was changed to 24, like I've been hearing from time to time). Try to wait a while, people aren't hear to answer your every question immediately. As for your issue, it's most likely something obvious I missed. To be honest, the dirty(ish) code makes it hard to read, so I can't immediately identify the issue.

 

ok it has been 16 hours
bUmP

 

Gamerdude157

He means your getX, getY and getZ aren't spaced, so for example x: 20 y: 50 and z: 150 will convert into 2050150 and you have absolutely no way to differentiate between each coordinate.

 

OMG :O I AM SO STUPID
What would you reccomend for me to do then Watto and/or Dudemister1999

 

Gamerdude157 Use my class I posted.

 

Gamerdude157

Space them
getX() + " " + getY() + " " + getZ()