export PDF: bloc de jours non observés

[jean-io]

Dans certain export PDF un bloc de jours non observé est present dans le tableau, alors qu'en réalité, les jours sont observés.

[Limezy]

Je n'arrive pas à reproduire

[jean-io]

Non moi non plus. Une utilisatrice m'a envoyé ce bug. Je vais devoir retrouver les observations en bdd pour comprendre.

[mtctn]

Bonjour,

Même problème pour moi, observé lors de l'export en PDF ou en CSV.
En regardant le code j'ai l'impression que ça vient du fait que deux enregistrements sont présents à la même date (par exemple deux observations datées du 18/06), et l'algorithme qui prépare les données pour l'affichage ajoute des jours pour combler l'intervalle entre le 18 du mois et... le 18 du mois, qui apparaissent comme étant du 19/06 au 30/06 puis du 01/06 au 17/06.

Je pense donc qu'il y a deux problèmes indépendants :

• il est possible que deux enregistrements soient faits pour la même date
• lorsque c'est le cas, l'export en PDF ou en CSV ajoute des jours non observés

Merci beaucoup pour l'appli en tout cas !

[jean-io]

Bonjour @mtctn, merci pour l’analyse et la contribution 🙂 est-ce que vous avez réussi à reproduire le problème ?

Si le problème est confirmé, il y a deux méthodes :

1. supprimer l'observation dans l'interface web, il restera logiquement l'autre si l'instruction DELETE efface pas les deux.
2. supprimer l'une des deux observations au choix dans la base de donnée.

[mtctn]

Je rencontre actuellement le problème sur plusieurs cycles sur l'instance tableau.moncycle.app si vous voulez regarder les données.

Pour la solution 1, est-ce que je suis sûr que ça n'effacera qu'une seule des deux observations ?

Et pour la solution 2 il faut avoir un accès direct à la bdd ?

[jean-io]

Je pensais que vous utilisiez une version auto hébergé, c'est pour ca que je vous ai proposé l'option 2.

il est possible que deux enregistrements soient faits pour la même date

Comment êtes vous arrivé à cette conclusion ? en théorie c'est pas possible.

Pour l'option 1, je regarde.

[mtctn]

La solution 1 fonctionne mais supprime les deux lignes, il faut donc re-saisir l'observation.

Merci beaucoup !

[jean-io]

Top voila l'explication du fix :

Le call API s’exécute ici : https://github.com/jean-io/moncycle.app/blob/60b6ebdab2655c5e8b562a6c46374c4d670fe001/www_data/api/observation.php#L122

Le nombre de lignes supprimées est bien renvoyé vers le navigateur mais non affiché :
https://github.com/jean-io/moncycle.app/blob/60b6ebdab2655c5e8b562a6c46374c4d670fe001/www_data/api/observation.php#L126

Il n'y a pas de limite du nombre de lignes supprimées dans la querie SQL :
https://github.com/jean-io/moncycle.app/blob/60b6ebdab2655c5e8b562a6c46374c4d670fe001/www_data/lib/db.php#L188

Est-ce que vous pouvez ouvrir votre console est verifier le nombre de lignes supprimées?

Sinon une petit étoile sur le repo nous aide 🙂 merci.

[mtctn]

Je suis arrivé à cette conclusion en regardant le code qui fait l'export et les données produites.
Si vous regardez les lignes J12 et J36 dans l'image qui illustre le premier post la donnée est dupliquée, et dans les cas que j'ai vus la date était la même.
Après je ne suis pas certain non plus de mon interprétation.

[jean-io]

Je viens de regarder le PDF qui a illustré cette issue, et il y a bien cette incohérence ! bien vu !

Avez vous une idée de comment deux observations ont pu être renseigné pour la même date ? Il y avait pas de problème au niveau de l'affichage web ? C'est peut être un bug seulement au moment de l'export PDF.

[mtctn]

Est-ce que vous pouvez ouvrir votre console est verifier le nombre de lignes supprimées?

C'est malheureusement trop tard, on a corrigé toutes les occurrences avant de voir ce message :'( J'essaierai si ça se reproduit (mais on avait au moins 3 ou 4 occurrences...)

[Limezy]

@jean-io je viens de découvrir que j'avais un cas sur mon instance auto hébergée, l'épidémie semble un peu plus sévère que prévu !

Je ne touche à rien pour le moment et attends tes instructions si tu souhaites en profiter pour mener l'enquête.

J'imagine qu'une requête SQL sur la base de données de la version tableau.moncycle.app permettra de filtrer les doublons et identifier le nombre de cycles et d'utilisateurs touchés 🙃

Reste à retrouver pourquoi et comment plusieurs observations peuvent être crées pour la même date...

[Limezy]

Je confirme, après vérification dans ma base, que j'ai deux lignes d'observations pour la même date, ce qui semble confirmer l'analyse de @mtctn (bien joué!)

[jean-io]

Hum ce sont pas des bonnes nouvelles... 🤔 il y a un seul endroit dans le code où il y a une insertion d'une entrée dans la table observation:

https://github.com/jean-io/moncycle.app/blob/60b6ebdab2655c5e8b562a6c46374c4d670fe001/www_data/api/observation.php#L85

Reste à comprendre pourquoi cette condition n'est pas suffisante...

@Limezy peux-tu partager la requête SQL qui t'a permis d'identifier les doublons?

Pour moi il y a deux manières complémentaires de fixer le problème :

1. Améliorer la condition ci-dessus.
2. Rajouter une contrainte dans le schéma de la base interdisant plusieurs dates pour un même compte.

[jean-io]

Il y a aucun bug d'affichage web ? C'est uniquement sur les PDF ?