Merge pull request '0.3 - /user info für alle (#119)' (#122) from #119 into 0.3

Reviewed-on: sh-edraft.de/kd_discord_bot#122
Reviewed-by: Ebola-Chan <nick.jungmann@gmail.com>
Closes #119

kdb-bot/src/modules/base/command/user_group.py

@@ -62,10 +62,14 @@ class UserGroup(DiscordCommandABC):
     @user.command()
     @commands.guild_only()
     @CommandChecks.check_is_ready()
-    @CommandChecks.check_is_member_moderator()
     async def info(self, ctx: Context, member: Optional[discord.Member] = None, *, wait: int = None):
         self._logger.debug(__name__, f'Received command user-info {ctx}:{member},{wait}')
 
+        is_mod = self._permissions.is_member_moderator(ctx.author)
+        if member is not None and not is_mod:
+            await self._message_service.send_ctx_msg(ctx, self._t.transform('common.no_permission_message'))
+            return
+
         if member is None or not isinstance(member, discord.Member):
             member = ctx.author
 
@@ -94,20 +98,26 @@ class UserGroup(DiscordCommandABC):
             roles += f'{role.name}\n'
         embed.add_field(name=self._t.transform('modules.base.user_info.fields.roles'), value=roles, inline=False)
 
-        joins_string = ''
+        if is_mod or member == ctx.author:
-        for join in joins:
+            joins_string = ''
-            joins_string += f'{self._date.transform(join.joined_on)}\n'
+            for join in joins:
-        embed.add_field(name=self._t.transform('modules.base.user_info.fields.joins'), value=joins_string)
+                joins_string += f'{self._date.transform(join.joined_on)}\n'
+            embed.add_field(name=self._t.transform('modules.base.user_info.fields.joins'), value=joins_string)
 
-        lefts_string = ''
+        if is_mod or member == ctx.author:
-        for join in joins:
+            lefts_string = ''
-            if join.leaved_on is None:
+            for join in joins:
-                if lefts_string == '':
+                if join.leaved_on is None:
-                    lefts_string = '/'
+                    if lefts_string == '':
-                continue
+                        lefts_string = '/'
-            lefts_string += f'{self._date.transform(join.leaved_on)}\n'
+                    continue
-        embed.add_field(name=self._t.transform('modules.base.user_info.fields.lefts'), value=lefts_string)
+                lefts_string += f'{self._date.transform(join.leaved_on)}\n'
-        embed.add_field(name=self._t.transform('modules.base.user_info.fields.warnings'), value=self._t.transform('common.not_implemented_yet'), inline=False)
 
-        await self._message_service.send_ctx_msg(ctx, embed, wait_before_delete=wait)
+            embed.add_field(name=self._t.transform('modules.base.user_info.fields.lefts'), value=lefts_string)
+
+        if is_mod or member == ctx.author:
+            embed.add_field(name=self._t.transform('modules.base.user_info.fields.warnings'), value=self._t.transform('common.not_implemented_yet'), inline=False)
+
+        # send to interaction because of sensitive data
+        await self._message_service.send_interaction_msg(ctx.interaction, embed, wait_before_delete=wait)
         self._logger.trace(__name__, f'Finished user-info command')

kdb-bot/src/modules/permission/service/permission_service.py

@@ -128,8 +128,7 @@ class PermissionService(PermissionServiceABC):
         return member.guild.id in self._admins and member in self._admins[member.guild.id]
 
     def is_member_moderator(self, member: discord.Member) -> bool:
-        return member.guild.id in self._moderators \
+        return member.guild.id in self._moderators and member in self._moderators[member.guild.id] or self.is_member_admin(member)
-               and member in self._moderators[member.guild.id] or self.is_member_admin(member)
 
     def is_member_technician(self, member: discord.Member) -> bool:
         return member in self._technicians