Produits Entr'ouvert » Authentic 2

Je propose de simplement overrider clean() et de ne pas appeler super().clean() et d'adapter éventuellement le traitement en ajoutant un if self.username: puisque chez nous le champ n'est pas obligatoire (je ne sais pas exactement ce que ça fait).

Et oui j'aimerai bien que username soit égal à None et pas "None" puisque j'ai l'impression que c'est ça qui se passe maintenant avec dj111, c'est bien cela ?

Et oui j'aimerai bien que username soit égal à None et pas "None" puisque j'ai l'impression que c'est ça qui se passe maintenant avec dj111, c'est bien cela ?

Non, username égal testbot@entrouvert.com.

Emmanuel Cazenave a écrit :

Et oui j'aimerai bien que username soit égal à None et pas "None" puisque j'ai l'impression que c'est ça qui se passe maintenant avec dj111, c'est bien cela ?

Non, username égal testbot@entrouvert.com.

Et d'où sort cette copie de l'email dans le username ? Je ne vois rien de tel dans le commit Django que tu pointes.

Dans le commit il y a :

def clean(self):
+        setattr(self, self.USERNAME_FIELD, self.normalize_username(self.get_username()))

Et dans src/authentic2/custom_user/models.py::User :

def get_username(self):
    "Return the identifying username for this User" 
     return self.username or self.email or self.get_full_name() or self.uuid

Et donc dans le test que je pointais l'utilisateur est crée avec un seul champ définis (parmi ceux utilisés par get_username) : email.

Et voilà c'est ma faute :) Je reste sur ma suggestion, ré-implémenter clean() comme on a envie qu'il fonctionne, de toute façon on converge vers le fait d'abandonner complètement django.contrib.auth à force.