Produits Entr'ouvert » Authentic 2

Je me rends compte que c'est un peu plus compliqué que ça :

                    if app_settings.A2_EMAIL_IS_UNIQUE and email:
                        user = User.objects.get(email__iexact=email)
                    elif provider.ou and provider.ou.email_is_unique:
                        user = User.objects.get(ou=provider.ou, email__iexact=email)
                    linked = True

Autrement dit, c'est seulement en cas d'email unique (sinon, création de doublon).

Donc c'est « création si aucun compte n'a le même mail et que le mail doit être unique » et ça commence à être un peu long, mais j'ai pas d'autre idée.

Je pense que c'est la recherche qui dépend de l'unicité du mail, et pas la création... (mais de toute façon c'est pas simple à expliquer "sur une ligne", je l'accorde)

Thomas Noël a écrit :

Je pense que c'est la recherche qui dépend de l'unicité du mail, et pas la création... (mais de toute façon c'est pas simple à expliquer "sur une ligne", je l'accorde)

Oui tu as raison, à trop faire dans la brévité on se retrouve avec une explication inexacte. Je pense ce c’est un peu plus précis dans ce patch, en espérant que ça reste compréhensible…

Ca donne donc :

create if standard account matching on email address failed (matching will fail if global- and provider's ou-wise email uniqueness is deactivated in Authentic configuration)

C'est pas un peu foufou ? Désolé de le dire comme ça, mais bon :)

On peut retirer "standard" parce que c'est "on email address". Et "in Authentic configuration" parce qu'on est déjà dedans. Et le "-" après global qui est a priori une typo ?

create if account matching on email address failed (matching will fail if global and provider's ou-wise email uniqueness is deactivated

Oui ce serait plus simple si la stratégie était un multiple choice field, plutôt que de préciser un message compliqué je dirai qu'il faudrait aller vers un champ avec des checkboxes. La configuration email unique devrait être ignorée dans ce cas, simplement si on choisit find-by-email on applique une politique d'email unique.

Benjamin Dauvergne a écrit :

Je valide, la suite dans #67022.

Ok, je me note ça sur ma todoliste.

Automatic expiration