Produits Entr'ouvert » w.c.s.

Je suis un peu réticent à l'idée de les mettre directement dans l'espace de nommage "user", avec le risque qu'un jour quelqu'un nomme mal une variable, genre "get_formdef", et boum, conflit...

Par ailleurs, avoir juste la valeur mais ne pas savoir le label du champ, ça ne simplifie pas l'écriture de templates génériques.

Quid d'un "user.fields" ? Quitte même à ce que chaque fields contienne label et value...

user.fields.last_name.label
user.fields.last_name.value

avec la possibilité de faire un "for field in user.fields" pour parcourir les champs dans l'ordre de définition du formdef, ce genre d'outils.

Ça ne va pas dans le sens d'une simplification, mais plutôt d'une rationalisation ; ça serait un principe générique pour les "champs formdef" des objets user, formdata, etc... L'idée serait aussi que "value" serait un type de donnée propre (je pense surtout aux dates qui seraient des datetime).

Je suis un peu réticent à l'idée de les mettre directement dans l'espace de nommage "user", avec le risque qu'un jour quelqu'un nomme mal une variable, genre "get_formdef", et boum, conflit...

L'objectif est d'être compatible avec l'«API» exposée par Django, qui a user.first_name/user.last_name.

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

L'objectif est d'être compatible avec l'«API» exposée par Django, qui a user.first_name/user.last_name.

Ok, pigé. Et je viens de faire le test, les attributs existants sont bien préservés, tout va bien. Ack.

Au cas où, j'ai même testé mon inquiétude

diff --git a/tests/test_users.py b/tests/test_users.py
index f7c857ea..e2bcb09c 100644
--- a/tests/test_users.py
+++ b/tests/test_users.py
@@ -61,14 +61,19 @@ def test_get_users_with_role():
 def test_user_formdef_getattr():
     from wcs.admin.settings import UserFieldsFormDef
     formdef = UserFieldsFormDef(pub)
-    formdef.fields = [fields.StringField(id='3', label='test', type='string', varname='plop')]
+    formdef.fields = [
+        fields.StringField(id='3', label='test', type='string', varname='plop'),
+        fields.StringField(id='9', label='noop', type='string', varname='get_formdef'),
+    ]
     formdef.store()

     user = pub.user_class()
     assert user.plop is None
+    assert user.get_formdef()  # get_formdef is not overrided by varname "get_formdef" 

-    user.form_data = {'3': 'Bar'}
+    user.form_data = {'3': 'Bar', '9': 'Foo'}
     assert user.plop == 'Bar'
+    assert user.get_formdef()

     with pytest.raises(AttributeError):
         user.xxx