[Communication] Revoir les related names #18

New Issue

Sulley · 2020-10-26T10:21:52+01:00

Sulley commented

2020-10-26 10:21:52 +01:00

Les 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(...).

Les 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(...)`.

Sulley added the

enhancement

label 2020-10-26 10:25:02 +01:00

Fred added this to the Revue de l'application `communication` milestone 2020-10-28 09:27:47 +01:00

Sulley commented

2020-11-01 08:33:02 +01:00

Yo !

dans mes vues, j'ai déjà deux fonctions dans mes vues qui s'appellent get_sent_messages et get_received_message. J'ai peur qu'en utilisant sent_messages et receveid_messages cela porte à confusion.

Quoi t'en penses ?

Yo ! dans mes vues, j'ai déjà deux fonctions dans mes vues qui s'appellent `get_sent_messages` et `get_received_message`. J'ai peur qu'en utilisant `sent_messages` et `receveid_messages` cela porte à confusion. Quoi t'en penses ?

Fred was assigned by Sulley

2020-11-01 08:33:20 +01:00

Sulley added a new dependency 2020-11-01 09:05:02 +01:00

#45 [Communication] Refactoring `get_messages()`

Fred commented

2020-11-01 14:57:45 +01:00

A priori, ce ne devrait pas porter à confusion dans la mesure où la fonction get_sent_messages est différente de request.user.sent_messages.

Le contexte est différent; les objets sur lesquels les fonctions et méthodes s'appliquent aussi.

(ou j'aurais mal compris ta question ?)

A priori, ce ne devrait pas porter à confusion dans la mesure où la fonction `get_sent_messages` est différente de `request.user.sent_messages`. Le contexte est différent; les objets sur lesquels les fonctions et méthodes s'appliquent aussi. (ou j'aurais mal compris ta question ?)

Sulley referenced this issue from a commit

2020-11-01 15:46:27 +01:00

Close #18

Sulley closed this issue

2020-11-01 15:46:27 +01:00

Sign in to join this conversation.