Ticket #1401 (closed defect: fixed)
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:3 in reply to: ↑ 2 Changed 10 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:7 Changed 10 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 10 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 10 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 10 years ago by Denis Jean-Chirstian <contact@…>
(In [9ab4292a9cb0]) Refix (or not) comments counts on trigger comments, addresses #1401
comment:10 Changed 10 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.
comment:11 Changed 10 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/
Pareil chez moi côté public. Pas de souci côté admin.