Verwarnungssystem #35 #235

Merged
edraft merged 7 commits from #35 into 1.0.0 2023-02-24 08:49:09 +01:00
Owner

#35

#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
@ -0,0 +16,4 @@
logger: DatabaseLogger,
db_context: DatabaseContextABC,
users: UserRepositoryABC,
servers: ServerRepositoryABC,
Contributor

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
@ -344,0 +371,4 @@
title=member.name, description=self._t.transform("modules.base.user.atr.warnings"), color=int("ef9d0d", 16)
)
warnings_id_string = ""
Contributor

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
@ -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"))
Contributor

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
@ -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"))
Contributor

Auch hier wird zuerst eine Nachricht versendet, bevor die eine Aktion stattfindet. Auch hier die beiden Zeilen tauschen.

Auch hier wird zuerst eine Nachricht versendet, bevor die eine Aktion stattfindet. Auch hier die beiden Zeilen tauschen.
edraft marked this conversation as resolved
edraft added 1 commit 2023-02-23 08:46:53 +01:00
edraft added 1 commit 2023-02-23 08:58:08 +01:00
edraft requested review from Ebola-Chan 2023-02-23 08:58:47 +01:00
Ebola-Chan approved these changes 2023-02-23 23:35:19 +01:00
edraft merged commit a588f8bf72 into 1.0.0 2023-02-24 08:49:09 +01:00
edraft deleted branch #35 2023-02-24 08:49:09 +01:00
Sign in to join this conversation.
No description provided.