themaster1970sf/TShock
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

TShock the string.Format edition specially for high6.

Browse source
This commit is contained in:
ricky 2011-06-11 00:44:11 +10:00
parent c964b3b2eb
commit 3c0cb13222
3 changed files with 81 additions and 88 deletions
Download patch file Download diff file Expand all files Collapse all files

101
TShockAPI/Commands.cs
View file

@ -226,7 +226,7 @@ namespace TShockAPI
int playerTeam = Main.player[args.PlayerID].team; int playerTeam = Main.player[args.PlayerID].team;
if (playerTeam != 0) if (playerTeam != 0)
{ {
string msg = "<" + Main.player[args.PlayerID].name + "> " + args.Message.Remove(0, 2); string msg = string.Format("<{0}> {1}", Main.player[args.PlayerID].name, args.Message.Remove(0, 2));
for (int i = 0; i < Main.player.Length; i++) for (int i = 0; i < Main.player.Length; i++)
{ {
if (Main.player[i].team == Main.player[args.PlayerID].team) if (Main.player[i].team == Main.player[args.PlayerID].team)
@ -244,40 +244,33 @@ namespace TShockAPI
public static void ThirdPerson(CommandArgs args) public static void ThirdPerson(CommandArgs args)
{ {
string msg = args.Message.Remove(0, 3); string msg = args.Message.Remove(0, 3);
Tools.Broadcast("*" + Tools.FindPlayer(args.PlayerID) + " " + msg, 205, 133, 63); Tools.Broadcast(string.Format("*{0} {1}", Tools.FindPlayer(args.PlayerID), msg, 205, 133, 63));
} }
public static void Playing(CommandArgs args) public static void Playing(CommandArgs args)
{ {
Tools.SendMessage(args.PlayerID, "Current players: " + Tools.GetPlayers(), 255f, 240f, 20f); Tools.SendMessage(args.PlayerID, string.Format("Current players: {0}.", Tools.GetPlayers()), 255f, 240f, 20f);
} }
public static void DebugConfiguration(CommandArgs args) public static void DebugConfiguration(CommandArgs args)
{ {
int ply = args.PlayerID; int ply = args.PlayerID;
Tools.SendMessage(ply, "TShock Config:"); Tools.SendMessage(ply, "TShock Config:");
string lineOne = ""; string lineOne = string.Format("BanCheater : {0}, KickCheater : {1}, BanGriefer : {2}, KickGriefer : {3}",
lineOne += "KickCheater : " + ConfigurationManager.kickCheater + ", "; ConfigurationManager.banCheater, ConfigurationManager.kickCheater,
lineOne += "BanCheater : " + ConfigurationManager.banCheater + ", "; ConfigurationManager.banGriefer, ConfigurationManager.kickGriefer);
lineOne += "KickGriefer : " + ConfigurationManager.kickGriefer + ", ";
lineOne += "BanGriefer : " + ConfigurationManager.banGriefer;
Tools.SendMessage(ply, lineOne, Color.Yellow); Tools.SendMessage(ply, lineOne, Color.Yellow);
string lineTwo = ""; string lineTwo = string.Format("BanTnt : {0}, KickTnt : {1}, BanBoom : {2}, KickBoom : {3}",
lineTwo += "BanTnt : " + ConfigurationManager.banTnt + ", "; ConfigurationManager.banTnt, ConfigurationManager.kickTnt,
lineTwo += "KickTnt : " + ConfigurationManager.kickTnt + ", "; ConfigurationManager.banBoom, ConfigurationManager.kickBoom);
lineTwo += "BanBoom : " + ConfigurationManager.banBoom + ", ";
lineTwo += "KickBoom : " + ConfigurationManager.kickBoom;
Tools.SendMessage(ply, lineTwo, Color.Yellow); Tools.SendMessage(ply, lineTwo, Color.Yellow);
string lineThree = ""; string lineThree = string.Format("InvMultiplier : {0}, ProtectS : {1}, ProtectR : {2}, DMS : {3}, SpawnRate {4}",
lineThree += "InvMultiplier : " + ConfigurationManager.invasionMultiplier + ", "; ConfigurationManager.invasionMultiplier, ConfigurationManager.spawnProtect,
lineThree += "ProtectS : " + ConfigurationManager.spawnProtect + ", "; ConfigurationManager.spawnProtectRadius, ConfigurationManager.defaultMaxSpawns,
lineThree += "ProtectR : " + ConfigurationManager.spawnProtectRadius + ", "; ConfigurationManager.defaultSpawnRate);
lineThree += "DMS : " + ConfigurationManager.defaultMaxSpawns + ", ";
lineThree += "SpawnRate: " + ConfigurationManager.defaultSpawnRate;
Tools.SendMessage(ply, lineThree, Color.Yellow); Tools.SendMessage(ply, lineThree, Color.Yellow);
string lineFour = ""; string lineFour = string.Format("MaxSlots : {0}, RangeChecks : {1}",
lineFour += "MaxSlots : " + ConfigurationManager.maxSlots + ", "; ConfigurationManager.maxSlots, ConfigurationManager.rangeChecks);
lineFour += "RangeChecks : " + ConfigurationManager.rangeChecks + ", ";
Tools.SendMessage(ply, lineFour, Color.Yellow); Tools.SendMessage(ply, lineFour, Color.Yellow);
} }
@ -468,7 +461,7 @@ namespace TShockAPI
public static void Bloodmoon(CommandArgs args) public static void Bloodmoon(CommandArgs args)
{ {
int ply = args.PlayerID; int ply = args.PlayerID;
Tools.Broadcast(Tools.FindPlayer(ply) + " turned on blood moon."); Tools.Broadcast(string.Format("{0} turned on blood moon.", Tools.FindPlayer(ply)));
Main.bloodMoon = true; Main.bloodMoon = true;
Main.time = 0; Main.time = 0;
Main.dayTime = false; Main.dayTime = false;
@ -482,7 +475,7 @@ namespace TShockAPI
int y = args.PlayerY; int y = args.PlayerY;
int ply = args.PlayerID; int ply = args.PlayerID;
Tools.NewNPC((int)ConfigurationManager.NPCList.WORLD_EATER, x, y, ply); Tools.NewNPC((int)ConfigurationManager.NPCList.WORLD_EATER, x, y, ply);
Tools.Broadcast(Tools.FindPlayer(ply) + " has spawned an eater of worlds!"); Tools.Broadcast(string.Format("{0} has spawned an eater of worlds!", Tools.FindPlayer(ply)));
} }
public static void Eye(CommandArgs args) public static void Eye(CommandArgs args)
@ -491,7 +484,7 @@ namespace TShockAPI
int y = args.PlayerY; int y = args.PlayerY;
int ply = args.PlayerID; int ply = args.PlayerID;
Tools.NewNPC((int)ConfigurationManager.NPCList.EYE, x, y, ply); Tools.NewNPC((int)ConfigurationManager.NPCList.EYE, x, y, ply);
Tools.Broadcast(Tools.FindPlayer(ply) + " has spawned an eye!"); Tools.Broadcast(string.Format("{0} has spawned an eye!", Tools.FindPlayer(ply)));
} }
public static void Skeletron(CommandArgs args) public static void Skeletron(CommandArgs args)
@ -500,7 +493,7 @@ namespace TShockAPI
int y = args.PlayerY; int y = args.PlayerY;
int ply = args.PlayerID; int ply = args.PlayerID;
Tools.NewNPC((int)ConfigurationManager.NPCList.SKELETRON, x, y, ply); Tools.NewNPC((int)ConfigurationManager.NPCList.SKELETRON, x, y, ply);
Tools.Broadcast(Tools.FindPlayer(ply) + " has spawned skeletron!"); Tools.Broadcast(string.Format("{0} has spawned skeletron!", Tools.FindPlayer(ply)));
} }
public static void Hardcore(CommandArgs args) public static void Hardcore(CommandArgs args)
@ -512,7 +505,7 @@ namespace TShockAPI
{ {
Tools.NewNPC(i, x, y, ply); Tools.NewNPC(i, x, y, ply);
} }
Tools.Broadcast(Tools.FindPlayer(ply) + " has spawned all 3 bosses!"); Tools.Broadcast(string.Format("{0} has spawned all 3 bosses!", Tools.FindPlayer(ply)));
} }
public static void Invade(CommandArgs args) public static void Invade(CommandArgs args)
@ -520,12 +513,12 @@ namespace TShockAPI
int ply = args.PlayerID; int ply = args.PlayerID;
if (Main.invasionSize <= 0) if (Main.invasionSize <= 0)
{ {
Tools.Broadcast(Main.player[ply].name + " has started an invasion."); Tools.Broadcast(string.Format("{0} has started an invasion.", Tools.FindPlayer(ply)));
TShock.StartInvasion(); TShock.StartInvasion();
} }
else else
{ {
Tools.Broadcast(Main.player[ply].name + " has ended an invasion."); Tools.Broadcast(string.Format("{0} has ended an invasion.", Tools.FindPlayer(ply)));
Main.invasionSize = 0; Main.invasionSize = 0;
} }
} }
@ -535,7 +528,7 @@ namespace TShockAPI
int ply = args.PlayerID; int ply = args.PlayerID;
string passwd = args.Message.Remove(0, 9).Trim(); string passwd = args.Message.Remove(0, 9).Trim();
Netplay.password = passwd; Netplay.password = passwd;
Tools.SendMessage(ply, "Server password changed to: " + passwd); Tools.SendMessage(ply, string.Format("Server password changed to: {0}", passwd));
} }
public static void Save(CommandArgs args) public static void Save(CommandArgs args)
@ -600,7 +593,7 @@ namespace TShockAPI
else else
{ {
TShock.Teleport(adminplr, Main.player[player].position.X, Main.player[player].position.Y); TShock.Teleport(adminplr, Main.player[player].position.X, Main.player[player].position.Y);
Tools.SendMessage(adminplr, "Teleported to " + Tools.FindPlayer(player)); Tools.SendMessage(adminplr, string.Format("Teleported to {0}", Tools.FindPlayer(player)));
} }
} }
@ -623,8 +616,8 @@ namespace TShockAPI
else else
{ {
TShock.Teleport(player, Main.player[adminplr].position.X, Main.player[adminplr].position.Y); TShock.Teleport(player, Main.player[adminplr].position.X, Main.player[adminplr].position.Y);
Tools.SendMessage(player, "You were teleported to " + Tools.FindPlayer(adminplr) + "."); Tools.SendMessage(player, string.Format("You were teleported to {0}.", Tools.FindPlayer(adminplr)));
Tools.SendMessage(adminplr, "You brought " + Tools.FindPlayer(player) + " here."); Tools.SendMessage(adminplr, string.Format("You brought {0} here.", Tools.FindPlayer(player)));
} }
} }
@ -705,7 +698,7 @@ namespace TShockAPI
Main.item[id].stack = Main.item[id].maxStack; Main.item[id].stack = Main.item[id].maxStack;
//TShock.SendDataAll(21, -1, "", id); //TShock.SendDataAll(21, -1, "", id);
NetMessage.SendData(21, -1, -1, "", id, 0f, 0f, 0f); NetMessage.SendData(21, -1, -1, "", id, 0f, 0f, 0f);
Tools.SendMessage(adminplr, "Got some " + Main.item[id].name + "."); Tools.SendMessage(adminplr, string.Format("Got some {0}.", Main.item[id].name));
//TShock.UpdateInventories(); //TShock.UpdateInventories();
flag = true; flag = true;
break; break;
@ -832,7 +825,7 @@ namespace TShockAPI
killcount++; killcount++;
} }
} }
Tools.Broadcast("Killed " + killcount + " NPCs."); Tools.Broadcast(string.Format("Killed {0} NPCs.", killcount));
} }
public static void MaxSpawns(CommandArgs args) public static void MaxSpawns(CommandArgs args)
@ -849,7 +842,7 @@ namespace TShockAPI
int.TryParse(args.Parameters[0], out amount); int.TryParse(args.Parameters[0], out amount);
NPC.defaultMaxSpawns = amount; NPC.defaultMaxSpawns = amount;
ConfigurationManager.defaultMaxSpawns = amount; ConfigurationManager.defaultMaxSpawns = amount;
Tools.Broadcast(Tools.FindPlayer(adminplr) + " changed the maximum spawns to: " + amount); Tools.Broadcast(string.Format("{0} changed the maximum spawns to: {1}", Tools.FindPlayer(adminplr), amount));
} }
public static void SpawnRate(CommandArgs args) public static void SpawnRate(CommandArgs args)
@ -866,14 +859,13 @@ namespace TShockAPI
int.TryParse(args.Parameters[0], out amount); int.TryParse(args.Parameters[0], out amount);
NPC.defaultSpawnRate = amount; NPC.defaultSpawnRate = amount;
ConfigurationManager.defaultSpawnRate = amount; ConfigurationManager.defaultSpawnRate = amount;
Tools.Broadcast(Tools.FindPlayer(adminplr) + " changed the spawn rate to: " + amount); Tools.Broadcast(string.Format("{0} changed the spawn rate to: {1}", Tools.FindPlayer(adminplr), amount));
} }
public static void Help(CommandArgs args) public static void Help(CommandArgs args)
{ {
int ply = args.PlayerID; int ply = args.PlayerID;
Tools.SendMessage(ply, "TShock Commands:"); Tools.SendMessage(ply, "TShock Commands:");
string tempstring = "";
int page = 1; int page = 1;
if (args.Parameters.Count > 0) if (args.Parameters.Count > 0)
int.TryParse(args.Parameters[0], out page); int.TryParse(args.Parameters[0], out page);
@ -885,26 +877,29 @@ namespace TShockAPI
cmdlist.Add(commands[j]); cmdlist.Add(commands[j]);
} }
} }
var sb = new StringBuilder();
if (cmdlist.Count > (15 * (page - 1))) if (cmdlist.Count > (15 * (page - 1)))
{ {
for (int j = (15 * (page - 1)); j < (15 * page); j++) for (int j = (15 * (page - 1)); j < (15 * page); j++)
{ {
tempstring += "/" + cmdlist[j].Name() + ", "; if (sb.Length != 0)
sb.Append(", ");
sb.Append("/").Append(cmdlist[j].Name());
if (j == cmdlist.Count - 1) if (j == cmdlist.Count - 1)
{ {
Tools.SendMessage(ply, tempstring.TrimEnd(new[] { ' ', ',' }), Color.Yellow); Tools.SendMessage(ply, sb.ToString(), Color.Yellow);
break; break;
} }
if ((j + 1) % 5 == 0) if ((j + 1) % 5 == 0)
{ {
Tools.SendMessage(ply, tempstring.TrimEnd(new[] { ' ', ',' }), Color.Yellow); Tools.SendMessage(ply, sb.ToString(), Color.Yellow);
tempstring = ""; sb.Clear();
} }
} }
} }
if (cmdlist.Count > (15 * page)) if (cmdlist.Count > (15 * page))
{ {
Tools.SendMessage(ply, "Type /help " + (page + 1) + " for more commands.", Color.Yellow); Tools.SendMessage(ply, string.Format("Type /help {0} for more commands.", (page + 1)), Color.Yellow);
} }
} }
@ -925,35 +920,35 @@ namespace TShockAPI
Main.dayTime = true; Main.dayTime = true;
NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY); NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY);
NetMessage.syncPlayers(); NetMessage.syncPlayers();
Tools.Broadcast(Tools.FindPlayer(args.PlayerID) + " set time to day."); Tools.Broadcast(string.Format("{0} set time to day.", Tools.FindPlayer(args.PlayerID)));
break; break;
case "night": case "night":
Main.time = 0; Main.time = 0;
Main.dayTime = false; Main.dayTime = false;
NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY); NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY);
NetMessage.syncPlayers(); NetMessage.syncPlayers();
Tools.Broadcast(Tools.FindPlayer(args.PlayerID) + " set time to night."); Tools.Broadcast(string.Format("{0} set time to night.", Tools.FindPlayer(args.PlayerID)));
break; break;
case "dusk": case "dusk":
Main.dayTime = false; Main.dayTime = false;
Main.time = 0.0; Main.time = 0.0;
NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY); NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY);
NetMessage.syncPlayers(); NetMessage.syncPlayers();
Tools.Broadcast(Tools.FindPlayer(args.PlayerID) + " set time to dusk."); Tools.Broadcast(string.Format("{0} set time to dusk.", Tools.FindPlayer(args.PlayerID)));
break; break;
case "noon": case "noon":
Main.dayTime = true; Main.dayTime = true;
Main.time = 27000.0; Main.time = 27000.0;
NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY); NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY);
NetMessage.syncPlayers(); NetMessage.syncPlayers();
Tools.Broadcast(Tools.FindPlayer(args.PlayerID) + " set time to noon."); Tools.Broadcast(string.Format("{0} set time to noon.", Tools.FindPlayer(args.PlayerID)));
break; break;
case "midnight": case "midnight":
Main.dayTime = false; Main.dayTime = false;
Main.time = 16200.0; Main.time = 16200.0;
NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY); NetMessage.SendData(18, -1, -1, "", 0, 0, Main.sunModY, Main.moonModY);
NetMessage.syncPlayers(); NetMessage.syncPlayers();
Tools.Broadcast(Tools.FindPlayer(args.PlayerID) + " set time to midnight."); Tools.Broadcast(string.Format("{0} set time to midnight.", Tools.FindPlayer(args.PlayerID)));
break; break;
default: default:
Tools.SendMessage(adminplr, "Invalid syntax! Proper syntax: /time <day/night/dusk/noon/midnight>", Color.Red); Tools.SendMessage(adminplr, "Invalid syntax! Proper syntax: /time <day/night/dusk/noon/midnight>", Color.Red);
@ -979,8 +974,8 @@ namespace TShockAPI
Tools.SendMessage(adminplr, "More than one player matched!", Color.Red); Tools.SendMessage(adminplr, "More than one player matched!", Color.Red);
else else
{ {
Tools.SendMessage(adminplr, "You just killed " + Tools.FindPlayer(player) + "!"); Tools.SendMessage(adminplr, string.Format("You just killed {0}!", Tools.FindPlayer(player)));
Tools.SendMessage(player, Tools.FindPlayer(adminplr) + " just killed you!"); Tools.SendMessage(player, string.Format("{0} just killed you!", Tools.FindPlayer(adminplr)));
TShock.KillMe(player); TShock.KillMe(player);
} }
} }
@ -1014,8 +1009,8 @@ namespace TShockAPI
int.TryParse(args.Parameters[1], out damage); int.TryParse(args.Parameters[1], out damage);
} }
NetMessage.SendData(26, -1, -1, "", player, ((new Random()).Next(-1, 1)), damage, (float)0); NetMessage.SendData(26, -1, -1, "", player, ((new Random()).Next(-1, 1)), damage, (float)0);
Tools.Broadcast(Tools.FindPlayer(adminplr) + " slapped " + Tools.FindPlayer(player) + " for " + Tools.Broadcast(string.Format("{0} slapped {1} for {2} damage.",
damage + " damage."); Tools.FindPlayer(adminplr), Tools.FindPlayer(player), damage));
} }
} }
@ -1023,7 +1018,7 @@ namespace TShockAPI
{ {
ConfigurationManager.spawnProtect = (ConfigurationManager.spawnProtect == false); ConfigurationManager.spawnProtect = (ConfigurationManager.spawnProtect == false);
Tools.SendMessage(args.PlayerID, Tools.SendMessage(args.PlayerID,
"Spawn is now " + (ConfigurationManager.spawnProtect ? "protected" : "open") + "."); string.Format("Spawn is now {0}.", (ConfigurationManager.spawnProtect ? "protected" : "open")));
} }
public static void UpdateNow(CommandArgs args) public static void UpdateNow(CommandArgs args)

32
TShockAPI/TShock.cs
View file

@ -134,7 +134,7 @@ namespace TShockAPI
{ {
Console.WriteLine(ex.ToString()); Console.WriteLine(ex.ToString());
} }
string version = "TShock Version " + Version + " (" + VersionCodename + ") now running."; string version = string.Format("TShock Version {0} ({1}) now running.", Version, VersionCodename);
Console.WriteLine(version); Console.WriteLine(version);
Log.Initialize(FileTools.SaveDir + "log.txt", LogLevel.All, false); Log.Initialize(FileTools.SaveDir + "log.txt", LogLevel.All, false);
Log.Info(version); Log.Info(version);
@ -243,7 +243,7 @@ namespace TShockAPI
var ban = Bans.GetBanByName(Main.player[e.Msg.whoAmI].name); var ban = Bans.GetBanByName(Main.player[e.Msg.whoAmI].name);
if (ban != null) if (ban != null)
{ {
Tools.ForceKick(e.Msg.whoAmI, "You are banned: " + ban.Reason); Tools.ForceKick(e.Msg.whoAmI, string.Format("You are banned: {0}", ban.Reason));
return true; return true;
} }
string name = Encoding.ASCII.GetString(e.Msg.readBuffer, e.Index + 23, (e.Length - (e.Index + 23)) + e.Index - 1); string name = Encoding.ASCII.GetString(e.Msg.readBuffer, e.Index + 23, (e.Length - (e.Index + 23)) + e.Index - 1);
@ -480,14 +480,14 @@ namespace TShockAPI
if (lava && !players[e.Msg.whoAmI].group.HasPermission("canlava")) if (lava && !players[e.Msg.whoAmI].group.HasPermission("canlava"))
{ {
Tools.SendMessage(e.Msg.whoAmI, "You do not have permission to use lava", Color.Red); Tools.SendMessage(e.Msg.whoAmI, "You do not have permission to use lava", Color.Red);
Tools.SendLogs(Main.player[e.Msg.whoAmI].name + " tried using lava", Color.Red); Tools.SendLogs(string.Format("{0} tried using lava", Main.player[e.Msg.whoAmI].name), Color.Red);
return true; return true;
} }
if (!lava && !players[e.Msg.whoAmI].group.HasPermission("canwater")) if (!lava && !players[e.Msg.whoAmI].group.HasPermission("canwater"))
{ {
Tools.SendMessage(e.Msg.whoAmI, "You do not have permission to use water", Color.Red); Tools.SendMessage(e.Msg.whoAmI, "You do not have permission to use water", Color.Red);
Tools.SendLogs(Main.player[e.Msg.whoAmI].name + " tried using water", Color.Red); Tools.SendLogs(string.Format("{0} tried using water", Main.player[e.Msg.whoAmI].name), Color.Red);
return true; return true;
} }
@ -541,7 +541,7 @@ namespace TShockAPI
tilex, tiley, tilex, tiley,
Main.tile[tilex, tiley].type Main.tile[tilex, tiley].type
)); ));
Tools.ForceKick(e.Msg.whoAmI, "Tile Kill abuse (" + Main.tile[tilex, tiley].type + ")"); Tools.ForceKick(e.Msg.whoAmI, string.Format("Tile Kill abuse ({0})", Main.tile[tilex, tiley].type));
return true; return true;
} }
return false; return false;
@ -552,7 +552,8 @@ namespace TShockAPI
if (Main.netMode != 2) if (Main.netMode != 2)
return; return;
Log.Info(Tools.FindPlayer(who) + " (" + Tools.GetPlayerIP(who) + ") from '" + players[who].group.GetName() + "' group joined."); Log.Info(string.Format("{0} ({1}) from '{2}' group joined.", Tools.FindPlayer(who), Tools.GetPlayerIP(who), players[who].group.GetName()));
Tools.ShowMOTD(who); Tools.ShowMOTD(who);
if (HackedHealth(who)) if (HackedHealth(who))
{ {
@ -612,12 +613,12 @@ namespace TShockAPI
{ {
if (!cmd.CanRun(players[ply])) if (!cmd.CanRun(players[ply]))
{ {
Tools.SendLogs(Tools.FindPlayer(ply) + " tried to execute " + cmd.Name(), Color.Red); Tools.SendLogs(string.Format("{0} tried to execute {1}", Tools.FindPlayer(ply), cmd.Name()), Color.Red);
Tools.SendMessage(ply, "You do not have access to that command.", Color.Red); Tools.SendMessage(ply, "You do not have access to that command.", Color.Red);
} }
else else
{ {
Tools.SendLogs(Tools.FindPlayer(ply) + " executed: /" + text, Color.Red); Tools.SendLogs(string.Format("{0} executed: /{1}", Tools.FindPlayer(ply), text), Color.Red);
cmd.Run(text, players[ply], args); cmd.Run(text, players[ply], args);
} }
} }
@ -647,7 +648,7 @@ namespace TShockAPI
var ban = Bans.GetBanByIp(ip); var ban = Bans.GetBanByIp(ip);
if (ban != null) if (ban != null)
{ {
Tools.ForceKick(ply, "You are banned: " + ban.Reason); Tools.ForceKick(ply, string.Format("You are banned: {0}", ban.Reason));
handler.Handled = true; handler.Handled = true;
} }
else if (!FileTools.OnWhitelist(ip)) else if (!FileTools.OnWhitelist(ip))
@ -763,23 +764,22 @@ namespace TShockAPI
switch (random) switch (random)
{ {
case 0: case 0:
Tools.Broadcast("You call that a lot? " + ConfigurationManager.killCount + " goblins killed!"); Tools.Broadcast(string.Format("You call that a lot? {0} goblins killed!", ConfigurationManager.killCount));
break; break;
case 1: case 1:
Tools.Broadcast("Fatality! " + ConfigurationManager.killCount + " goblins killed!"); Tools.Broadcast(string.Format("Fatality! {0} goblins killed!", ConfigurationManager.killCount));
break; break;
case 2: case 2:
Tools.Broadcast("Number of 'noobs' killed to date: " + ConfigurationManager.killCount); Tools.Broadcast(string.Format("Number of 'noobs' killed to date: {0}", ConfigurationManager.killCount));
break; break;
case 3: case 3:
Tools.Broadcast("Duke Nukem would be proud. " + ConfigurationManager.killCount + Tools.Broadcast(string.Format("Duke Nukem would be proud. {0} goblins killed.", ConfigurationManager.killCount));
" goblins killed.");
break; break;
case 4: case 4:
Tools.Broadcast("You call that a lot? " + ConfigurationManager.killCount + " goblins killed!"); Tools.Broadcast(string.Format("You call that a lot? {0} goblins killed!", ConfigurationManager.killCount));
break; break;
case 5: case 5:
Tools.Broadcast(ConfigurationManager.killCount + " copies of Call of Duty smashed."); Tools.Broadcast(string.Format("{0} copies of Call of Duty smashed.", ConfigurationManager.killCount));
break; break;
} }
} }

36
TShockAPI/Tools.cs
View file

@ -1,6 +1,7 @@
using System; using System;
using System.Collections.Generic; using System.Collections.Generic;
using System.IO; using System.IO;
using System.Text;
using Microsoft.Xna.Framework; using Microsoft.Xna.Framework;
using Terraria; using Terraria;
@ -36,22 +37,19 @@ namespace TShockAPI
/// <returns>String of players seperated by commas.</returns> /// <returns>String of players seperated by commas.</returns>
public static string GetPlayers() public static string GetPlayers()
{ {
string str = ""; var sb = new StringBuilder();
for (int i = 0; i < Main.maxPlayers; i++) for (int i = 0; i < Main.maxPlayers; i++)
{ {
if (Main.player[i].active) if (Main.player[i].active)
{ {
if (str == "") if (sb.Length != 0)
{ {
str = str + Main.player[i].name; sb.Append(", ");
}
else
{
str = str + ", " + Main.player[i].name;
} }
sb.Append(Main.player[i].name);
} }
} }
return str; return sb.ToString();
} }
/// <summary> /// <summary>
@ -83,7 +81,7 @@ namespace TShockAPI
{ {
SendMessage(i, msg); SendMessage(i, msg);
} }
Log.Info("Broadcast: " + msg); Log.Info(string.Format("Broadcast: {0}", msg));
} }
public static void Broadcast(string msg, float red, float green, float blue) public static void Broadcast(string msg, float red, float green, float blue)
@ -92,7 +90,7 @@ namespace TShockAPI
{ {
SendMessage(i, msg, red, green, blue); SendMessage(i, msg, red, green, blue);
} }
Log.Info("Broadcast: " + msg); Log.Info(string.Format("Broadcast: {0}", msg));
} }
/// <summary> /// <summary>
@ -241,7 +239,7 @@ namespace TShockAPI
{ {
string ip = GetPlayerIP(ply); string ip = GetPlayerIP(ply);
NetMessage.SendData(0x2, ply, -1, reason, 0x0, 0f, 0f, 0f); NetMessage.SendData(0x2, ply, -1, reason, 0x0, 0f, 0f, 0f);
Log.Info(ip + " was force kicked for : " + reason); Log.Info(string.Format("{0} was force kicked for : {1}", ip, reason));
} }
/// <summary> /// <summary>
@ -256,12 +254,12 @@ namespace TShockAPI
if (!TShock.players[ply].group.HasPermission("immunetokick")) if (!TShock.players[ply].group.HasPermission("immunetokick"))
{ {
string playerName = Main.player[ply].name; string playerName = Main.player[ply].name;
NetMessage.SendData(0x2, ply, -1, "Kicked: " + reason, 0x0, 0f, 0f, 0f); NetMessage.SendData(0x2, ply, -1, string.Format("Kicked: {0}", reason), 0x0, 0f, 0f, 0f);
Log.Info("Kicked " + playerName + " for : " + reason); Log.Info(string.Format("Kicked {0} for : {1}", playerName, reason));
if (adminUserName.Length == 0) if (adminUserName.Length == 0)
Broadcast(playerName + " was kicked for " + reason.ToLower()); Broadcast(string.Format("{0} was kicked for {1}", playerName, reason.ToLower()));
else else
Tools.Broadcast(adminUserName + " kicked " + playerName + " for " + reason.ToLower()); Tools.Broadcast(string.Format("{0} kicked {1} for {2}", adminUserName, playerName, reason.ToLower()));
return true; return true;
} }
return false; return false;
@ -281,12 +279,12 @@ namespace TShockAPI
string ip = GetPlayerIP(plr); string ip = GetPlayerIP(plr);
string playerName = Main.player[plr].name; string playerName = Main.player[plr].name;
TShock.Bans.AddBan(ip, playerName, reason); TShock.Bans.AddBan(ip, playerName, reason);
NetMessage.SendData(0x2, plr, -1, "Banned: " + reason, 0x0, 0f, 0f, 0f); NetMessage.SendData(0x2, plr, -1, string.Format("Banned: {0}", reason), 0x0, 0f, 0f, 0f);
Log.Info("Banned " + playerName + " for : " + reason); Log.Info(string.Format("Banned {0} for : {1}", playerName, reason));
if (adminUserName.Length == 0) if (adminUserName.Length == 0)
Broadcast(playerName + " was banned for " + reason.ToLower()); Broadcast(string.Format("{0} was banned for {1}", playerName, reason.ToLower()));
else else
Tools.Broadcast(adminUserName + " banned " + playerName + " for " + reason.ToLower()); Tools.Broadcast(string.Format("{0} banned {1} for {2}", adminUserName, playerName, reason.ToLower()));
return true; return true;
} }
return false; return false;