Project

General

Profile

Bug #101515

Snapshot: count_recent_changes() ne compte pas la même chose que ce que renvoie get_recent_changes()

Added by Benjamin Dauvergne 11 days ago. Updated 4 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
31 January 2025
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

count() compte les lignes de la table snapshots regroupé par object_type, object_id alors que get_recent_changes() renvoie ces lignes en contrôlant l'existence effective de ces objets.

PS: et donc conséquence on a des pages vides dans la pagination sur /backoffice/studio/all-changes/ voir ticket lié.

À voir s'il faut corriger ces méthodes, ou prévoir du nettoyage de la table snapshot pour retirer les lignes référençant des objets supprimés (qui est peut-être une scorie de la migration elle même et pas un souci de manque de trigger pour le nettoyage des snapshots).

D'un rapide coup d’œil je ne vois pas de méthode clean_snapshots() ou de trigger pour nettoyer les snapshots d'objets disparues, et donc c'est peut-être voulu et c'est le comptage qu'il faudrait corriger.

>>> len(Snapshot.get_recent_changes(object_types=['datasource'], limit=1000))
65
>>> len(Snapshot._get_recent_changes(object_types=['datasource'], limit=10000))
1348

Associated revisions

Revision 90ade811 (diff)
Added by Frédéric Péters 4 days ago

misc: keep track of deleted objects in snapshots (#101515)

History

#1

Updated by Benjamin Dauvergne 11 days ago

  • Tracker changed from Développement to Bug
#3

Updated by Benjamin Dauvergne 11 days ago

  • Description updated (diff)
#4

Updated by Benjamin Dauvergne 11 days ago

  • Description updated (diff)
#5

Updated by Benjamin Dauvergne 10 days ago

  • Description updated (diff)
#6

Updated by Robot Gitea 8 days ago

  • Status changed from Nouveau to En cours
  • Assignee set to Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#7

Updated by Robot Gitea 7 days ago

  • Status changed from En cours to Solution proposée
#8

Updated by Robot Gitea 6 days ago

  • Status changed from Solution proposée to Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#9

Updated by Robot Gitea 4 days ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#10

Updated by Transition automatique 4 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF