Don't send message about DMing ACL when command's called from DM (#4188)

* Only send a message telling the user the file will be send via DM if the original message didn't come from a DM

* Remove unused import for AsyncIter

* Changed channel instance comparison to discord.DMChannel

Co-authored-by: douglas-cpp <douglasc.dev@gmail.com>
This commit is contained in:
Douglas 2020-08-17 10:22:38 -03:00 committed by GitHub
parent 929fd04613
commit 306de3a64c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -10,7 +10,6 @@ from schema import And, Or, Schema, SchemaError, Optional as UseOptional
from redbot.core import checks, commands, config from redbot.core import checks, commands, config
from redbot.core.bot import Red from redbot.core.bot import Red
from redbot.core.i18n import Translator, cog_i18n from redbot.core.i18n import Translator, cog_i18n
from redbot.core.utils import AsyncIter
from redbot.core.utils.chat_formatting import box from redbot.core.utils.chat_formatting import box
from redbot.core.utils.menus import start_adding_reactions from redbot.core.utils.menus import start_adding_reactions
from redbot.core.utils.predicates import ReactionPredicate, MessagePredicate from redbot.core.utils.predicates import ReactionPredicate, MessagePredicate
@ -333,7 +332,8 @@ class Permissions(commands.Cog):
except discord.Forbidden: except discord.Forbidden:
await ctx.send(_("I'm not allowed to DM you.")) await ctx.send(_("I'm not allowed to DM you."))
else: else:
await ctx.send(_("I've just sent the file to you via DM.")) if not isinstance(ctx.channel, discord.DMChannel):
await ctx.send(_("I've just sent the file to you via DM."))
finally: finally:
file.close() file.close()