Dotclear

Ticket #903 (closed enhancement: fixed)

Opened 14 years ago

Last modified 11 years ago

Ajouter le bouton "Supprimer tous les spams" dans la page des commentaires

Reported by: Jean-Michel Owned by: Ergaut
Priority: normal Milestone: 2.5
Component: refonte:ergo Version:
Severity: normal Keywords: spam
Cc:

Description

Ne serait-ce pas possible de créer un lien/bouton "Supprimer tous les spams maintenant" comme dans la messagerie Gmail ?

Pourquoi cette demande? Depuis quelques jours, je me retrouve (et je ne crois pas être le seul) avec -/+ 600 spams (retroliens) par jour. Un peu comme à la belle époque de Dotclear 1.

Alors bien sûr, si on se (veut) pose deux minutes, on peut déjà prévoir de filtrer par X le nombres de "commentaires indésirables" afficher dans la page et selectionner l'action "supprimer".

La création de ce lien/bouton a un effet positif, ça permet de vider rapidement les milliers de spams en un clic.

Effet négatif, ça n'oblige pas l'utilisateur a vérifié un commentaire réel qui se serait retrouver dans le paquet d'indésirables. En même temps, un simple avertissement avant de valider l'action pourrait régler l'affaire.

Change History

comment:1 Changed 14 years ago by franck

Ce bouton existe déjà, en tout cas pour le blog actif.

Il suffit d'aller dans l'extension Anti-Spam (haut de la page des filtres)

comment:2 Changed 14 years ago by franck

  • Status changed from new to closed
  • Resolution set to invalid

comment:3 Changed 14 years ago by kozlika

  • Status changed from closed to reopened
  • Resolution invalid deleted
  • Component changed from module:blog to ergo

Bah oui mais il faudrait en effet qu'il soit également présent sur la liste des comm (et c'est prévu dans le chantier ergo).

comment:4 Changed 14 years ago by philippe

  • Milestone changed from 2.2 to 2.3

comment:5 Changed 14 years ago by kozlika

  • Status changed from reopened to new
  • Component changed from module:ergo to refonte:ergo
  • Summary changed from Créer un bouton "Supprimer tous les antispam" to Ajouter le bouton "Supprimer tous les spams" dans la page des commentaires
  • Version 2.1 deleted
  • Milestone changed from 2.3 to Refonte admin
  • Owner changed from xave to Ergaut

comment:6 Changed 14 years ago by franck

  • Type changed from idea to enhancement

comment:7 follow-up: ↓ 11 Changed 11 years ago by franck

J'hésite entre fournir un behaviour pour permettre à un plugin antispam d'offrir le formulaire qui permettrait la suppression (ou autre) et le formulaire directement créé et géré par le core. Un avis ?

comment:8 Changed 11 years ago by franck

  • Milestone changed from Refonte admin to 2.5

comment:9 Changed 11 years ago by franck

De plus il faudrait ajouter une demande de confirmation avant la suppression de tous les indésirables dans le plugin Antispam.

comment:10 Changed 11 years ago by JcDenis

(In [c39391d78a5d]) Add confirm dialog on spam deletion, addresses #903

comment:11 in reply to: ↑ 7 Changed 11 years ago by JcDenis

Replying to franck:

J'hésite entre fournir un behaviour pour permettre à un plugin antispam d'offrir le formulaire qui permettrait la suppression (ou autre) et le formulaire directement créé et géré par le core. Un avis ?

Pas d'hésitation, en ce moment on essai de sortir les plugins du core donc il faut un behavior qui va bien et du coup sortir également la phrase "Vous avez x spams, les afficher" de la page des commentaires pour la mettre dans un behavior de antispam

comment:12 Changed 11 years ago by franck

Sortir tous les plugins du core, c'est un des objectifs de la branche sexy, et pour l'instant la gestion/notion du spam est encore bien intégrée au noyau*, donc on va simplement faire en sorte que ça fonctionne (la suppression de tous les spams sur cette page) pour la prochaine version.

(* Je pense entre autre au statut "indésirable" des commentaires et rétroliens qui est partie intégrante de la liste des statuts gérés nativement. Une réflexion un peu plus poussée est nécessaire à ce sujet.)

comment:13 Changed 11 years ago by JcDenis

  • Status changed from new to closed
  • Resolution set to fixed

(In [1d17674d50ad]) Fix #903 : Added action "Delete all spam" to comments page.

comment:14 Changed 11 years ago by JcDenis

Sur ordre du chef j'ai fait une solution ne séparant pas antispam du core ;-)

comment:15 Changed 11 years ago by franck

Je me demande s'il ne vaudrait pas mieux ne pas s'appuyer sur le plugin antispam pour la suppression. Le statut -2 (junk) est géré par le core, il ne devrait (normalement) pas s'appuyer sur un plugin qui peut être désactivé pour supprimer les commentaires qui ont ce statut.

comment:16 Changed 11 years ago by JcDenis

Oui mais dans comments_actions.php c'est un peu plus compliqué vu qu'on utilise pas de comment_id... et du coup ça sera encore plus le bazar... Ici j'ai ajouté le test sur l'existence du plugin antispam pour le bouton "supprimer tous les spam"
Dans tous les cas pour l'instant on est bancale avec ce plugins, je propose ici une solution fonctionnelle transitoire ;-)

comment:17 Changed 11 years ago by franck

Ok on laisse comme ça et on verra ce que ça donnera dans la sexy à venir.

comment:18 Changed 11 years ago by franck <carnet.franck.paul@…>

(In [1f59a06fbf6c]) Locales (addresses #903)

comment:19 Changed 11 years ago by franck <carnet.franck.paul@…>

(In [d65efdccc4fb]) Direct access to blog pref only if antispam plugin is activated (addresses #903)

comment:20 Changed 11 years ago by franck

C'est génial, faut tout refaire :-)

Je viens de voir que triggerComments() n'était pas appelé par le plugin et que du coup, sur une suppression massive de spam, les compteurs deviennent incorrects.

Je vais revoir le fonctionnement de tout ça tranquillement…

comment:21 Changed 11 years ago by franck

Pas de triggerComments() nécessaire, seuls les commentaires et trackbacks en ligne sont comptés. Du coup ça va être rudement plus simple.

comment:22 Changed 11 years ago by franck <carnet.franck.paul@…>

(In [de274bbd7b80]) Cope with junk comments without antispam plugin enabled (addresses #903)

comment:23 Changed 11 years ago by franck <carnet.franck.paul@…>

(In [142da2b40d5d]) Avoid using get param to transport result as it may interfere with pager (addresses #903)

Note: See TracTickets for help on using tickets.

Sites map