Fix reform updating
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.
- Non, la vérification doit être
-
Réaliser une comparaison plus claire dans chaque input ValueEdit
. Actuellement, deux variablesshowBillValue
etshowLawValue
sont utilisées pour contrôler l'UI. Cependant, on a aussi plusieurs autres checks qui utilisent les variablesvalueFormatted
,billValueFormatted
etlawValueFormatted
, 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 variablesisBillActive
etisValueModified
qui seraient utilisées pour contrôler l'UI. -
Voir si on peut utiliser assert.deepStrictEqual
à la place de ma fonctionisObjectEqual
, qui je crois est meilleure ? À voir.- Utilisation du package
deep-equal
avecdeepEqual(object1, object2)
.
- Utilisation du package
Edited by Toufic Batache