Skip to content
This repository has been archived by the owner on Jul 7, 2024. It is now read-only.

La suppression de messages (peut-être que lors de la suppression d'un utilisateur) ne supprime pas les commentaires #204

Open
rastapopougros opened this issue Mar 28, 2019 · 5 comments
Labels

Comments

@rastapopougros
Copy link

Pour mémoire. Même si je pense toujours qu'on devrait le moins possible supprimer des messages mais les rendre anonyme (cf ma proposition dans un autre ticket).

J'ai trouvé par l'intermédiaire des petites flèches qui s'ajoutent après l'URL sous le champ d'édition, un commentaire qui contenait une URL que j'écrivais. Ces flèches ne pointent jamais vers le seen racine d'une discussion mais directement vers la page du seen même quand c'est un commentaire (les commentaires sont des seens aussi). Je me fais pas comprendre ?
https://seenthis.net/messages/412977

C'est un bug, sûrement, mais au moins ça permet de retrouver des contenus cachés CAR… le seen racine de ce commentaire a lui disparu, soit supprimé par la personne soit suite au script de suppression de la personne. Ça on le voit en cliquant sur la date du commentaire, qui là contient l'URL du seen racine avec l'ancre vers le commentaire :
https://seenthis.net/messages/412967#message412977

On a donc sûrement plein de seens-commentaires dans la base qui ne sont plus rattachés à rien, qui sont invisibles sauf par les petites flèches d'URL sous l'édition uniquement.

Pour l'instant je suis d'avis de ne pas les supprimer, si jamais on trouve encore un moyen de les utiliser (par exemple arriver à recréer des fils de discussions avec à la racine la mention "Message d'origine supprimé" et que tout ça continue de sortir dans la recherche).

@brunob
Copy link
Member

brunob commented Jul 27, 2022

Ha ben vi, on voit bien ici https://github.com/seenthis/seenthis-cli/blob/master/supprimer_auteur.php#L43 qu'on ne supprime que les messages et le texte de ceux postés par l'auteur cible.

On a donc sûrement plein de seens-commentaires dans la base qui ne sont plus rattachés à rien, qui sont invisibles sauf par les petites flèches d'URL sous l'édition uniquement.

Ça craint.

Pour l'instant je suis d'avis de ne pas les supprimer, si jamais on trouve encore un moyen de les utiliser (par exemple arriver à recréer des fils de discussions avec à la racine la mention "Message d'origine supprimé" et que tout ça continue de sortir dans la recherche).

Moi je suis d'avis de les supprimer, ça permettrait certainement de réduire la taille de la base.

De plus, il faudrait créer un ticket dans https://github.com/seenthis/seenthis-cli pour signaler le bug.

@brunob
Copy link
Member

brunob commented Jul 27, 2022

SELECT a.id_me FROM spip_mea LEFT JOINspip_me b ON a.id_parent = b.id_me WHERE b.id_me IS NULL AND a.id_parent > 0; renvoie 7121 rows, même si ça n'est pas énorme, il reste aussi en base le texte de tous ces messages.

Et maintenant qu'on a bloqué l'affichage direct des commentaires cf fb1bc79, on ne peut plus afficher ces messages.

@rastapopougros
Copy link
Author

rastapopougros commented Jul 27, 2022

Perso je suis toujours de cet avis :

si jamais on trouve encore un moyen de les utiliser (par exemple arriver à recréer des fils de discussions avec à la racine la mention "Message d'origine supprimé" et que tout ça continue de sortir dans la recherche)

Il n'y en a pas tant que ça, et en revanche il y a eu par le passé vraiment plein PLEIN de conversations entières qui étaient autant ou plus intéressantes/utiles que le seen de départ, avec des gros débats, des argumentations, des références etc. Et ces personnes n'ont jamais demandé à avoir leur contenu supprimé.

Donc je pense vraiment que ça devrait toujours sortir, un jour, même si on note "Message d'orgine supprimé" tout en haut.

Cela va d'ailleurs dans le sens de mon avis sur la suppression des users, qui, sauf cas de force majeur ultra rare, ne devrait jamais supprimer ses contenus, comme le propose #158 (comment)

@brunob
Copy link
Member

brunob commented Jul 27, 2022

Donc je pense vraiment que ça devrait toujours sortir, un jour, même si on note "Message d'orgine supprimé" tout en haut.

Soit, gardons les jusqu'au jour ou quelqu'un s'occupera de mettre en place ce que tu proposes :)

Cela va d'ailleurs dans le sens de mon avis

Hihi, quelle cohérence :D

@rastapopougros
Copy link
Author

Juste pour avoir au moins un exemple sous la main dans le fil, un truc récent par ex : https://seenthis.net/messages/968549
Dans ce fil, les commentaires donnant les sources scientifiques sont plus importants que le message de départ (enfin en tout cas que le lien donné dans le message de départ). Si @biggrizzly supprimait son compte et qu'on restait sur le truc de tout supprimer, il faudrait toujours que ce fil sorte en cherchant "covid + hépatite" pour avoir une argumentation du lien ou pas entre les deux. (Mais yen a littéralement des centaines d'exemples comme ça depuis 12 ans.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants