Produits Entr'ouvert » w.c.s.

• si on supprime la condition de la page 1, tout marche
• si la condition est "True" alors la page 2 s'affiche bien (ce qui laisserait penser à une absence de la variable form_var_val1 ?)

• 'field_val1': 'x'
• 'var_val1': 'x'
  mais
• 'form_field_val1': None
• 'form_var_val1': None

Un test qui fait planter :

diff --git a/tests/test_form_pages.py b/tests/test_form_pages.py
index 12d28fbe..2d2ae075 100644
--- a/tests/test_form_pages.py
+++ b/tests/test_form_pages.py
@@ -757,6 +757,22 @@ def test_form_multi_page_condition_on_first_page(pub):
     with pytest.raises(AssertionError):
         resp.form.get('previous')

+def test_form_multi_page_condition_with_form_var(pub):
+    formdef = create_formdef()
+    formdef.fields = [
+            fields.PageField(id='0', label='1st page', type='page',
+                condition={'type': 'python', 'value': 'True'}),
+            fields.StringField(id='1', label='string', varname='val1'),
+            fields.PageField(id='2', label='2nd page', type='page',
+                condition={'type': 'python', 'value': 'vars().get("form_var_val1") == "foo"'}),
+            fields.StringField(id='3', label='string 2')]
+    formdef.store()
+    resp = get_app(pub).get('/test/')
+    formdef.data_class().wipe()
+    resp.forms[0]['f1'] = 'foo'
+    resp = resp.form.submit('submit')
+    assert resp.form['f3']
+
 def test_form_multi_page_condition_no_visible_page(pub):
     formdef = create_formdef()
     formdef.fields = [

Résultat :

E       AssertionError: No field by the name 'f3' found

Tout soucis disparait si on ajoute une première page sans condition.

Mon analyse actuelle : tout est lié à la gestion des conditions sur la première page.

Parce que lors de la génération des variables de substitution, on ajoute des ConditionVars ainsi que:

        class ConditionVars(object):
            def get_substitution_variables(self):
                return form_live_data

            def __eq__(self, other):
                # Assume all ConditionVars are equal (as they are because they
                # are created using the same live data); this avoids filling
                # the substitution sources with duplicates and invalidating its
                # cache.
                try:
                    return self.__class__.__name__ == other.__class__.__name__
                except AttributeError:
                    return False

Mais si on a une première page avec une condition, on fait ça :

        ...
        if self.fields and self.fields[0].type == 'page' and self.fields[0].condition:
            # multipage form with conditions on the first page
            for field in self.fields:
                if field.type != 'page':
                    continue
                if field.is_visible({}, self):    <--- boum
                    break
                ...

et donc on génère dans ce cas un premier ConditionVars avec un contenu "{}" dans le formdata, donc tous les form_* à None, qui prendra toujours le dessus sur les ConditionVars des pages suivantes, à cause du "__eq__" qui dit de ne pas ajouter de ConditionVars quand il y en a déjà un.

Pour minimiser le diff je laisserais le ConditionVars à son niveau d'indentation actuel, je ne conditionnerais que le .feed().

J'aurais fait des test sur "is None" (et modifier l'appel .is_visible({}...)

all will be "None"

Ça fait penser que ce serait un None transformé par erreur en chaine de caractères :/ Peut-être "(they will all have been set to None)". Et peut-être inclure le commentaire à l'intérieur de la condition ?

Et plutôt, reprise total, proposition pour davantage d'explications :

  # ConditionsVars is not set when evaluating first page but we need to have form_var_*
  # variables already; add them from form_live_data (where all variables will have been
  # set to None).

Go.