Produits Entr'ouvert » w.c.s.

Je regarde cette fonction:

    def get_page(self, page_no):
        return [x for x in self.fields if x.type == 'page'][page_no]

et je me rends compte que contrairement par exemple à add_fields_to_form() ça ne prend pas en compte le cas du champ initial qui est une page ou non:

    def add_fields_to_form(self, form, page_no = 0, displayed_fields = None, form_data = None):
        current_page = 0
        for field in self.fields:
            if field.type == 'page':
                if field is self.fields[0]:
                    continue
                current_page += 1
                if current_page > page_no:
                    break
                continue
            if current_page != page_no:
                continue
            if type(displayed_fields) is list:
                displayed_fields.append(field)
            value = None
            if form_data:
                value = form_data.get(field.id)
            field.add_to_form(form, value)

ici on ignore bien le premier champ PageField si il est là (la page zéro est sinon implicite).

La fonction n'est utilisé que deux fois dans tout le code (pour trouver la page suivante/précédente visible), mais j'ai l'impression que si on a pas fields⁰.type == 'page' on a une off-by-one error et donc il me semble qu'en l'absence d'une première page explicite, les validations de condition de page sont toutes fausses (décalés d'une pages).

Un test simple serait un formulaire avec un seul champ sur la première page implicite puis une page avec une condition qui ne passe jamais, on doit pouvoir passer à la seconde page alors qu'on ne devrait pas.

Une implémentation de get_page() qui je pense marche serait:

diff --git a/wcs/formdef.py b/wcs/formdef.py
index 0e894a9c..b32dbc15 100644
--- a/wcs/formdef.py
+++ b/wcs/formdef.py
@@ -555,7 +555,14 @@ class FormDef(StorableObject):
             field.add_to_form(form, value)

     def get_page(self, page_no):
-        return [x for x in self.fields if x.type == 'page'][page_no]
+        # if there is no first field being a PageField, PageField for page_no
+        # zero does not exist
+        if page_no == 0:
+            if self.fields and self.fields[0].type == 'page':
+                return self.fields[0]
+            raise IndexError
+        else:
+            return [x for x in self.fields[1:] if x.type == 'page'][page_no - 1]

     def create_view_form(self, dict={}, use_tokens=True, visible=True):
         form = Form(enctype='multipart/form-data', use_tokens=use_tokens)