Produits Entr'ouvert » w.c.s.

Voire même en un seul coup :

|getlistdict:"prenom:first_name, nom:last_name, id:user_external_id" 

(idéalement faire fonctionner aussi ce filtre sur des queryset, des blocs, des listes à choix multiple qui sont pas des fiches... ok là j'ai pas d'usage mais je les imagine arriver par la fenêtre)

Oui pour supporter également queryset et liste à choix multiple obtenue depuis un JSON quelconque, par contre les blocs je ne vois pas vraiment ce qui serait attendu donc je vais passer.

Voire même en un seul coup :

J'ai pas d'avis donc je vais faire comme tu dis, on attendra un autre cas d'usage pour ajouter |rename_key.

Il faudrait striper après name, new_name = key.strip().split(':', 1) pour gérer "toto: tata".

Il faut aussi gérer le cas où unlazy(keys) va être vide, parce que split(',') va renvoyer [''] et non pas une liste vide.

• on va renvoyer une chaine vide pour LazyFieldVarItems. Je comprends que c'est pour ressembler à getlist, mais on n'est pas dans le même usage, je pense qu'on devrait plutôt ne rien renvoyer dans le dico (oublier l'attribut)
• on va déclencher une AttributeError sur un block, là encore imitation de getlist mais là aussi, on devrait juste ne rien renvoyer, sans planter.
• idem sur LazyFormDefObjectsManager, ne pas renvoyer None, mais rien du tout.

Si le code devient sale à cause de ça, on peut décider de renvoyer attr:None, mais je préfère que attr ne fasse pas partie de la réponse, ça sera le signe que cet attribut n'existe pas dans l'objet.

La partie :

                value = lazy_formdata.get(key)
                if value is None:
                    continue

... rien qu'à cause de ça, je me demande s'il ne faudrait pas changer le patch d'épaule, et donc revenir un peu en arrière, ie renvoyer clé:None quand la clé ne "semble" pas exister.

C'est vraiment un détail, désolé Valentin.

Ce qui nous empêche de différencier une valeur à None d'une valeur qui n'existe pas c'est le return compat_dict.get(key) de la méthode get... On peut peut-être envisager d'ajouter un paramètre raise_if_not_found, ou une nouvelle méthode contains qui fasse ce qu'on veut ?

Ce qui nous empêche de différencier une valeur à None d'une valeur qui n'existe pas c'est le return compat_dict.get(key) de la méthode get... On peut peut-être envisager d'ajouter un paramètre raise_if_not_found, ou une nouvelle méthode contains qui fasse ce qu'on veut ?

Je rate peut-être quelque chose dans le contexte précis de ce ticket mais ce compact_dict.get() avec raise_if_not_found évoqué, ça ne serait pas déjà simplement compat_dict[key] ?

(Pdb) p type(substvars)
<class 'wcs.qommon.substitution.CompatibilityNamesDict'>
(Pdb) substvars['form_var_plop']
<wcs.variables.LazyFieldVar object at 0x7f7cdaa7b790>
(Pdb) substvars['form_var_plop2']
*** KeyError: 'form_var_plop2'

Sur :

    for key in unlazy(keys).split(','):
        if not key:
            continue

ça serait plutôt « if not key.strip(): » pour gérer le cas getlistdict:"key1, ,key2"

Ca me semble ma dernière remarque.

Automatic expiration