Produits Entr'ouvert » w.c.s.

Tu peux attacher un export du formulaire à ce ticket ? Éventuellement réduire aux seuls champs impliqués ? Éventuellement faire une vidéo du problème ?

Parce que j'ai essayé de suivre la description et reproduire sans ces éléments, sans succès,
https://perso.entrouvert.org/~fred/tmp/condition-obligatoire.ogv

Vidéo ou produire une liste d'actions courtes amenant le résultat,

• choisir oui
• cliquer sur suivant
• cliquer sur précédent
• choisir non
• cliquer sur suivant
  • ça aurait du dire que le champ xxx était obligatoire.

Sauf que c'est là normal, parce que le <select> a un seul élément, forcément sélectionné.

L'exemple côté 06 est plutôt :

        fields.ItemField(type='item', id='2', label='Foo', size='40',
            required=True,
            hint='---',
            items=['plop'],
            condition={'type': 'django', 'value': 'form_var_bar == "oui"'}),

et si je fais ça, le test fonctionne correctement, l'usager est bien interrompu.

Trouvé ce qui enclenche le bug, c'est la présence d'une condition sur la page qui suit, j'ai donc maintenant un test unitaire qui échoue correctement.

        fields.PageField(id='3', label='1st page', type='page',
            condition={'type': 'python', 'value': 'True'}),

Ce qui se passe c'est qu'à l'évaluation des pages (notamment pour l'affichage en barre latérale), il y a un ConditionVars() qui est ajouté au contexte, avec les données de la demande dans son état présent (i.e. avant traitement des champs de la page en cours).

Du coup plus loin quand ce sont les champs qui sont évalués pour voir s'ils sont visibles ou pas, ça tape dessus, ça considère le champ pas visible, et donc son critère "obligatoire" n'est pas pris en compte.

Tous les tests (de test_form_pages) passent, y compris le nouveau, avec cette modification, qui basiquement annule l'ajout du ConditionVars.

--- a/wcs/forms/root.py
+++ b/wcs/forms/root.py
@@ -522,6 +522,7 @@ class FormPage(Directory, FormTemplateMixin):
         if self._pages:
             return self._pages
         current_data = self.get_transient_formdata().data
+        sources = get_publisher().substitutions.sources[:]
         pages = []
         field_page = None
         for field in self.formdef.fields:
@@ -532,6 +533,7 @@ class FormPage(Directory, FormTemplateMixin):
         if not field_page:  # form without page fields
             pages = [None]
         self._pages = pages
+        get_publisher().substitutions.sources = sources
         return pages

     def reset_pages_cache(self):

C'est plutôt moche, il y aurait bien sûr moyen de faire ça avec un with ...: mais ce serait sur le fond pareillement moche.

Une autre possibilité ça me semble être de construire le CompatibilityNamesDict en y conservant l'ordre des sources, pour qu'à l'évaluation, la source "transient&lazy formdata" qui est tapée en haut de la pile soit testé avant de tomber sur le ConditionVars (mais ça demande de remplacer le CompatibilityNamesDict(dict) par quelque chose vraisemblablement moins performant).

Aussi, il y aurait à voir comment tout simplement dégager ConditionVars(), voir dans quelle mesure le "transient formdata" pourrait être créé plus tôt et déjà utiliser pour l'évaluation des étapes. (mais quelques tentatives naïves ont amenés quelques tests en erreur).

Au final, ça n'est de toute façon pas pour ce soir, je vais donc dormir dessus et voir. (mais il y a des chances quand même pour finalement m'arrêter à la solution moche et reporter un travail de fond à plus tard).

J'aurais pensé ce contextmanager dans un PageCondition::evaluate du genre :

class PageCondition(Condition):
    ...

    def evaluate(self):
        with get_publisher().substitutions.freeze():
            return super().evaluate()

... mauvaise idée ? ("sécurité" superflue ?)

Il est possible qu'à d'autres moments on profite de l'effet de bord, donc je suis frileux là.