Produits Entr'ouvert » w.c.s.

Quand une valeur lazy est demandée dans un gabarit (_raw, ou __str__),

     @property
     def raw(self):
         if self._field.store_display_value or self._field.key in ('file', 'date'):
-            return self._data.get(self._field.id)
+            raw_value = self._data.get(self._field.id)
+            return get_publisher().cache_complex_data(raw_value)
         raise AttributeError('raw')
...
     def __str__(self):
         value = self.get_value()
         if not isinstance(value, six.string_types):
-            value = str(value)
+            value = get_publisher().cache_complex_data(value)
         return force_str(value)

On la passe dans cache_complex_data() qui garde un dictionnaire de correspondance entre la représentation textuelle de la valeur et celle-ci,

+        # Keep a temporary cache of assocations between a complex data value
+        # (value) and a string reprensentation (str(value) or the dedicated
+        # str_value parameter).

Pour assurer que deux fichiers différents qui seraient représentés pareils soient confondus (parce que la représentation d'un fichier est son nom), on ajoute à la chaine une marque unique ("caractère" unicode privé),

+        # It ensures string values are unique by appending a private unicode
+        # code point, that will be removed in wcs/qommon/template.py.
...
+        str_value += chr(0xE000 + len(self.complex_data_cache))

Au moment d'utiliser la valeur (actions données de traitement ou création d'une demande, citées dans ce ticket), on regarde dans le dictionnaire et on prend la valeur qui peut y être,

+                    if formdef_field.allow_complex:
+                        complex_value = get_publisher().get_cached_complex_data(new_value)
+                        if complex_value:
+                            new_value = complex_value

Pour les endroits qui ne gèrent pas les données complexes, ces caractères sont juste virés,

+        return re.sub(r'[\uE000-\uF8FF]', '', rendered)

Les tests couvrent les deux actions citées et comme types de champs les fichiers, listes à choix multiples et blocs de champs (bout qui corrigera #48436).

Tout ça me semble tout bon.

• il faudra pousser l'affaire ensuite afin qu'un pre-fill avec {{ cards|...|getlist:... }} fonctionne sur un champ liste à choix multiple
• et avoir dans le debug de l'inspecteur une information concernant le résultat d'un {{form_var_file}} en tant que donnée complexe (pour "montrer" que ça renvoie bien un fichier quand c'est utilisé en mode donnée de traitement et autres).

commit 8c551089165e62ef8ac494fa10aee7d8d0c943c8
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Nov 29 17:50:37 2020 +0100

    general: allow assigning complex types from rendered templates (#41847)