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

/user add now returns sensible errors

Browse source
This commit is contained in:
Lucas Nicodemus 2015-02-21 17:34:39 -07:00
parent d3ed04c947
commit 964115dbb5
1 changed files with 20 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

26
TShockAPI/Commands.cs
View file

@ -910,29 +910,37 @@ namespace TShockAPI
{ {
var user = new User(); var user = new User();
try
{
if (args.Parameters.Count == 4) if (args.Parameters.Count == 4)
{ {
user.Name = args.Parameters[1]; user.Name = args.Parameters[1];
user.Password = args.Parameters[2]; user.Password = args.Parameters[2];
user.Group = args.Parameters[3]; user.Group = args.Parameters[3];
args.Player.SendSuccessMessage("Account " + user.Name + " has been added to group " + user.Group + "!"); try
{
TShock.Users.AddUser(user); TShock.Users.AddUser(user);
TShock.CharacterDB.SeedInitialData(TShock.Users.GetUser(user)); TShock.CharacterDB.SeedInitialData(TShock.Users.GetUser(user));
args.Player.SendSuccessMessage("Account " + user.Name + " has been added to group " + user.Group + "!");
Log.ConsoleInfo(args.Player.Name + " added Account " + user.Name + " to group " + user.Group); Log.ConsoleInfo(args.Player.Name + " added Account " + user.Name + " to group " + user.Group);
} }
catch (GroupNotExistsException e)
{
args.Player.SendErrorMessage("Group " + user.Group + " does not exist!");
}
catch (UserExistsException e)
{
args.Player.SendErrorMessage("User " + user.Name + " already exists!");
}
catch (UserManagerException e)
{
args.Player.SendErrorMessage("User " + user.Name + " could not be added, check console for details.");
Log.ConsoleError(e.ToString());
}
}
else else
{ {
args.Player.SendErrorMessage("Invalid syntax. Try /user help."); args.Player.SendErrorMessage("Invalid syntax. Try /user help.");
} }
}
catch (UserManagerException ex)
{
args.Player.SendErrorMessage(ex.Message);
Log.ConsoleError(ex.ToString());
}
} }
// User deletion requires a username // User deletion requires a username
else if (subcmd == "del" && args.Parameters.Count == 2) else if (subcmd == "del" && args.Parameters.Count == 2)