Merge pull request 'Fixed permission check' (#397) from dev into master
All checks were successful
Deploy dev on push / on-push-deploy_sh-edraft (push) Successful in 3m20s

Reviewed-on: sh-edraft.de/kd_discord_bot#397
This commit is contained in:
Sven Heidemann 2023-10-02 17:31:24 +02:00
commit 7d5733fa5c

View File

@ -47,11 +47,15 @@ class PermissionService(PermissionServiceABC):
return self._team_role_members(g_id, TeamMemberTypeEnum.moderator) return self._team_role_members(g_id, TeamMemberTypeEnum.moderator)
def is_member_admin(self, member: discord.Member) -> bool: def is_member_admin(self, member: discord.Member) -> bool:
return member in self.get_admins(member.guild.id) return member is not None and member in self.get_admins(member.guild.id)
def is_member_moderator(self, member: discord.Member) -> bool: def is_member_moderator(self, member: discord.Member) -> bool:
return member in self.get_admins(member.guild.id) or member in self.get_moderators(member.guild.id) return (
member is not None
and member in self.get_admins(member.guild.id)
or member in self.get_moderators(member.guild.id)
)
def is_member_technician(self, member: discord.Member) -> bool: def is_member_technician(self, member: discord.Member) -> bool:
config = self._technician_configs.get_technician_config() config = self._technician_configs.get_technician_config()
return str(member.id) in config.technician_ids.select(lambda x: str(x)) return member is not None and str(member.id) in config.technician_ids.select(lambda x: str(x))