From 506a6c96c9d7336f87be6ad46d654958fe6249e9 Mon Sep 17 00:00:00 2001
From: Christopher F <13098994+foxbot@users.noreply.github.com>
Date: Thu, 17 Aug 2017 01:59:58 -0400
Subject: [PATCH] Throw when attempting to add or remove a member's
 EveryoneRole (#781)

* Throw when attempting to add or remove a member's EveryoneRole

This resolves #780

* Removed braces
---
 src/Discord.Net.Core/Utils/Preconditions.cs  | 8 ++++++++
 src/Discord.Net.Rest/DiscordRestApiClient.cs | 4 ++++
 2 files changed, 12 insertions(+)

diff --git a/src/Discord.Net.Core/Utils/Preconditions.cs b/src/Discord.Net.Core/Utils/Preconditions.cs
index 705a152491..bec8de9dce 100644
--- a/src/Discord.Net.Core/Utils/Preconditions.cs
+++ b/src/Discord.Net.Core/Utils/Preconditions.cs
@@ -192,5 +192,13 @@ public static void YoungerThanTwoWeeks(ulong[] collection, string name)
                     throw new ArgumentOutOfRangeException(name, "Messages must be younger than two weeks old.");
             }
         }
+        public static void NotEveryoneRole(ulong[] roles, ulong guildId, string name)
+        {
+            for (var i = 0; i < roles.Length; i++)
+            {
+                if (roles[i] == guildId)
+                    throw new ArgumentException($"The everyone role cannot be assigned to a user", name);
+            }
+        }
     }
 }
diff --git a/src/Discord.Net.Rest/DiscordRestApiClient.cs b/src/Discord.Net.Rest/DiscordRestApiClient.cs
index 78e768ccdb..a6c42782a6 100644
--- a/src/Discord.Net.Rest/DiscordRestApiClient.cs
+++ b/src/Discord.Net.Rest/DiscordRestApiClient.cs
@@ -392,6 +392,7 @@ public async Task AddRoleAsync(ulong guildId, ulong userId, ulong roleId, Reques
             Preconditions.NotEqual(guildId, 0, nameof(guildId));
             Preconditions.NotEqual(userId, 0, nameof(userId));
             Preconditions.NotEqual(roleId, 0, nameof(roleId));
+            Preconditions.NotEqual(roleId, guildId, nameof(roleId), "The Everyone role cannot be added to a user.");
             options = RequestOptions.CreateOrClone(options);
 
             var ids = new BucketIds(guildId: guildId);
@@ -402,6 +403,7 @@ public async Task RemoveRoleAsync(ulong guildId, ulong userId, ulong roleId, Req
             Preconditions.NotEqual(guildId, 0, nameof(guildId));
             Preconditions.NotEqual(userId, 0, nameof(userId));
             Preconditions.NotEqual(roleId, 0, nameof(roleId));
+            Preconditions.NotEqual(roleId, guildId, nameof(roleId), "The Everyone role cannot be removed from a user.");
             options = RequestOptions.CreateOrClone(options);
 
             var ids = new BucketIds(guildId: guildId);
@@ -1000,6 +1002,8 @@ public async Task ModifyGuildMemberAsync(ulong guildId, ulong userId, Rest.Modif
 
             bool isCurrentUser = userId == CurrentUserId;
 
+            if (args.RoleIds.IsSpecified)
+                Preconditions.NotEveryoneRole(args.RoleIds.Value, guildId, nameof(args.RoleIds));
             if (isCurrentUser && args.Nickname.IsSpecified)
             {
                 var nickArgs = new Rest.ModifyCurrentUserNickParams(args.Nickname.Value ?? "");