Verwarnungssystem #35 #235

edraft · 2023-02-22T08:44:54+01:00

edraft commented

2023-02-22 08:44:54 +01:00

#35

edraft self-assigned this 2023-02-22 08:44:54 +01:00

edraft added 5 commits 2023-02-22 08:44:55 +01:00

edraft added this to the Sprint 1.0.0 project 2023-02-22 08:44:59 +01:00

edraft requested review from Ebola-Chan 2023-02-22 08:45:04 +01:00

Ebola-Chan requested changes 2023-02-22 22:26:33 +01:00

kdb-bot/src/bot_data/service/user_warnings_repository_service.py Outdated

						
				@ -0,0 +16,4 @@

				        logger: DatabaseLogger,

				        db_context: DatabaseContextABC,

				        users: UserRepositoryABC,

				        servers: ServerRepositoryABC,

Ebola-Chan commented

2023-02-22 21:19:25 +01:00

Du übergibst servers: ServerRepositoryABC aber dies wird in der Definition von des Constructors nirgendswo zugewiesen.
Ich sehe auch nicht dass self._servers in der Klasse irgendwo verwendet wird. Somit könnte der Parameter weg.

Du übergibst ```servers: ServerRepositoryABC``` aber dies wird in der Definition von des Constructors nirgendswo zugewiesen. Ich sehe auch nicht dass ```self._servers``` in der Klasse irgendwo verwendet wird. Somit könnte der Parameter weg.

edraft marked this conversation as resolved

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

						
				@ -344,0 +371,4 @@

				            title=member.name, description=self._t.transform("modules.base.user.atr.warnings"), color=int("ef9d0d", 16)

				        )

				        warnings_id_string = ""

Ebola-Chan commented

2023-02-22 22:07:43 +01:00

Evtl. die Zugriffe auf die Datenbank hier verringern, in dem das Ergebnis in einer Variable zwischengespeichert wird.

Von:

warnings_id_string = ""
for warning in self._user_warnings.get_user_warnings_by_user_id(user.id):
    warnings_id_string += f"{warning.id}\n"

warnings_description_string = ""
for warning in self._user_warnings.get_user_warnings_by_user_id(user.id):
    warnings_description_string += f"{warning.description}\n"

Zu:

warnings = self._user_warnings.get_user_warnings_by_user_id(user.id)

warnings_id_string = ""
for warning in warnings:
    warnings_id_string += f"{warning.id}\n"

warnings_description_string = ""
for warning in warnings:
    warnings_description_string += f"{warning.description}\n"

Auch wenn unwahrscheinlich, aber ich meine dass dadurch auch eine zwischenzeitliche Änderung in der Datenbank keine Auswirkung auf die Formatierung des Embeds hat.

Evtl. die Zugriffe auf die Datenbank hier verringern, in dem das Ergebnis in einer Variable zwischengespeichert wird. Von: ```python warnings_id_string = "" for warning in self._user_warnings.get_user_warnings_by_user_id(user.id): warnings_id_string += f"{warning.id}\n" warnings_description_string = "" for warning in self._user_warnings.get_user_warnings_by_user_id(user.id): warnings_description_string += f"{warning.description}\n" ``` Zu: ```python warnings = self._user_warnings.get_user_warnings_by_user_id(user.id) warnings_id_string = "" for warning in warnings: warnings_id_string += f"{warning.id}\n" warnings_description_string = "" for warning in warnings: warnings_description_string += f"{warning.description}\n" ``` Auch wenn unwahrscheinlich, aber ich meine dass dadurch auch eine zwischenzeitliche Änderung in der Datenbank keine Auswirkung auf die Formatierung des Embeds hat.

edraft marked this conversation as resolved

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

						
				@ -344,0 +399,4 @@

				    async def add(self, ctx: Context, member: discord.Member, description: str):

				        self._logger.debug(__name__, f"Received command user warning add {ctx}:{member},{description}")

				        try:

				            await self._message_service.send_ctx_msg(ctx, self._t.transform("modules.base.warnings.add.success"))

Ebola-Chan commented

2023-02-22 22:10:45 +01:00

Hier wird zuerst eine Message gesendet bevor versucht wird eine Warnung hinzuzufügen.
Bitte die beiden Zeilen tauschen:

await self._message_service.send_ctx_msg(ctx, self._t.transform("modules.base.warnings.add.success"))
await self._user_warnings_service.add_warnings(member, description, ctx.author.id)

Hier wird zuerst eine Message gesendet bevor versucht wird eine Warnung hinzuzufügen. Bitte die beiden Zeilen tauschen: ```python await self._message_service.send_ctx_msg(ctx, self._t.transform("modules.base.warnings.add.success")) await self._user_warnings_service.add_warnings(member, description, ctx.author.id) ```

edraft marked this conversation as resolved

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

						
				@ -344,0 +413,4 @@

				    async def remove(self, ctx: Context, warning_id: int):

				        self._logger.debug(__name__, f"Received command user warning remove {ctx}:{warning_id}")

				        try:

				            await self._message_service.send_ctx_msg(ctx, self._t.transform("modules.base.warnings.remove.success"))