Produits Entr'ouvert » w.c.s.

C'est dans le cadre du connecteur MDEL dans passerelle qui va créer des formulaires dans w.c.s. dont un champ ItemField est lié à une source de donnée passerelle (villes raccordées avec id étant le code INSEE, text le nom de la commune).

Ça me semble raisonnable. (je vois en passant qu'on ne documente pas les champs liste dans help/fr/api-fill.page).

Je n'autoriserais pas l'absence de l'un ou l'autre (_display ou _structured), je les récupérerais uniquement quand aucun des deux n'est défini.

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

Je n'autoriserais pas l'absence de l'un ou l'autre (_display ou _structured), je les récupérerais uniquement quand aucun des deux n'est défini.

Ce qu'on accepte actuellement:

• varname -> ok
• varname_raw -> erreur
• varname_raw + varname_display -> ok
• varname + varname_structured -> ok (parce que "store_display_value and raw in data)
• varname_raw + varname_display + varname_structured -> ok

Tu me demande d'interdire varname + varname_structured ?

Déjà simplement en ne remplissant les champs quand ils sont tous les deux absents j'en suis là :

--- a/wcs/api.py
+++ b/wcs/api.py
@@ -60,14 +60,20 @@ def posted_json_data_to_formdata_data(formdef, data):
         if field.store_structured_value and structured in data:
             data['%s_structured' % field.id] = data.pop(structured)

-    # parse special fields
+    # complete/adapt field values
     for field in formdef.get_all_fields():
-        if not hasattr(field, 'from_json_value'):
-            continue
+        structured = '%s_structured' % field.id
+        display = '%s_display' % field.id
         if data.get(field.id) is None:
             continue
-        data[field.id] = field.from_json_value(data[field.id])
-
+        if field.store_display_value and display not in data:
+            data[display] = field.store_display_value(data, field.id)
+        if field.store_structured_value and structured not in data:
+            structured_value = field.store_structured_value(data, field.id)
+            if structured_value:
+                data[structured] = structured_value
+        if hasattr(field, 'from_json_value'):
+            data[field.id] = field.from_json_value(data[field.id])
     return data

s/La valeur peut-être/La valeur peut être/

Cette relecture n'est pas exhaustive.

Voilà pour la doc.

le label de la donnée (2x)

Plutôt libellé.

            "text": "Valeur1",
            "foo": "bar",
         },

La virgule derrière "foo": "bar" n'est pas tolérée.

(moins important mais jusqu'à présent les lignes de documentation étaient wrappées à ~80 caractères).