From 5386f24b693bfe982cb7ccb4ced7e7e1e2d1791c Mon Sep 17 00:00:00 2001 From: Ske Date: Wed, 22 Apr 2020 09:27:55 +0200 Subject: [PATCH] Fix permissions check when proxying --- PluralKit.Bot/Services/ProxyService.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PluralKit.Bot/Services/ProxyService.cs b/PluralKit.Bot/Services/ProxyService.cs index a4e39f8a..87470a6d 100644 --- a/PluralKit.Bot/Services/ProxyService.cs +++ b/PluralKit.Bot/Services/ProxyService.cs @@ -227,8 +227,8 @@ namespace PluralKit.Bot var permissions = guildUser.GetPermissions(channel); // If we can't send messages at all, just bail immediately. - // TODO: can you have ManageMessages and *not* SendMessages? What happens then? - if (!permissions.SendMessages && !permissions.ManageMessages) return false; + // 2020-04-22: Manage Messages does *not* override a lack of Send Messages. + if (!permissions.SendMessages) return false; if (!permissions.ManageWebhooks) {