Projet

Général

Profil

Bug #20078

la liaison automatique via l'email permet de créer plusieurs liaison pour un compte

Ajouté par Benjamin Dauvergne il y a plus de 6 ans. Mis à jour il y a plus de 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
14 novembre 2017
Echéance:
% réalisé:

100%

Temps estimé:
Hors marché:
Non
Patch proposed:
Oui
Planning:

Description

Ça ne devrait pas. Soit on nettoie avant de lier, soit on refuse simplement la liaison.


Fichiers

Révisions associées

Révision e9b7ed13 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 6 ans

views: forbid automatic linking based on email if multiples users are found or if target user has already a link (fixes #20078)

Révision a671f6c9 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 6 ans

views: do not permit linking if the user has already a link to FC (#20078)

Révision 89fabbc4 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 6 ans

views: change messages levels (#20078)

Révision 2e5bd982 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 6 ans

update french translations (#20078)

Révision 387252a3 (diff)
Ajouté par Benjamin Dauvergne il y a presque 5 ans

views: forbid automatic linking based on email if multiples users are found or if target user has already a link (fixes #20078)

Révision 8c3387c4 (diff)
Ajouté par Benjamin Dauvergne il y a presque 5 ans

views: do not permit linking if the user has already a link to FC (#20078)

Révision 3594460b (diff)
Ajouté par Benjamin Dauvergne il y a presque 5 ans

views: change messages levels (#20078)

Révision 5da8849b (diff)
Ajouté par Benjamin Dauvergne il y a presque 5 ans

update french translations (#20078)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

J'en ai profité pour corriger l'absence de bloquage sur la vue de liaison si le compte authentic en a déjà une (à une époque on supprimait automatiquement la liaison, ça me semble plus sûr de bloquer maintenant). J'ai aussi changé le niveau de certains messages de notification vers erreur ou warning.

#2

Mis à jour par Mikaël Ates il y a plus de 6 ans

            if len(qs) > 1:
                self.logger.warning(u'multiple accounts with the same mail %s, %s', email, list(qs))

Les emails devant être uniques, ce serait peut-être intéressant de mettre cela en erreur.

Je testerais bien les parcours sur une dev.

#3

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

Posé en dév (et passage du log en erreur).

#4

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

  • Statut changé de Nouveau à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#5

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF