From 2eeabb4c2f94b4071401b7306af35fbf273b564f Mon Sep 17 00:00:00 2001 From: Lycaonj Date: Sat, 30 Jul 2011 22:34:04 -0500 Subject: [PATCH] Added null reference check when unbanning a player name and EnableBanOnUsernames is set to false. Few grammar related fixes related to unbanning. --- TShockAPI/Commands.cs | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/TShockAPI/Commands.cs b/TShockAPI/Commands.cs index 6daaf9c0..714e06b0 100644 --- a/TShockAPI/Commands.cs +++ b/TShockAPI/Commands.cs @@ -1,4 +1,4 @@ -/* +/* TShock, a server mod for Terraria Copyright (C) 2011 The TShock Team @@ -704,15 +704,18 @@ namespace TShockAPI if (TShock.Bans.RemoveBan(ban.IP)) args.Player.SendMessage(string.Format("Unbanned {0} ({1})!", ban.Name, ban.IP), Color.Red); else - args.Player.SendMessage(string.Format("Failed to Unbanned {0} ({1})!", ban.Name, ban.IP), Color.Red); + args.Player.SendMessage(string.Format("Failed to unban {0} ({1})!", ban.Name, ban.IP), Color.Red); } else if (!TShock.Config.EnableBanOnUsernames) { ban = TShock.Bans.GetBanByIp(plStr); - if (TShock.Bans.RemoveBan(ban.IP)) + + if (ban == null) + args.Player.SendMessage(string.Format("Failed to unban {0}, not found.", args.Parameters[0]), Color.Red); + else if (TShock.Bans.RemoveBan(ban.IP)) args.Player.SendMessage(string.Format("Unbanned {0} ({1})!", ban.Name, ban.IP), Color.Red); else - args.Player.SendMessage(string.Format("Failed to Unbanned {0} ({1})!", ban.Name, ban.IP), Color.Red); + args.Player.SendMessage(string.Format("Failed to unban {0} ({1})!", ban.Name, ban.IP), Color.Red); } else { @@ -781,7 +784,7 @@ namespace TShockAPI if (TShock.Bans.RemoveBan(ban.IP)) args.Player.SendMessage(string.Format("Unbanned {0} ({1})!", ban.Name, ban.IP), Color.Red); else - args.Player.SendMessage(string.Format("Failed to Unbanned {0} ({1})!", ban.Name, ban.IP), Color.Red); + args.Player.SendMessage(string.Format("Failed to unban {0} ({1})!", ban.Name, ban.IP), Color.Red); } else {