agateblue / tempo

Your personal diary and mood tracker
GNU Affero General Public License v3.0
106 stars 12 forks source link

Bug in annotation parsing/edit #45

Closed entropyqueen closed 1 year ago

entropyqueen commented 1 year ago

Quand on édite un tag annotation, le contenu n'est pas re-parsé, il faut nécessairement l'éditer dans les champs en dessous. Je pense que ça ferai sens d'avoir un re-parsing de l'entrée.

Par exemple, je veux créer 2 annotations: tempo_annotation_bug1

Je me suis ratée, ça donne ça: tempo_annotation_bug2

Donc j'édite pour remettre le retour à la ligne, ce qui donne: tempo_annotation_bug3

Je pense qu'il serait préférable que la valeur du tag soit prise jusqu'au \n ou s'arrêter avant si il y a une autre annotation ? Et sinon juste pour l'édition, re-parser l'entrée pour corriger les valeurs.

agateblue commented 1 year ago

Il y avait en fait deux bugs:

  1. Un souci dans le parsing, corrigé
  2. Un comportement que j'avais introduit, qui affichait les champs de formulaires correspondant pour les annotations lors de l'édition d'une entrée. Sauf que c'était les données du formulaire qui prenaient le dessus, et pas celles des annotations dans le corps de l'entrée. J'ai retiré cette logique de formulaire pour les annotatiors libres, sans formulaires, c'est plus simple.