search

zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Permet de verrouiller les commentaires d'un contenu #6598

Open Situphen opened 7 months ago

Situphen commented 7 months ago

Fixes #6563

Une partie de la fonctionnalité est déjà codée (content.is_locked sur la même base que topic.is_locked) mais il manquait quelques bouts que j'ai rajouté pour avoir quelque chose de fonctionnel. Je me suis inspiré de bouts de code existants donc j'espère que c'est quand même assez propre au final. Je veux bien vos retours sur le code si possible !

Je met l'étiquette Bloquant car c'est une fonctionnalité assez demandée par les modérateurs vu tout le spam que l'on a dans les commentaires.

coveralls commented 7 months ago

Coverage Status

coverage: 88.645% (-0.05%) from 88.699% when pulling 7e8ee01255b9d59c5de9758e5c8a6810f1862df8 on Situphen:locked-comments into 6e0d801152dd36927a2d3a94dae1dcefd67cc833 on zestedesavoir:dev.

philippemilink commented 7 months ago

J'ai parcouru le code rapidement, ça me semble bien. J'ai juste une remarque : renommer is_locked en is_content_reaction_locked (ou quelque chose d'autre) pour que ce soit explicite que c'est les commentaires du contenu qu'on souhaite verrouiller (on pourrait imaginer dans le futur qu'on puisse verrouiller un contenu, quelque soit le sens qu'on mette derrière).

Et il manque les tests :)

Arnaud-D commented 7 months ago

"content_reaction" n'est pas forcément un très bon nom non plus (ce sont des commentaires plus que des réactions). Après, c'est peut-être hors périmètre de cette PR de changer ça.

philippemilink commented 7 months ago

"content_reaction" n'est pas forcément un très bon nom non plus (ce sont des commentaires plus que des réactions).

Je suis bien d'accord, mais il me semble que c'est comme ça que sont nommés les commentaires de contenu dans le code.

Après, c'est peut-être hors périmètre de cette PR de changer ça.

Bien d'accord.

Arnaud-D commented 7 months ago

Oui, si jamais on changeait ce nom, ça serait partout. ^^

Situphen commented 7 months ago

@philippemilink J'ai bien en tête le renommage et les tests, mais il manque d'autres choses. Je vous en parlerai lors de la prochaine réunion mais j'ai ouvert une boîte de pandore car les commentaires c'est un méli-mélo du module des contenus et de celui du forum, donc il y a un peu de refacto en perspective si on veut bien faire les choses.