--- miscJava/bukkit-minecraft-plugins/HoerupUtils/src/main/java/dk/thoerup/bukkit/hoeruputils/chests/AdvancedChest.java	2012/11/08 19:47:51	1871
+++ miscJava/bukkit-minecraft-plugins/HoerupUtils/src/main/java/dk/thoerup/bukkit/hoeruputils/chests/AdvancedChest.java	2012/12/18 10:33:47	1904
@@ -137,6 +137,7 @@
 				addChest(loc, chest);
 				player.sendMessage("Chest is now " + modeStr);
 			} else {
+				server.getLogger().info( player.getName() + " tried to protect a chest owned by " + chest.getOwner() );
 				player.sendMessage("This chest is already protected");
 			}
 			return true;
@@ -149,6 +150,7 @@
 			}
 			if (! chest.getOwner().equals( player.getName() ) ) {
 				player.sendMessage("You can not remove lock from a chest you don't own");
+				server.getLogger().info( player.getName() + " tried to remove protection froma chest owned by " + chest.getOwner() );
 				return true;
 			}
 
@@ -165,6 +167,7 @@
 			}
 			if (! chest.getOwner().equals( player.getName() ) ) {
 				player.sendMessage("You can not set new owner of a chest you dont own");
+				server.getLogger().info( player.getName() + " tried to set owner on a chest owned by " + chest.getOwner() );
 				return true;
 			}
 			if (args.length != 2) {
@@ -189,6 +192,7 @@
 			}
 			if (! chest.getOwner().equals( player.getName() ) ) {
 				player.sendMessage("You can not add/remove players from a chest you don't own");
+				server.getLogger().info( player.getName() + " tried to add/remove player on a chest owned by " + chest.getOwner() );
 				return true;
 			}
 			if (args.length != 2) {
@@ -218,8 +222,9 @@
 				player.sendMessage("This chest is not protected");
 				return true;
 			}
-			if (! chest.getOwner().equals( player.getName() ) ) {
+			if (! chest.getOwner().equals( player.getName() ) ) {				
 				player.sendMessage("You can not comment a chest you don't own");
+				server.getLogger().info( player.getName() + " tried to comment on a chest owned by " + chest.getOwner() );
 				return true;
 			}
 
@@ -465,7 +470,7 @@
 					return; //chest is opened by it's owner
 				}
 				
-				Set<String> players = Util.stringToSet( chest.getModifyPlayers() );
+				Set<String> players = chest.getModifyPlayersSet() ;
 				if ( players.contains(player.getName()) ) {
 					return; //this player is on the whitelist so he may open
 				}
@@ -506,7 +511,7 @@
 				return; //chest is owned by it's own player
 			}
 			
-			Set<String> players = Util.stringToSet( chest.getModifyPlayers() );
+			Set<String> players = chest.getModifyPlayersSet();
 			if ( players.contains(player.getName()) ) {
 				return; //this player is on the whitelist so he may open
 			}