-
Notifications
You must be signed in to change notification settings - Fork 161
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Un message modifié par son auteurice puis modéré est présenté comme modifié par le membre l’ayant modéré #6575
Comments
J'ai regardé un peu dans le code et c'est géré à deux endroits (ce qui n'est pas optimal mais c'est un autre sujet). Pour les messages du forum : Lines 102 to 115 in 9b93b43
Pour les commentaires : Lines 535 to 545 in 9b93b43
On y voit en effet que c'est l'auteur de l'action de masquage qui est marqué comme "éditeur" du message dans la base de données. Pour modifier cette situation, il faudrait d'une part modifier ces bouts de code mais aussi faire une passe sur tous les messages actuellement en base de données pour corriger éventuellement les "éditeurs" des messages. Un gros chantier ! Alternativement, on pourrait ne plus stocker cette notion en base de données et la calculer à la volée, mais niveau performance ce ne serait pas idéal. Bref, je ne pense pas que ce sera fait d'aussitôt ! |
Merci pour l'investigation ! J'aurais tendance à penser que le plus propre serait d'avoir :
Par contre, ce n'est vraiment pas prioritaire vu que le bug est très peu gênant en pratique — ce serait si quelqu'un est motivé·e :p Ça pourrait faire un ticket facile ! |
Il y a l'écriture de la migration à prévoir donc pas sûr qu'on puisse considérer ce ticket facile :D |
Description du bug
Lorsqu'un message est modifié par son auteur ou autrice initiale, puis modéré (masqué) par un·e membre, le texte « Modifié par… » au dessus du message indique qu'il a été modifié par le ou la modératrice. Si on clique sur le lien pour voir l’historique (auteurice ou modérateurice uniquement), on constate que ce n'est pas le cas.
On peut observer (si on a les droits de modération) un exemple réel ici.
Le bug est également présent si l’on est connecté·e comme simple membre, ou si l’on est déconnecté·e. Dans un tel cas, il n'y a aucun moyen de savoir que c'est une information erronée, car on n'a pas accès à l'historique de modification.
Comment reproduire ?
La liste des étapes qui permet de reproduire le bug :
Comportement attendu
Le message est masqué, et le ou la modératrice est correctement attribuée de ce fait ; le message est juste marqué comme “Modifié” — et non comme “Modifié par <le ou la modératrice>”.
Capture d'écran
Voici le bug reproduit en image. Tout d'abord, voici le message sus-donné comme exemple, tel que visible actuellement.
Voici ensuite, la liste des modifications si l’on clique sur le lien.
Informations techniques
The text was updated successfully, but these errors were encountered: