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

/user group has proper error handling

Browse source 
This blast of commits fixes #543
This commit is contained in:
Lucas Nicodemus 2015-02-21 17:50:19 -07:00
parent 70104f795c
commit deb4113c73
1 changed files with 16 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

24
TShockAPI/Commands.cs
View file

@ -1000,23 +1000,31 @@ namespace TShockAPI
var user = new User(); var user = new User();
user.Name = args.Parameters[1]; user.Name = args.Parameters[1];
try if (args.Parameters.Count == 3)
{ {
if (args.Parameters.Count == 3) try
{ {
args.Player.SendSuccessMessage("Account " + user.Name + " has been changed to group " + args.Parameters[2] + "!");
TShock.Users.SetUserGroup(user, args.Parameters[2]); TShock.Users.SetUserGroup(user, args.Parameters[2]);
Log.ConsoleInfo(args.Player.Name + " changed account " + user.Name + " to group " + args.Parameters[2] + "."); Log.ConsoleInfo(args.Player.Name + " changed account " + user.Name + " to group " + args.Parameters[2] + ".");
args.Player.SendSuccessMessage("Account " + user.Name + " has been changed to group " + args.Parameters[2] + "!");
} }
else catch (GroupNotExistsException e)
{ {
args.Player.SendErrorMessage("Invalid user group syntax. Try /user help."); args.Player.SendErrorMessage("That group does not exist!");
} }
catch (UserNotExistException e)
{
args.Player.SendErrorMessage("User " + user.Name + " does not exist!");
}
catch (UserManagerException e)
{
args.Player.SendErrorMessage("User " + user.Name + " could not be added. Check console for details.");
}
} }
catch (UserManagerException ex) else
{ {
args.Player.SendMessage(ex.Message, Color.Green); args.Player.SendErrorMessage("Invalid user group syntax. Try /user help.");
Log.ConsoleError(ex.ToString());
} }
} }
else if (subcmd == "help") else if (subcmd == "help")