Produits Entr'ouvert » w.c.s.

+            elif filter_field.type == 'bool':
+                filter_field_value = filter_field_value == 'true'

Je serais pour zapper le filtre si jamais la valeur n'est ni 'true' ni 'false', comme on fait déjà ici :

            elif filter_field.type == 'bool' and filter_field_value is not None:
                if filter_field_value == 'true':
                    criterias.append(Equal('f%s' % filter_field.id, True))
                elif filter_field_value == 'false':
                    criterias.append(Equal('f%s' % filter_field.id, False))

Peut-être même, à penser à #59224 ("filtre qui n'existe pas"), faire que si ni 'true' ni 'false' on retourne une erreur ?

+        if getattr(filter_field, 'block_field', None):
+            field1 = "jsonb_array_elements(%s->'data')->> '%s'" % (
+                sql.get_field_id(filter_field.block_field),
+                filter_field.id,
+            )

Je serais assez pour l'ajout d'un commentaire qui explique en début du bloc if ce qu'on fait là, en inversant la condition pour avoir le cas "non bloc" en premier, je pense à quelque chose comme :

        # for item/items fields, get actual option values from database
        if not getattr(filter_field, 'block_field', None):
            criterias.append(NotNull(sql.get_field_id(filter_field)))
            options = self.formdef.data_class().select_distinct(
                [sql.get_field_id(filter_field), '%s_display' % sql.get_field_id(filter_field)], clause=criterias
            )
        else:
            # in case of blocks, this requires digging into the jsonb columns,
            # jsonb_array_elements(BLOCK->'data')->> 'FOOBAR' will return all
            # values used in repeated blocks, ex:
            # {"data": [{"FOOBAR": "value1"}, {"FOOBAR": "value2}]}
            # → ["value1", "value2"}
            ...

Plus loin, j'ai un autre commentaire "ajouter un commentaire", pour

+            is_in_block_field = getattr(field, 'block_field', None)
+            field.contextual_id = field.id
+            field.contextual_varname = None
+            ...

Ajouter quelques mots avant, de l'ordre de "# add contextual_id/contextual_varname attributes, they are id/varname for normal fields, and they are concatenation of block id/varname + field id/varname in case of blocks".

_~~

Voilà, c'est déjà lisible comme c'est mais je pense qu'on sera aidé par ces commentaires plus tard.

Automatic expiration