Skip to content

Commit

Permalink
[commands] Change default Bot.owner_ids to consider team roles
Browse files Browse the repository at this point in the history
  • Loading branch information
Rapptz committed Aug 31, 2023
1 parent f74eb14 commit e414d0f
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions discord/ext/commands/bot.py
Original file line number Diff line number Diff line change
Expand Up @@ -499,6 +499,12 @@ async def is_owner(self, user: User, /) -> bool:
``user`` parameter is now positional-only.
.. versionchanged:: 2.4
This function now respects the team member roles if the bot is team-owned.
In order to be considered an owner, they must be either an admin or
a developer.
Parameters
-----------
user: :class:`.abc.User`
Expand All @@ -516,9 +522,13 @@ async def is_owner(self, user: User, /) -> bool:
return user.id in self.owner_ids
else:

app = await self.application_info() # type: ignore
app: discord.AppInfo = await self.application_info() # type: ignore
if app.team:
self.owner_ids = ids = {m.id for m in app.team.members}
self.owner_ids = ids = {
m.id
for m in app.team.members
if m.role in (discord.TeamMemberRole.admin, discord.TeamMemberRole.developer)
}
return user.id in ids
else:
self.owner_id = owner_id = app.owner.id
Expand Down

0 comments on commit e414d0f

Please sign in to comment.