PvE/Torch
3
0
Fork 0
Code Issues 1 Pull Requests Actions Releases 21 Wiki Activity

Catch exceptions thrown by commands

Browse Source
This commit is contained in:
John Gross
2017-07-26 00:15:31 -07:00
parent 3dba4f744f
commit 87d9825c91
4 changed files with 45 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
CHANGELOG.md Normal file
View File

@@ -0,0 +1,9 @@
# Torch 1.0.182.329
* Improved logging, logs now to go the Logs folder and aren't deleted on start
* Fixed chat tab not enabling with -autostart
* Fixed player list
* Watchdog time-out is now configurable in TorchConfig.xml
* Fixed infinario log spam
* Fixed crash when sending empty message from chat tab
* Fixed permissions on Torch commands
* Changed plugin StoragePath to the current instance path (per-instance configs)

4
Torch.Client/Properties/AssemblyInfo.cs
View File

@@ -1,4 +1,4 @@
using System.Reflection;
[assembly: AssemblyVersion("1.0.203.595")]
[assembly: AssemblyFileVersion("1.0.203.595")]
[assembly: AssemblyVersion("1.0.207.7")]
[assembly: AssemblyFileVersion("1.0.207.7")]

4
Torch.Server/Properties/AssemblyInfo.cs
View File

@@ -1,4 +1,4 @@
using System.Reflection;
[assembly: AssemblyVersion("1.1.203.596")]
[assembly: AssemblyFileVersion("1.1.203.596")]
[assembly: AssemblyVersion("1.1.207.7")]
[assembly: AssemblyFileVersion("1.1.207.7")]

52
Torch/Commands/Command.cs
View File

@@ -8,6 +8,7 @@ using NLog;
using Torch.API;
using Torch.API.Plugins;
using Torch.Commands.Permissions;
using VRage.Game;
using VRage.Game.ModAPI;
namespace Torch.Commands
@@ -26,6 +27,7 @@ namespace Torch.Commands
private readonly MethodInfo _method;
private ParameterInfo[] _parameters;
private int? _requiredParamCount;
private static readonly Logger Log = LogManager.GetLogger(nameof(Command));
public Command(ITorchPlugin plugin, MethodInfo commandMethod)
{
@@ -84,31 +86,41 @@ namespace Torch.Commands
public bool TryInvoke(CommandContext context)
{
var parameters = new object[_parameters.Length];
if (context.Args.Count < _requiredParamCount)
return false;
//Convert args from string
for (var i = 0; i < _parameters.Length && i < context.Args.Count; i++)
try
{
if (context.Args[i].TryConvert(_parameters[i].ParameterType, out object obj))
parameters[i] = obj;
else
var parameters = new object[_parameters.Length];
if (context.Args.Count < _requiredParamCount)
return false;
}
//Fill remaining parameters with default values
for (var i = 0; i < parameters.Length; i++)
//Convert args from string
for (var i = 0; i < _parameters.Length && i < context.Args.Count; i++)
{
if (context.Args[i].TryConvert(_parameters[i].ParameterType, out object obj))
parameters[i] = obj;
else
return false;
}
//Fill remaining parameters with default values
for (var i = 0; i < parameters.Length; i++)
{
if (parameters[i] == null)
parameters[i] = _parameters[i].DefaultValue;
}
var moduleInstance = (CommandModule)Activator.CreateInstance(Module);
moduleInstance.Context = context;
_method.Invoke(moduleInstance, parameters);
return true;
}
catch (Exception e)
{
if (parameters[i] == null)
parameters[i] = _parameters[i].DefaultValue;
context.Respond(e.Message, "Error", MyFontEnum.Red);
Log.Error($"Command '{SyntaxHelp}' from '{Plugin.Name ?? "Torch"}' threw an exception. Args: {string.Join(", ", context.Args)}");
Log.Error(e);
return true;
}
var moduleInstance = (CommandModule)Activator.CreateInstance(Module);
moduleInstance.Context = context;
_method.Invoke(moduleInstance, parameters);
return true;
}
}