Merge pull request #6 from versx/multi-channels

Add support for multiple channel ids to accept commands from

Author: versx

config.example.json

@@ -4,7 +4,7 @@
   "token": "",
   "guildId": 0000000001,
   "ownerId": 0000000001,
-  "channelId": 0000000001,
+  "channelIds": [0000000001],
   "commandPrefix": "!",
   "requiredRoles": [0000000001]
 }

src/Bot.cs

@@ -26,7 +26,7 @@ public class Bot
 
         public Bot(Config config)
         {
-            _logger.Trace($"WhConfig [OwnerId={config.OwnerId}, GuildId={config.GuildId}, ChannelId={config.ChannelId}]");
+            _logger.Trace($"WhConfig [OwnerId={config.OwnerId}, GuildId={config.GuildId}, ChannelIds={string.Join(", ", config.ChannelIds)}]");
             _config = config;
 
             AppDomain.CurrentDomain.UnhandledException += async (sender, e) =>

src/Commands/PhoneControl.cs

@@ -410,7 +410,7 @@ private bool HasRequiredRoles(DiscordMember member)
         private bool IsValidChannel(ulong channelId)
         {
             //If no channel id is specified allow the command to execute in all channels, otherwise only the channel specified.
-            return _dep.Config.ChannelId == 0 || _dep.Config.ChannelId == channelId;
+            return _dep.Config.ChannelIds.Count == 0 || _dep.Config.ChannelIds.Contains(channelId);
         }
 
         #endregion

src/Configuration/Config.cs

@@ -26,8 +26,8 @@ public class Config
         [JsonProperty("ownerId")]
         public ulong OwnerId { get; set; }
 
-        [JsonProperty("channelId")]
-        public ulong ChannelId { get; set; }
+        [JsonProperty("channelIds")]
+        public List<ulong> ChannelIds { get; set; }
 
         [JsonProperty("token")]
         public string Token { get; set; }
@@ -44,6 +44,7 @@ public class Config
 
         public Config()
         {
+            ChannelIds = new List<ulong>();
             Host = "*";
             Port = 6542;
             RequiredRoles = new List<ulong>();