Produits Entr'ouvert » w.c.s.

On imagine des situations où on voudrait matcher sur une liste précise de valeurs ? (parce que moi pas et du coup je serais plutôt pour en rester à filter_value, qu'il adapte son comportement sur les champs "liste à choix multiple").

+                # match againts first value only
+                criterias = Intersects(field_id, value[:1], field=field)

Avec les tests présents je capte bien que ça correspond à :

+    tmpl = Template('{{forms|objects:"foobarlazy"|filter_by:"items"|filter_value:"1|2"|count}}')

Mais ignorer les paramètres ne m'a pas l'air d'être le comportement qu'on souhaiterait; on voudra plutôt soit que toutes les valeurs matchent, soit une des valeurs, jamais "je file plusieurs valeurs mais regarde que la première merci".

(quant à choisir entre les deux options qui ont du sens, je ne sais pas)

Aussi il y aurait à gérer la branche exclude.

Pourquoi un nouveau NotIntersects plutôt qu'utiliser Not & Intersects qui existent ?

question qui est peut-être aussi celle-ci : pourquoi sur une valeur vide on a le même code dans ce nouveau NotIntersects que le code qui existe dans Intersects ?

        if not self.value:
            return 'ARRAY_LENGTH(%s, 1) IS NULL' % self.attribute

Frédéric Péters a écrit :

Pourquoi un nouveau NotIntersects plutôt qu'utiliser Not & Intersects qui existent ?

Comme on a NotEqual et NotContains, j'ai suivi le même principe.

question qui est peut-être aussi celle-ci : pourquoi sur une valeur vide on a le même code dans ce nouveau NotIntersects que le code qui existe dans Intersects ?

Betise de ma part, ça doit être plutôt un: return 'FALSE'

Comme on a NotEqual et NotContains, j'ai suivi le même principe.

Je serais plutôt pour ne pas.

Misère fallait dire qu'il n'y avait pas de Not pas SQL (j'avais oublié). Tu peux ajouter cette partie dans #6108, avec test adéquat ?

Et de ce côté, tu pourrais simplifier le code pour ne pas utiliser NotEqual, que le code soit :

# code qui crée criterias
[...]
if exclude:
    criterias = Not(criterias)

?