La liste de lien "Derniers commentaires" dans le pied de page du site contient les liens qui ont été marqués comme spam.
Par exemple, on voit actuellement le commentaire "BUY … CERTIFICATE" dans cette liste. Si on clique sur ce liens, nous avons bien la page "Vous ne passerez pas".
# Erreur de traitement
Posté par Benoît Sibaud (site web personnel) . Évalué à 3 (+0/-0). Dernière modification le 28 avril 2020 à 18:42.
Un des deux commentaires n'avait pas été masqué : il restait un commentaire visible sur un contenu masqué. Donc encore accessible en accédant directement au commentaire, mais non affichage dans le contexte complet du contenu associé.
[^] # Re: Erreur de traitement
Posté par Adrien Dorsaz (site web personnel, Mastodon) . Évalué à 2 (+0/-0).
Oh, c'est très intéressant. En fait, je me disais justement que je n'arrivais pas à reproduire aujourd'hui sur ma machine.
Je pense qu'il faut quand même traiter ce cas, car la liste des "Derniers commentaires" contient des liens qui pointent vers le contexte des contenus complets.
L'idée, ça serait de permettre d'afficher dans "Les derniers commentaires" uniquement les commentaires publiés sur des journaux publiés.
Une autre idée serait de changer les liens pour afficher directement le commentaire sans le contexte du contenu.
[^] # Re: Erreur de traitement
Posté par Adrien Dorsaz (site web personnel, Mastodon) . Évalué à 2 (+0/-0).
Ce que je voulais proposer d'abord, ça correspondrait à cette correction.
Ça ajoute bien sûr une jointure SQL entre Comment et Node pour filtrer correctement les contenus visibles d'abord.
[^] # Re: Erreur de traitement
Posté par Benoît Sibaud (site web personnel) . Évalué à 3 (+0/-0).
ça marche fonctionnellement, mais ça explose en prod (mysqld à 800% de cpu).
[^] # Re: Erreur de traitement
Posté par Bruno Michel (site web personnel) . Évalué à 3 (+0/-0). Dernière modification le 31 janvier 2021 à 23:00.
Ça va mieux avec https://github.com/linuxfrorg/linuxfr.org/commit/27561a03eecfb121c010132027140127bd06e35a
[^] # Re: Erreur de traitement
Posté par Benoît Sibaud (site web personnel) . Évalué à 3 (+0/-0). Dernière modification le 01 février 2021 à 07:29.
Nettement mieux oui :). Merci.
cf https://linuxfr.org/images/load/load.png
[^] # Re: Erreur de traitement
Posté par Adrien Dorsaz (site web personnel, Mastodon) . Évalué à 2 (+0/-0).
Hello,
Je me suis permis de rouvrir l'entrée du suivi, parce que je ne suis pas sûr que mes commentaires plus hauts ont été lu/compris :)
J'essaie de réexpliquer pourquoi je pense que ce suivi n'est pas invalide: il se trouve que les liens du bloque "Derniers commentaires" pointent sur le commentaire dans le contexte du contenu.
Ce qui a pour conséquence que, si le dernier commentaire a été publié sur un contenu retiré, alors le lien dans "Derniers commentaires" renvoie la page "Vous ne passerez pas" (car le contenu ne peut pas être affiché).
Pour résoudre ce problème, j'ai proposé la correction suivante sur github: si le contenu est masqué, les commentaires n'apparaissent pas dans le bloque "Derniers commentaires".
Ça c'est une première solution que j'aime bien, parce que si le contenu entier a été retiré, je doute que les commentaires liés soient intéressants/utiles.
Une autre solution, serait de modifier la cible des liens du bloque "Derniers commentaires" pour pointer vers le commentaire hors-contexte du contenu.
[^] # Re: Erreur de traitement
Posté par Benoît Sibaud (site web personnel) . Évalué à 4 (+0/-0).
Je suis d'accord que ça doit être corrigé, que l'entrée devait être réouverte et avec ta première solution
Envoyer un commentaire
Suivre le flux des commentaires
Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.