Skip to content

Fix reform updating

Toufic Batache requested to merge fix_update_reform into master

Bug

Lorsqu'on modifie une valeur dans le panneau gauche "modifier le droit" et qu'on la remet à sa valeur initiale, elle reste bloquée et on la considère comme changement actif dans la réforme.

Résolution

Vérification en comparant la valeur initiale et la valeur courante (marche aussi avec un objet en vérifiant une égalité profonde).

À résoudre

  • Vérifier si la vérification value === billValue.value && value === lawValue.value est correcte ou s'il y a des changements à faire pour la version PLF/PLFSS.
    • Non, la vérification doit être value === billValue.value. Une modification est présente si la valeur entrée est différente de la valeur du PLF s'il y a un PLF en cours, ou si la valeur entrée est différente de la loi s'il n'y a pas de PLF. Or, la valeur du PLF est celle de la loi en temps normal, d'où la logique : il y a une modification si la valeur entrée est différente de la valeur du PLF.
  • Réaliser une comparaison plus claire dans chaque input ValueEdit. Actuellement, deux variables showBillValue et showLawValue sont utilisées pour contrôler l'UI. Cependant, on a aussi plusieurs autres checks qui utilisent les variables valueFormatted, billValueFormatted et lawValueFormatted, elles-mêmes utilisées par les deux variables mentionnées avant. Ma solution pour rendre le code plus clair serait d'avoir seulement deux variables isBillActive et isValueModified qui seraient utilisées pour contrôler l'UI.
  • Voir si on peut utiliser assert.deepStrictEqual à la place de ma fonction isObjectEqual, qui je crois est meilleure ? À voir.
    • Utilisation du package deep-equal avec deepEqual(object1, object2).
Edited by Toufic Batache

Merge request reports