commands.has_permissions() and DMs

[tender steeple]

A slash command decorated with @commands.has_permissions() or @commands.bot_has_permissions() will raise

AttributeError: 'PartialMessageable' object has no attribute 'permissions_for' when it is invoked via DMs.

Does anyone know a nice way to conditionally apply these decorators so they only apply for commands invoked in a guild and not in DMs?

[true yarrow]

@guild_only() i believe

[halcyon escarp]

that function is only available on guild as it have channel check permission that cant be used on dms. Avoid using that if you want to use that command outside a guild

[tender steeple]

thank you, that makes sense. in this case should I be using this instead (if the source of the interaction is a guild)?

https://docs.pycord.dev/en/master/api.html#discord.abc.GuildChannel.permissions_for

i assume it works self referentially for the bot user itself

[true yarrow]

just slap on this decorator

its a check like the one in the title

[tender steeple]

I do use that decorator in combination with the decorators in the OP and still get the AttributeError

[true yarrow]

hmm

whats your pycord version?

[tender steeple]

@commands.guild_only()

2.0.0b7

[true yarrow]

try updating to master

or reinstall

[tender steeple]

okay, ill give that a go

this seems related: https://github.com/Pycord-Development/pycord/issues/497

[tender steeple]

ty this worked along with moving @commands.guild_only() closest to async def so it runs first before the permissions checks

last change I need to make is fixing this for when I want a permissions check in guild and not in DM for commands which are not guild only

[true yarrow]

oh yeah, i did forget to mention that decorator order matters

[tender steeple]

yeah no worries i figured that out, it used to work in earlier pycord unordered for some reason

[tender steeple]

to solve this for non guild only commands i used a check with predicate

async def check_view_read(ctx):
    '''
    Check and return if the bot has the View Channels
    and Read Message History channel permissions
    '''
    if isinstance(ctx.channel, PartialMessageable):
        return True

    permissions = ctx.channel.permissions_for(ctx.guild.get_member(ctx.bot.user.id))

    return permissions.view_channel and permissions.read_message_history

instead of @commands.bot_has_permissions()