Browse Source

Added some ModuleService null checks

pull/11/head
RogueException 9 years ago
parent
commit
abc24cde5c
1 changed files with 3 additions and 0 deletions
  1. +3
    -0
      src/Discord.Net.Modules/ModuleService.cs

+ 3
- 0
src/Discord.Net.Modules/ModuleService.cs View File

@@ -24,6 +24,8 @@ namespace Discord.Modules


public void Install(IModule module, string name, FilterType type) public void Install(IModule module, string name, FilterType type)
{ {
if (module == null) throw new ArgumentNullException(nameof(module));
if (name == null) throw new ArgumentNullException(nameof(name));
if (_client == null) throw new InvalidOperationException("Service needs to be added to a DiscordClient before modules can be installed."); if (_client == null) throw new InvalidOperationException("Service needs to be added to a DiscordClient before modules can be installed.");
if (_modules.ContainsKey(module)) throw new InvalidOperationException("This module has already been added."); if (_modules.ContainsKey(module)) throw new InvalidOperationException("This module has already been added.");


@@ -35,6 +37,7 @@ namespace Discord.Modules
public ModuleManager GetManager(IModule module) public ModuleManager GetManager(IModule module)
{ {
if (module == null) throw new ArgumentNullException(nameof(module)); if (module == null) throw new ArgumentNullException(nameof(module));

ModuleManager result = null; ModuleManager result = null;
_modules.TryGetValue(module, out result); _modules.TryGetValue(module, out result);
return result; return result;


Loading…
Cancel
Save