github/Discord.Net
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add null checks to ModuleManager.EnableServers and EnableChannels

Browse Source
This commit is contained in:
RogueException
2015-12-01 15:07:01 -04:00
parent 736e5ed268
commit de831001b5
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/Discord.Net.Modules/ModuleManager.cs
View File

@@ -132,6 +132,7 @@ namespace Discord.Modules
public void EnableServers(IEnumerable<Server> servers) public void EnableServers(IEnumerable<Server> servers)
{ {
if (servers == null) throw new ArgumentNullException(nameof(servers)); if (servers == null) throw new ArgumentNullException(nameof(servers));
if (servers.Contains(null)) throw new ArgumentException("Collection cannot contain null.", nameof(servers));
if (!_useServerWhitelist) throw new InvalidOperationException("This module is not configured to use a server whitelist."); if (!_useServerWhitelist) throw new InvalidOperationException("This module is not configured to use a server whitelist.");
lock (this) lock (this)
@@ -194,6 +195,7 @@ namespace Discord.Modules
public void EnableChannels(IEnumerable<Channel> channels) public void EnableChannels(IEnumerable<Channel> channels)
{ {
if (channels == null) throw new ArgumentNullException(nameof(channels)); if (channels == null) throw new ArgumentNullException(nameof(channels));
if (channels.Contains(null)) throw new ArgumentException("Collection cannot contain null.", nameof(channels));
if (!_useChannelWhitelist) throw new InvalidOperationException("This module is not configured to use a channel whitelist."); if (!_useChannelWhitelist) throw new InvalidOperationException("This module is not configured to use a channel whitelist.");
lock (this) lock (this)