diff --git a/src/communication/models.py b/src/communication/models.py index 1b48701..c8b2e23 100644 --- a/src/communication/models.py +++ b/src/communication/models.py @@ -13,28 +13,6 @@ class Message(models.Model): is_read (bool): Indique si le message a été lu ou non - par défaut: False message_title (str): Titre ou sujet du message. message_body (text): Contenu du message. - - Remarks: - . Les champs `message_body` et `message_title` me posent problème, - parce qu'on est déjà dans la classe `Message`. Pourquoi pas juste `title` et `body` ? - - . Le champ `is_read` est superflu. Si on veut savoir si le message est lu, il suffit de - faire une méthode avec un décorateur @property et retourner si `date_of_reading` est vide. - - . Par convention, j'aurais appelé les champs `writer` et `reader` plutôt `sender` et `recipient`. - - . Pareil pour les champs "date" auto-remplis. J'aime bien les conventions type `_at`.() - (Ici, `written_at` et `read_at`). - - . Est-ce qu'un message ne peut pas être envoyé à plusieurs personnes en même temps ? - - . Tes related_names ne sont pas corrects: `have_write` et `have_read` n'ont pas de signification. - Quand on a une instance d'un utilisateur en main, ça reviendrait à faire `user_instance.have_read.all`. - Sémantiquement, c'est pas top. Il faudrait plutôt les appeler `sent_messages` et `received_messages`. - Dans le code, cela donnerait `user_instance.sent_messages.all()` ou `user_instance.sent_messages.filter(...)`. - - . Est-ce qu'un `Message` ne devrait pas être `Markdownizable` ? - Mais alors le champ `information` devrait s'appeler `content` ou quelque chose de plus générique. """ writer = models.ForeignKey( diff --git a/src/communication/urls.py b/src/communication/urls.py index 0b440e3..f6e00b3 100644 --- a/src/communication/urls.py +++ b/src/communication/urls.py @@ -5,11 +5,6 @@ from django.urls import path, re_path from . import views -""" NOTE: Attention à bien nommer les URLs. `message_sent` -> `sent_messages`... - -""" - - message_urlpatterns = [ path(r"sent/", views.get_message_sent, name="message_sent"), path(r"received/", views.get_message_received, name="message_received"),