Dotclear

Ticket #1401 (closed defect: fixed)

Opened 11 years ago

Last modified 11 years ago

Pas de recalcul si suppression commentaire

Reported by: Mirovinben Owned by: sogox
Priority: normal Milestone: 2.5.3
Component: module:core Version: 2.5
Severity: normal Keywords:
Cc:

Description

Comptage erroné après suppression d'un commentaire

Le commentaire reste comptabilisé à tort

Change History

comment:1 Changed 11 years ago by philippe

Pareil chez moi côté public. Pas de souci côté admin.

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

  • Milestone changed from A definir to 2.5.2

comment:3 in reply to: ↑ 2 Changed 11 years ago by sogox

 https://bitbucket.org/sogos/dotclear/commits/b11414cd4735f942b7dc8ee1e2bba0c55eea7953

Ce commit corrige chez moi le problème pour les commentaires, mais pas contre, il y a une subtilité que je n'ai pas peut être pas saisi concernant le comment_trackback que je n'ai pas encore géré.

A tester donc (et si quelqu'un peut m'expliquer le fonctionnement du trackback ca serait top)

comment:4 Changed 11 years ago by Evelf

  • Owner changed from xave to Team

comment:5 Changed 11 years ago by franck

  • Owner changed from Team to sogox

comment:6 Changed 11 years ago by franck

sogox tu peux faire un PR en tenant compte des remarques de JcDenis? sur les commits correspondants sur ton fork Bitbucket steuplé ?

Version 0, edited 11 years ago by franck (next)

comment:7 Changed 11 years ago by Denis Jean-Christian <contact@…>

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

(In [ec1e7bef6c1f]) Fix post comments number on comments deletion, fixes #1401, thanks sogos/sogox for patch and tests

comment:8 Changed 11 years ago by franck

  • Status changed from closed to reopened
  • Resolution fixed deleted
  • Milestone changed from 2.5.2 to 2.5.3

Je ne sais pas si c'est lié mais la suppression d'un unique commentaire sélectionné dans la liste génère deux notices :

Notice: Undefined offset: 0 in /Users/franck/Documents/EGO/Geekeries/Internet/htdocs/hg-dc2-2.5/inc/core/class.dc.blog.php on line 264

Notice: Undefined offset: 0 in /Users/franck/Documents/EGO/Geekeries/Internet/htdocs/hg-dc2-2.5/inc/core/class.dc.blog.php on line 275

comment:9 Changed 11 years ago by JcDenis

Je n'arrive pas à lever d'erreur chez moi (blog de test peut-être pas assez rempli) mais peux tu tenter de modifier dans les deux lignes citées : ça :

$a_tbs[$a_key]

par :

 !empty($a_tbs) && $a_tbs[$a_key]

j'ai pas poussé plus loin mais le premier tableau des trackbacks peut-être vide si on fourni la lise des commentaires à traiter.

comment:11 Changed 11 years ago by Denis Jean-Chirstian <contact@…>

(In [9ab4292a9cb0]) Refix (or not) comments counts on trigger comments, addresses #1401

comment:10 Changed 11 years ago by JcDenis

Bon j'ai modifié ce qui a été fait pour enlever les notices, j'ai fait quelques essais sur des commentaires, ça a l'air de fonctionner mais...

Mais je pense qu'on a (j'ai) cassé le comptage lors de la modification du traitement par lot des commentaires, qui permettait de diminuer le nombre de requêtes en base. A mon avis il faut complétement reprendre cette partie et voir si on ne peut pas faire le recomptage directement en SQL lors des requêtes de modifications de commentaires.

Last edited 11 years ago by JcDenis (previous) (diff)

comment:11 Changed 11 years ago by franck

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

La méthode la plus simple est finalement la plus solide. r1570 Testée et approuvée \o/

Note: See TracTickets for help on using tickets.

Sites map