Skip to content
This repository has been archived by the owner on Dec 18, 2024. It is now read-only.

Le rendu de l'analyse des dates de création et modification est incohérent #27

Closed
Guts opened this issue Nov 19, 2019 · 0 comments · Fixed by #28
Closed

Le rendu de l'analyse des dates de création et modification est incohérent #27

Guts opened this issue Nov 19, 2019 · 0 comments · Fixed by #28
Assignees
Labels
analisis Stats and charts bug Something isn't working
Milestone

Comments

@Guts
Copy link
Contributor

Guts commented Nov 19, 2019

Constat

Problème relevé par le CD 94.

Pour reproduire, il faut exporter un partage dans lequel des métadonnées n'ont jamais été éditées. Leur date de modification est donc la même que la date de création, ce qui crée une incohérence entre les listes à analyser.

Dans le fichier journal on a d'ailleurs ce message :

2019-11-19 14:35:02,028 || WARNING || stats - 138 || line_dates || Dates lists should have the same length. Creation: 553 | Modification: 547

Diagnostic

Le problème vient du fait qu'une date de modification n'est pas stockée dans la liste destinée à l'analyse si elle est égale à la date de création :

# add modification date (not datetime) for later stats, only if different from the creation date
if md._modified != md._created:
self.stats.li_dates_md_modified.append(
utils.hlpr_datetimes(md._modified).date()
)

C'est pour éviter de compter un événement de modification alors que la date est juste automatiquement créée lors de la création de la métadonnées par la BDD.

@Guts Guts added bug Something isn't working analisis Stats and charts labels Nov 19, 2019
@Guts Guts added this to the 1.2.3 milestone Nov 19, 2019
@Guts Guts self-assigned this Nov 19, 2019
Guts added a commit that referenced this issue Nov 19, 2019
@Guts Guts closed this as completed in #28 Dec 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
analisis Stats and charts bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant