Compare commits

..

3 Commits

Author SHA1 Message Date
7d5733fa5c 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
2023-10-02 17:31:24 +02:00
5c87a7a919 Merge branch 'master' into dev
All checks were successful
Deploy dev on push / on-push-deploy_sh-edraft (push) Successful in 3m33s
2023-10-02 17:31:17 +02:00
682c0034b0 Fixed permission check
All checks were successful
Deploy dev on push / on-push-deploy_sh-edraft (push) Successful in 4m3s
2023-10-02 17:25:22 +02:00

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))