Optimisation des traitements dans posts_actions et comments_actions

[bruno]

Le traitement "par lot" des commentaires ou des billets dans l'administration n'est pas optimal. Par exemple, pour supprimer certains billets cochés dans la page posts.php, dans $POST[] contient la liste des IDs des billets, posts_actions.php déroule l'algorithme :

• Récupération des billets concernés : "SELECT [...] from dc_post where post_id in ([liste]) and [contraintes]"
• Boucle sur le $rs sortant, et exécution de $core->blog->delPost sur chaque id

donc autant de requêtes qu'il y a de billets cochés.

Si on envisage l'évolution #849, ou tout simplement si on veut supprimer beaucoup de billets on risque de vite tomber en timeout avec de traitement

Un raccourci serait de faire un $core->blog->delPosts($params), qui ferait un DELETE FROM dc_post WHERE post_id in ([liste]) and [contraintes]

[Tomtom33]

Ça me parait être une très bonne idée. Pourquoi à la base, cela a été codé comme ça? Pour vérifier qu'un ID existe bien?

[JcDenis]

La plupart des actions d'effacement d'enregistrement se font une par une, c'est la même chose dans beaucoup de classes, une des principales raisons est de bien identifier ce que l'on souhaite définitivement effacer. Maintenant si tu veux faire un delPosts() il faut alors le blinder soit par de js de confirmation avant soit par de nombreuses limitations. (if type+id+...)

[bruno]

JdDenis?: Pas sûr de comprendre : faire un SELECT [] from dc_post where [conditions] suivi d'une itération sur tous les résultats et d'un effacement 1 à 1 est strictement identique à un DELETE from dc_post where [même conditions]. Dans l'implémentation actuelle, il n'y a pas de demande de confirmation entre le select et les delete...

[JcDenis]

Replying to bruno: Hum, aussi oui, dans ce cas la j'aurai mieux fait de me taire.

[xave]

Ué, ça paraît bien. À qui pourrait-on assigner ce ticket ? Ah, au même que le #849, tiens. :)

[Tomtom33]

Pareil. Du coup, ce sera plus simple avec cette structure pour gérer le #849 ;)

[Tomtom33]

Bon, comme je l'ai dit dans mon premier commentaire, je suis pour implémenter ça.

[xave]

Ça va être nécessaire au moment où on va vraiment changer le mode de fonctionnement des listes, non ?

[JcDenis]

(In [ad4942bddbdb]) Enhance mass posts actions by reducing SQL queries, addresses #943

[JcDenis]

Et c'est encore pire que ce que je croyais, en effet pour chaque commentaire modifié, la fonction triggerComment() ajoute encore 3 requêtes...

[JcDenis]

(In [ffd8fd14003c]) Enhance mass comments actions by reducing SQL queries, addresses #943

[JcDenis]

(In [40fe2ee5bdbd]) Enhance mass use of triggerComment() by reducing SQL queries, addresses #943