[Communication] Méthode is_read does not work #59

New Issue

Fred · 2021-05-14T21:36:38+02:00

Fred commented

2021-05-14 21:36:38 +02:00

Yop,

Dans l'application de communication, au niveau du modèle, tu as ceci:

class Message(...):
	[...]
	@property
	def is_read():
    		return read_at is not null or read_at <= datetime.today()

Mais cela ne peut pas fonctionner, parce que is_read devrait prendre le paramètre self (sans quoi, cela ne peut pas être une méthode).

Il faut aussi penser à modifier la dernière ligne, pour faire référence à self.read_at (aux deux endroits).

Aussi, c'est is not None et pas is not null 😉

Yop, Dans l'application de `communication`, au niveau du modèle, tu as ceci: ```python class Message(...): [...] @property def is_read(): return read_at is not null or read_at <= datetime.today() ``` Mais cela ne peut pas fonctionner, parce que `is_read` devrait prendre le paramètre `self` (sans quoi, cela ne peut pas être une méthode). Il faut aussi penser à modifier la dernière ligne, pour faire référence à `self.read_at` (aux deux endroits). Aussi, c'est `is not None` et pas `is not null` 😉

Fred added this to the Revue de l'application `communication` milestone 2021-05-14 21:36:38 +02:00

Fred added the

bug

label 2021-05-14 21:36:38 +02:00

Sulley was assigned by Fred

2021-05-14 21:36:38 +02:00

Fred referenced this issue from a commit

2021-05-14 21:40:13 +02:00

Review Communication app with pylint

Sulley referenced this issue from a commit

2021-05-15 16:51:53 +02:00

Close #59 and multiple improvement/bug fix for Gymnast <-> Django.User