Browse Source

Merge pull request #710 from FiniteReality/issue/704

Change wording of permission preconditions
tags/1.0
Christopher F GitHub 8 years ago
parent
commit
d44d5e7198
2 changed files with 7 additions and 5 deletions
  1. +5
    -3
      src/Discord.Net.Commands/Attributes/Preconditions/RequireBotPermissionAttribute.cs
  2. +2
    -2
      src/Discord.Net.Commands/Attributes/Preconditions/RequireUserPermissionAttribute.cs

+ 5
- 3
src/Discord.Net.Commands/Attributes/Preconditions/RequireBotPermissionAttribute.cs View File

@@ -44,14 +44,16 @@ namespace Discord.Commands


public override async Task<PreconditionResult> CheckPermissions(ICommandContext context, CommandInfo command, IServiceProvider services) public override async Task<PreconditionResult> CheckPermissions(ICommandContext context, CommandInfo command, IServiceProvider services)
{ {
var guildUser = await context.Guild.GetCurrentUserAsync();
IGuildUser guildUser = null;
if (context.Guild != null)
guildUser = await context.Guild.GetCurrentUserAsync().ConfigureAwait(false);


if (GuildPermission.HasValue) if (GuildPermission.HasValue)
{ {
if (guildUser == null) if (guildUser == null)
return PreconditionResult.FromError("Command must be used in a guild channel"); return PreconditionResult.FromError("Command must be used in a guild channel");
if (!guildUser.GuildPermissions.Has(GuildPermission.Value)) if (!guildUser.GuildPermissions.Has(GuildPermission.Value))
return PreconditionResult.FromError($"Command requires guild permission {GuildPermission.Value}");
return PreconditionResult.FromError($"Bot requires guild permission {GuildPermission.Value}");
} }


if (ChannelPermission.HasValue) if (ChannelPermission.HasValue)
@@ -65,7 +67,7 @@ namespace Discord.Commands
perms = ChannelPermissions.All(guildChannel); perms = ChannelPermissions.All(guildChannel);


if (!perms.Has(ChannelPermission.Value)) if (!perms.Has(ChannelPermission.Value))
return PreconditionResult.FromError($"Command requires channel permission {ChannelPermission.Value}");
return PreconditionResult.FromError($"Bot requires channel permission {ChannelPermission.Value}");
} }


return PreconditionResult.FromSuccess(); return PreconditionResult.FromSuccess();


+ 2
- 2
src/Discord.Net.Commands/Attributes/Preconditions/RequireUserPermissionAttribute.cs View File

@@ -52,7 +52,7 @@ namespace Discord.Commands
if (guildUser == null) if (guildUser == null)
return Task.FromResult(PreconditionResult.FromError("Command must be used in a guild channel")); return Task.FromResult(PreconditionResult.FromError("Command must be used in a guild channel"));
if (!guildUser.GuildPermissions.Has(GuildPermission.Value)) if (!guildUser.GuildPermissions.Has(GuildPermission.Value))
return Task.FromResult(PreconditionResult.FromError($"Command requires guild permission {GuildPermission.Value}"));
return Task.FromResult(PreconditionResult.FromError($"User requires guild permission {GuildPermission.Value}"));
} }


if (ChannelPermission.HasValue) if (ChannelPermission.HasValue)
@@ -66,7 +66,7 @@ namespace Discord.Commands
perms = ChannelPermissions.All(guildChannel); perms = ChannelPermissions.All(guildChannel);


if (!perms.Has(ChannelPermission.Value)) if (!perms.Has(ChannelPermission.Value))
return Task.FromResult(PreconditionResult.FromError($"Command requires channel permission {ChannelPermission.Value}"));
return Task.FromResult(PreconditionResult.FromError($"User requires channel permission {ChannelPermission.Value}"));
} }


return Task.FromResult(PreconditionResult.FromSuccess()); return Task.FromResult(PreconditionResult.FromSuccess());


Loading…
Cancel
Save