Produits Entr'ouvert » Authentic 2

Dans le code actuel, A2_PROFILE_FIELDS donne la liste à afficher et à modifier, et ça surpasse tout ce qui peut être raconté via user_editable ou user_visible, qui ne sont pas pris en compte pour définir l'affichage ou le formulaire.

Je proposerai bien de ne pas changer ce principe des "settings qui décident", mais de rajouter deux settings : A2_USER_VISIBLE_PROFILE_FIELDS et A2_USER_EDITABLE_PROFILE_FIELDS qui surpasseraient A2_PROFILE_FIELDS. Et si rien n'existe, on se base sur les user_visible/user_editable (c-a-d qu'on laisse en place le principe actuel).

C'est peut-être un peu plus lourd, mais ça me semble moins magique et ça casserait rien ?

(restera alors dans les settings_loader de hobo à gérer ces deux nouveaux settings)

genre :

diff --git a/src/authentic2/app_settings.py b/src/authentic2/app_settings.py
index 38e62e1..908563e 100644
--- a/src/authentic2/app_settings.py
+++ b/src/authentic2/app_settings.py
@@ -120,6 +120,8 @@ default_settings = dict(
     A2_PROFILE_FIELDS=Setting(default=(), definition='Fields to show to the user in the profile page'),
+    A2_VISIBLE_PROFILE_FIELDS=Setting(default=(), definition='Fields to show to the user in the profile page'),
+    A2_EDITABLE_PROFILE_FIELDS=Setting(default=(), definition='Fields to show to the user in the profile edit page'),
     A2_REGISTRATION_FIELDS=Setting(default=(), definition='Fields from the user model that must appear on the registration form'),
diff --git a/src/authentic2/views.py b/src/authentic2/views.py
index 08dc8f6..a9169f8 100644
--- a/src/authentic2/views.py
+++ b/src/authentic2/views.py
@@ -80,7 +80,7 @@ class EditProfile(cbv.TemplateNamesMixin, UpdateView):
     def get_fields(cls):
         default_fields = list(models.Attribute.objects.filter(user_editable=True).values_list('name', flat=True))
         return utils.get_fields_and_labels(
-            app_settings.A2_PROFILE_FIELDS,
+            app_settings.A2_EDITABLE_PROFILE_FIELDS or app_settings.A2_PROFILE_FIELDS,
             default_fields)
@@ -356,7 +356,7 @@ class ProfileView(cbv.TemplateNamesMixin, TemplateView):
         # User attributes management
         profile = []
-        field_names = app_settings.A2_PROFILE_FIELDS
+        field_names = app_settings.A2_VISIBLE_PROFILE_FIELDS or app_settings.A2_PROFILE_FIELDS
         if not field_names:

• changer l'ordre des champs en fonction du contexte
• changer le label du champ en fonction du contexte

Pour moi un champ doit être utilisé si le flag Attribute.flag du champ en question est vrai, c'est tout. Les A2_*_FIELDS ne doivent servir qu'à en changer l'ordre ou le label.

« Les A2_*_FIELDS ne doivent servir qu'à en changer l'ordre ou le label. »

• dans EditProfile, retirer les attributs qui n'ont pas user_editable
• dans ProfileView, retirer ceux qui n'ont pas user_visible

Ack.