Specify which item is causing a player to be disabled. Fixes #909.

This commit is contained in:
Enerdy 2015-05-02 03:03:41 +01:00
parent 03c1182f27
commit f4fa624936
2 changed files with 67 additions and 69 deletions

View file

@ -2294,12 +2294,12 @@ namespace TShockAPI
if (control[5])
{
if (TShock.Itembans.ItemIsBanned(args.TPlayer.inventory[item].name, args.Player))
string itemName = args.TPlayer.inventory[item].name;
if (TShock.Itembans.ItemIsBanned(itemName, args.Player))
{
control[5] = false;
args.Player.Disable("Using banned item");
args.Player.SendErrorMessage("You cannot use {0} on this server. Your actions are being ignored.",
args.TPlayer.inventory[item].name);
args.Player.Disable("Using banned item ({0})".SFormat(itemName));
args.Player.SendErrorMessage("You cannot use {0} on this server. Your actions are being ignored.", itemName);
}
if (args.TPlayer.inventory[item].name == "Mana Crystal" && args.Player.TPlayer.statManaMax <= 180)
@ -3232,8 +3232,7 @@ namespace TShockAPI
}
if (TShock.Config.RangeChecks &&
TShock.CheckRangePermission(args.Player, (int) (Main.npc[id].position.X/16f), (int) (Main.npc[id].position.Y/16f),
128))
TShock.CheckRangePermission(args.Player, (int)(Main.npc[id].position.X / 16f), (int)(Main.npc[id].position.Y / 16f), 128))
{
args.Player.SendData(PacketTypes.NpcUpdate, "", id);
return true;

View file

@ -871,7 +871,6 @@ namespace TShockAPI
}
player.IgnoreActionsForCheating = check;
check = "none";
//todo: pretty sure we check every place a players inventory can change, so do we really need to do this?
foreach (Item item in player.TPlayer.armor)
{
if (Itembans.ItemIsBanned(item.name, player))