Improved chest-sign protection.

src/main/java/com/sk89q/worldguard/bukkit/SignChestProtection.java

@@ -46,6 +46,10 @@ public boolean isProtected(Block block, Player player) {
         }
     }
     
+    public boolean isProtectedPlacement(Block block, Player player) {
+        return isProtectedSignAround(block, player);
+    }
+    
     private boolean isProtectedSignAround(Block searchBlock, Player player) {
         Block side;
         Boolean res;

src/main/java/com/sk89q/worldguard/bukkit/WorldConfiguration.java

@@ -347,4 +347,14 @@ public boolean isChestProtected(Block block) {
         return chestProtection.isProtected(block, null);
     }
     
+    public boolean isChestProtectedPlacement(Block block, Player player) {
+        if (!signChestProtection) {
+            return false;
+        }
+        if (plugin.hasPermission(player, "worldguard.chest-protection.override")) {
+            return false;
+        }
+        return chestProtection.isProtectedPlacement(block, player);
+    }
+
 }

src/main/java/com/sk89q/worldguard/bukkit/WorldGuardBlockListener.java

@@ -365,6 +365,7 @@ public void onBlockBurn(BlockBurnEvent event) {
             }
         }
 
+        if (wcfg.isChestProtected(event.getBlock())) {            event.setCancelled(true);            return;        }
         if (wcfg.useRegions) {
             Block block = event.getBlock();
             Vector pt = toVector(block);
@@ -512,6 +513,13 @@ public void onSignChange(SignChangeEvent event) {
         
         if (wcfg.signChestProtection) {
             if (event.getLine(0).equalsIgnoreCase("[Lock]")) {
+                if (wcfg.isChestProtectedPlacement(event.getBlock(), player)) {
+                    player.sendMessage(ChatColor.DARK_RED + "You do not own the adjacent chest.");
+                    dropSign(event.getBlock());
+                    event.setCancelled(true);
+                    return;
+                }
+                
                 if (event.getBlock().getType() != Material.SIGN_POST) {
                     player.sendMessage(ChatColor.RED
                             + "The [Lock] sign must be a sign post, not a wall sign.");
@@ -519,18 +527,32 @@ public void onSignChange(SignChangeEvent event) {
                     dropSign(event.getBlock());
                     event.setCancelled(true);
                     return;
-                } else if (!event.getLine(1).equalsIgnoreCase(player.getName())) {
+                }
+
+                if (!event.getLine(1).equalsIgnoreCase(player.getName())) {
                     player.sendMessage(ChatColor.RED
                             + "The first owner line must be your name.");
 
                     dropSign(event.getBlock());
                     event.setCancelled(true);
                     return;
-                } else {
-                    event.setLine(0, "[Lock]");
-                    player.sendMessage(ChatColor.YELLOW
-                            + "A chest or double chest above is now protected.");
                 }
+                
+                Material below = event.getBlock().getRelative(0, -1, 0).getType();
+
+                if (below == Material.TNT || below == Material.SAND
+                        || below == Material.GRAVEL || below == Material.SIGN_POST) {
+                    player.sendMessage(ChatColor.RED
+                            + "That is not a safe block that you're putting this sign on.");
+
+                    dropSign(event.getBlock());
+                    event.setCancelled(true);
+                    return;
+                }
+                
+                event.setLine(0, "[Lock]");
+                player.sendMessage(ChatColor.YELLOW
+                        + "A chest or double chest above is now protected.");
             }
         } else {
             if (event.getLine(0).equalsIgnoreCase("[Lock]")) {