@@ -25,8 +25,6 @@ public class EssentialsUpgrade
 	private final static Logger LOGGER = Logger.getLogger("Essentials");
 	private final transient IEssentials ess;
 	private final transient EssentialsConf doneFile;
-	private String banReason;
-	private Long banTimeout;
 
 	EssentialsUpgrade(final IEssentials essentials)
 	{
@@ -676,36 +674,62 @@ public void banFormatChange()
 		{
 			return;
 		}
-		File[] playerFiles = userdir.listFiles();
 
-		for (File pFile : playerFiles)
+		int countFiles = 0;
+
+		ess.getLogger().info("Found " + userdir.list().length + " files to convert...");
+
+		for (String string : userdir.list())
 		{
-			EssentialsConf conf = new EssentialsConf(pFile);
+			if (!string.endsWith(".yml") || string.length() < 5)
+			{
+				continue;
+			}
+
+			final int showProgress = countFiles % 250;
+
+			if (showProgress == 0)
+			{
+				ess.getLogger().info("Converted " + countFiles + "/" + userdir.list().length);
+			}
+
+			countFiles++;
+			final File pFile = new File(userdir, string);
+			final EssentialsConf conf = new EssentialsConf(pFile);
 			conf.load();
+			
+			String banReason;
+			Long banTimeout;
+
 			try
 			{
 				banReason = conf.getConfigurationSection("ban").getString("reason");
 			}
 			catch (NullPointerException n)
 			{
-				//No ban in userfile
-				banReason = "";
+				banReason = null;
 			}
 
-			String playerName = conf.getString("lastAccountName");
-			if (!banReason.equals(""))
+			final String playerName = conf.getString("lastAccountName");
+			if (playerName != null && playerName.length() > 1 && banReason != null && banReason.length() > 1)
 			{
 				try
 				{
-					banTimeout = Long.parseLong(conf.getConfigurationSection("ban").getString("timeout"));
+					if (conf.getConfigurationSection("ban").contains("timeout"))
+					{
+						banTimeout = Long.parseLong(conf.getConfigurationSection("ban").getString("timeout"));
+					}
+					else
+					{
+						banTimeout = 0L;
+					}
 				}
 				catch (NumberFormatException n)
 				{
-					//No ban timeout set, or malformed timeout.
 					banTimeout = 0L;
 				}
-				org.bukkit.OfflinePlayer player = ess.getServer().getOfflinePlayer(playerName);
-				if (player.isBanned())
+				
+				if (ess.getServer().getBanList(BanList.Type.NAME).isBanned(playerName))
 				{
 					updateBan(playerName, banReason, banTimeout);
 				}
 
@@ -62,11 +62,13 @@ public void run(final Server server, final CommandSource sender, final String co
 		{
 			banReason = tl("defaultBanReason");
 		}
-
+		
 		Bukkit.getBanList(BanList.Type.NAME).addBan(user.getName(), banReason, null, senderName);
-		user.getBase().kickPlayer(tl("banFormat", banReason, senderName));
 
-		server.getLogger().log(Level.INFO, tl("playerBanned", senderName, user.getName(), banReason));
+		String banDisplay = tl("banFormat",  banReason, senderName);
+		
+		user.getBase().kickPlayer(banDisplay);
+		server.getLogger().log(Level.INFO, tl("playerBanned", senderName, user.getName(), banDisplay));
 
 		if (nomatch)
 		{
 
@@ -264,15 +264,14 @@ private void run_cleanup(final Server server, final CommandSource sender, final
 		{
 			sender.sendMessage("This sub-command will delete users who havent logged in in the last <days> days.");
 			sender.sendMessage("Optional parameters define the minium amount required to prevent deletion.");
-			sender.sendMessage("Unless you define larger default values, this command wil ignore people who have more than 0 money/homes/bans.");
-			throw new Exception("/<command> cleanup <days> [money] [homes] [ban count]");
+			sender.sendMessage("Unless you define larger default values, this command wil ignore people who have more than 0 money/homes.");
+			throw new Exception("/<command> cleanup <days> [money] [homes]");
 		}
 		sender.sendMessage(tl("cleaning"));
 
 		final long daysArg = Long.parseLong(args[1]);
 		final double moneyArg = args.length >= 3 ? Double.parseDouble(args[2].replaceAll("[^0-9\\.]", "")) : 0;
 		final int homesArg = args.length >= 4 && NumberUtil.isInt(args[3]) ? Integer.parseInt(args[3]) : 0;
-		final int bansArg = args.length >= 5 && NumberUtil.isInt(args[4]) ? Integer.parseInt(args[4]) : 0;
 		final UserMap userMap = ess.getUserMap();
 
 		ess.runTaskAsynchronously(new Runnable()
@@ -289,8 +288,6 @@ public void run()
 						continue;
 					}
 
-					int ban = user.getBase().isBanned() ? 0 : 1;
-
 					long lastLog = user.getLastLogout();
 					if (lastLog == 0)
 					{
@@ -311,7 +308,7 @@ public void run()
 					int homeCount = user.getHomes().size();
 					double moneyCount = user.getMoney().doubleValue();
 
-					if ((lastLog == 0) || (ban > bansArg) || (timeDiff < milliDays)
+					if ((lastLog == 0) || (timeDiff < milliDays)
 						|| (homeCount > homesArg) || (moneyCount > moneyArg))
 					{
 						continue;
 
@@ -56,13 +56,15 @@ protected void seen(final Server server, final CommandSource sender, final Strin
 					seenIP(server, sender, args[0]);
 					return;
 				}
-				else if (FormatUtil.validIP(args[0]) && (server.getIPBans().contains(args[0])))
+				else if (Bukkit.getBanList(BanList.Type.IP).isBanned(args[0]))
 				{
 					sender.sendMessage(tl("isIpBanned", args[0]));
 					return;
 				}
-				else if (Bukkit.getBannedPlayers().contains(Bukkit.getOfflinePlayer(args[0]))) {
-					sender.sendMessage(tl("whoisBanned", showBan ? Bukkit.getBanList(BanList.Type.NAME).getBanEntry(Bukkit.getOfflinePlayer(args[0]).getName()).getReason() : tl("true")));
+				
+				else if (Bukkit.getBanList(BanList.Type.NAME).isBanned(args[0]))
+				{
+					sender.sendMessage(tl("whoisBanned", showBan ? Bukkit.getBanList(BanList.Type.NAME).getBanEntry(args[0]).getReason() : tl("true")));
 					return;
 				}
 				else
@@ -168,7 +170,7 @@ private void seenIP(final Server server, final CommandSource sender, final Strin
 	{
 		final UserMap userMap = ess.getUserMap();
 
-		if (server.getIPBans().contains(ipAddress))
+		if (Bukkit.getBanList(BanList.Type.IP).isBanned(ipAddress))
 		{
 			sender.sendMessage(tl("isIpBanned", ipAddress));
 		}
 
@@ -47,26 +47,27 @@ public void run(final Server server, final CommandSource sender, final String co
 		}
 		final String time = getFinalArg(args, 1);
 		final long banTimestamp = DateUtil.parseDateDiff(time, true);
-		String stringDregs = DateUtil.removeTimePattern(time);
-		
+		String banReason = DateUtil.removeTimePattern(time);
+
 		final long maxBanLength = ess.getSettings().getMaxTempban() * 1000;
 		if (maxBanLength > 0 && ((banTimestamp - GregorianCalendar.getInstance().getTimeInMillis()) > maxBanLength)
 			&& sender.isPlayer() && !(ess.getUser(sender.getPlayer()).isAuthorized("essentials.tempban.unlimited")))
 		{
 			sender.sendMessage(tl("oversizedTempban"));
 			throw new NoChargeException();
 		}
-		
-		if (stringDregs.length() < 2) 
+
+		if (banReason.length() < 2)
 		{
-			stringDregs = tl("defaultBanReason");
+			banReason = tl("defaultBanReason");
 		}
 
 		final String senderName = sender.isPlayer() ? sender.getPlayer().getDisplayName() : Console.NAME;
-		final String banReason = tl("tempBanned", DateUtil.formatDateDiff(banTimestamp), senderName, stringDregs);
-
 		Bukkit.getBanList(BanList.Type.NAME).addBan(user.getName(), banReason, new Date(banTimestamp), senderName);
-		user.getBase().kickPlayer(banReason);
+
+		String banDisplay = tl("tempBanned", DateUtil.formatDateDiff(banTimestamp), senderName, banReason);
+		user.getBase().kickPlayer(banDisplay);
+		server.getLogger().log(Level.INFO, tl("playerBanned", senderName, user.getName(), banDisplay));
 
 		final String message = tl("playerBanned", senderName, user.getName(), banReason, DateUtil.formatDateDiff(banTimestamp));
 		server.getLogger().log(Level.INFO, message);
 
@@ -27,7 +27,7 @@ balance=\u00a7aBalance\:\u00a7c {0}
 balanceOther=\u00a7aBalance of {0}\u00a7a\:\u00a7c {1}
 balanceTop=\u00a76Top balances ({0})
 banExempt=\u00a74You cannot ban that player.
-banFormat=\u00a74Banned\:\n\u00a7r{0}
+banFormat=\u00a7cYou have been banned\:\n\u00a7r{0}
 bed=\u00a7obed\u00a7r
 bedMissing=\u00a74Your bed is either unset, missing or blocked.
 bedNull=\u00a7mbed\u00a7r
@@ -155,7 +155,7 @@ holdBook=\u00a74You are not holding a writable book.
 holdFirework=\u00a74You must be holding a firework to add effects.
 holdPotion=\u00a74You must be holding a potion to apply effects to it.
 holeInFloor=\u00a74Hole in floor\!
-homeSet=\u00a76Home set.
+homeSet=\u00a76Home set to current location.
 homes=\u00a76Homes\:\u00a7r {0}
 hour=hour
 hours=hours
@@ -321,8 +321,8 @@ pWeatherPlayers=\u00a76These players have their own weather\:\u00a7r
 pWeatherReset=\u00a76Player weather has been reset for\: \u00a7c{0}
 pWeatherSet=\u00a76Player weather is set to \u00a7c{0}\u00a76 for\: \u00a7c{1}.
 pendingTeleportCancelled=\u00a74Pending teleportation request cancelled.
-playerBanIpAddress=\u00a76Player\u00a7c {0} \u00a76banned IP address\:\u00a7c {1}\u00a76.
-playerBanned=\u00a76Player\u00a7c {0} \u00a76banned\u00a7c {1} \u00a76for \u00a7c{2}\u00a76.
+playerBanIpAddress=\u00a76Player\u00a7c {0} \u00a76banned IP address\u00a7c {1} \u00a76for\: \u00a7c{2}\u00a76.
+playerBanned=\u00a76Player\u00a7c {0} \u00a76banned\u00a7c {1} \u00a76for\: \u00a7c{2}\u00a76.
 playerInJail=\u00a74Player is already in jail\u00a7c {0}\u00a74.
 playerJailed=\u00a76Player\u00a7c {0} \u00a76jailed.
 playerJailedFor=\u00a76Player\u00a7c {0} \u00a76jailed for {1}.
@@ -426,7 +426,7 @@ teleportationEnabled=\u00a76Teleportation \u00a7cenabled\u00a76.
 teleportationEnabledFor=\u00a76Teleportation \u00a7cenabled \u00a76for \u00a7c{0}\u00a76.
 teleporting=\u00a76Teleporting...
 teleportToPlayer=\u00a76Teleporting to \u00a7c{0}\u00a76.
-tempBanned=Temporarily banned from server for {0}.
+tempBanned=\u00a7cYou have been temporarily banned for {0}\:\n\u00a7r{2}
 tempbanExempt=\u00a74You may not tempban that player.
 thunder=\u00a76You\u00a7c {0} \u00a76thunder in your world.
 thunderDuration=\u00a76You\u00a7c {0} \u00a76thunder in your world for\u00a7c {1} \u00a76seconds.
Original file line number	Diff line number	Diff line change
`@@ -25,8 +25,6 @@ public class EssentialsUpgrade`
`25`	`25`	`private final static Logger LOGGER = Logger.getLogger("Essentials");`
`26`	`26`	`private final transient IEssentials ess;`
`27`	`27`	`private final transient EssentialsConf doneFile;`
`28`		`- private String banReason;`
`29`		`- private Long banTimeout;`
`30`	`28`
`31`	`29`	`EssentialsUpgrade(final IEssentials essentials)`
`32`	`30`	`{`
`@@ -676,36 +674,62 @@ public void banFormatChange()`
`676`	`674`	`{`
`677`	`675`	`return;`
`678`	`676`	`}`
`679`		`- File[] playerFiles = userdir.listFiles();`
`680`	`677`
`681`		`- for (File pFile : playerFiles)`
	`678`	`+ int countFiles = 0;`
	`679`	`+`
	`680`	`+ ess.getLogger().info("Found " + userdir.list().length + " files to convert...");`
	`681`	`+`
	`682`	`+ for (String string : userdir.list())`
`682`	`683`	`{`
`683`		`- EssentialsConf conf = new EssentialsConf(pFile);`
	`684`	`+ if (!string.endsWith(".yml") \|\| string.length() < 5)`
	`685`	`+ {`
	`686`	`+ continue;`
	`687`	`+ }`
	`688`	`+`
	`689`	`+ final int showProgress = countFiles % 250;`
	`690`	`+`
	`691`	`+ if (showProgress == 0)`
	`692`	`+ {`
	`693`	`+ ess.getLogger().info("Converted " + countFiles + "/" + userdir.list().length);`
	`694`	`+ }`
	`695`	`+`
	`696`	`+ countFiles++;`
	`697`	`+ final File pFile = new File(userdir, string);`
	`698`	`+ final EssentialsConf conf = new EssentialsConf(pFile);`
`684`	`699`	`conf.load();`
	`700`	`+`
	`701`	`+ String banReason;`
	`702`	`+ Long banTimeout;`
	`703`	`+`
`685`	`704`	`try`
`686`	`705`	`{`
`687`	`706`	`banReason = conf.getConfigurationSection("ban").getString("reason");`
`688`	`707`	`}`
`689`	`708`	`catch (NullPointerException n)`
`690`	`709`	`{`
`691`		`- //No ban in userfile`
`692`		`- banReason = "";`
	`710`	`+ banReason = null;`
`693`	`711`	`}`
`694`	`712`
`695`		`- String playerName = conf.getString("lastAccountName");`
`696`		`- if (!banReason.equals(""))`
	`713`	`+ final String playerName = conf.getString("lastAccountName");`
	`714`	`+ if (playerName != null && playerName.length() > 1 && banReason != null && banReason.length() > 1)`
`697`	`715`	`{`
`698`	`716`	`try`
`699`	`717`	`{`
`700`		`- banTimeout = Long.parseLong(conf.getConfigurationSection("ban").getString("timeout"));`
	`718`	`+ if (conf.getConfigurationSection("ban").contains("timeout"))`
	`719`	`+ {`
	`720`	`+ banTimeout = Long.parseLong(conf.getConfigurationSection("ban").getString("timeout"));`
	`721`	`+ }`
	`722`	`+ else`
	`723`	`+ {`
	`724`	`+ banTimeout = 0L;`
	`725`	`+ }`
`701`	`726`	`}`
`702`	`727`	`catch (NumberFormatException n)`
`703`	`728`	`{`
`704`		`- //No ban timeout set, or malformed timeout.`
`705`	`729`	`banTimeout = 0L;`
`706`	`730`	`}`
`707`		`- org.bukkit.OfflinePlayer player = ess.getServer().getOfflinePlayer(playerName);`
`708`		`- if (player.isBanned())`
	`731`	`+`
	`732`	`+ if (ess.getServer().getBanList(BanList.Type.NAME).isBanned(playerName))`
`709`	`733`	`{`
`710`	`734`	`updateBan(playerName, banReason, banTimeout);`
`711`	`735`	`}`
Original file line number	Diff line number	Diff line change
`@@ -62,11 +62,13 @@ public void run(final Server server, final CommandSource sender, final String co`
`62`	`62`	`{`
`63`	`63`	`banReason = tl("defaultBanReason");`
`64`	`64`	`}`
`65`		`-`
	`65`	`+`
`66`	`66`	`Bukkit.getBanList(BanList.Type.NAME).addBan(user.getName(), banReason, null, senderName);`
`67`		`- user.getBase().kickPlayer(tl("banFormat", banReason, senderName));`
`68`	`67`
`69`		`- server.getLogger().log(Level.INFO, tl("playerBanned", senderName, user.getName(), banReason));`
	`68`	`+ String banDisplay = tl("banFormat", banReason, senderName);`
	`69`	`+`
	`70`	`+ user.getBase().kickPlayer(banDisplay);`
	`71`	`+ server.getLogger().log(Level.INFO, tl("playerBanned", senderName, user.getName(), banDisplay));`
`70`	`72`
`71`	`73`	`if (nomatch)`
`72`	`74`	`{`
Original file line number	Diff line number	Diff line change
`@@ -56,13 +56,15 @@ protected void seen(final Server server, final CommandSource sender, final Strin`
`56`	`56`	`seenIP(server, sender, args[0]);`
`57`	`57`	`return;`
`58`	`58`	`}`
`59`		`- else if (FormatUtil.validIP(args[0]) && (server.getIPBans().contains(args[0])))`
	`59`	`+ else if (Bukkit.getBanList(BanList.Type.IP).isBanned(args[0]))`
`60`	`60`	`{`
`61`	`61`	`sender.sendMessage(tl("isIpBanned", args[0]));`
`62`	`62`	`return;`
`63`	`63`	`}`
`64`		`- else if (Bukkit.getBannedPlayers().contains(Bukkit.getOfflinePlayer(args[0]))) {`
`65`		`- sender.sendMessage(tl("whoisBanned", showBan ? Bukkit.getBanList(BanList.Type.NAME).getBanEntry(Bukkit.getOfflinePlayer(args[0]).getName()).getReason() : tl("true")));`
	`64`	`+`
	`65`	`+ else if (Bukkit.getBanList(BanList.Type.NAME).isBanned(args[0]))`
	`66`	`+ {`
	`67`	`+ sender.sendMessage(tl("whoisBanned", showBan ? Bukkit.getBanList(BanList.Type.NAME).getBanEntry(args[0]).getReason() : tl("true")));`
`66`	`68`	`return;`
`67`	`69`	`}`
`68`	`70`	`else`
`@@ -168,7 +170,7 @@ private void seenIP(final Server server, final CommandSource sender, final Strin`
`168`	`170`	`{`
`169`	`171`	`final UserMap userMap = ess.getUserMap();`
`170`	`172`
`171`		`- if (server.getIPBans().contains(ipAddress))`
	`173`	`+ if (Bukkit.getBanList(BanList.Type.IP).isBanned(ipAddress))`
`172`	`174`	`{`
`173`	`175`	`sender.sendMessage(tl("isIpBanned", ipAddress));`
`174`	`176`	`}`