Projet

Général

Profil

Bug #25197

Condition de sortie de page qui ne fonctionne pas à la première validation

Ajouté par Pierre Cros il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
11 juillet 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Constaté sur deux formulaires en recette (alors que le bug n'existe pas en prod me dit Victor) :

https://demarches-up.test.entrouvert.org/simulateur-apa/simulateur-de-demande-d-allocation-personnalisee-dautonomie-apa/

"Le bénéficiaire a-t-il besoin d'aide pour les gestes de la vie quotidienne ?" doit déclencher une erreur en sortie de page si on répond "non"
La condition est correcte : form_var_aide_quotidienne == "oui" sur https://demarches-up.test.entrouvert.org/backoffice/forms/39/fields/156/

Or si on répond "oui" on a quand même l'erreur. Puis en validant à nouveau sans rien changer ça passe.

Fichiers

0001-forms-make-sure-latest-live-data-are-used-to-evaluat.patch (5,17 ko) 0001-forms-make-sure-latest-live-data-are-used-to-evaluat.patch Frédéric Péters, 11 juillet 2018 11:04

Révisions associées

Révision 2a69ef50 (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

forms: make sure latest live data are used to evaluate conditions (#25197)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Sujet changé de Condition de page qui ne fonctionne pas à la première validation à Condition de sortie de page qui ne fonctionne pas à la première validation
#2

Mis à jour par Frédéric Péters il y a presque 6 ans

(en local chez moi c'est passé)

#3

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Assigné à mis à Frédéric Péters
#4

Mis à jour par Frédéric Péters il y a presque 6 ans

(ça arrive parce que source de données et .feed() de quelque chose etc.)

#5

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de Nouveau à En cours

Et en fait non, ça arrive parce que combinaison de conditions et post-conditions, l'évaluation des conditions de page (pour afficher les noms de page en étapes) crée un contexte d'évaluation qui est réutilisé par les conditions de sortie, sans prendre en compte le fait que de nouvelles données sont apparues. (tests en cours d'exécution en local).

#6

Mis à jour par Frédéric Péters il y a presque 6 ans

Et donc, pour assurer un nouveau ConditionsVars, on assure qu'on en a un par id(les données) différent, et on assure que cet id (du dictionnaire des données) soit bien différent avant d'évaluer les post-conditions.

#7

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Statut changé de Solution proposée à Solution validée

Ack.

#8

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 2a69ef50095286e20e656c14ee089d90d646d70e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jul 11 10:30:25 2018 +0200

    forms: make sure latest live data are used to evaluate conditions (#25197)
#9

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF