Skip to content

refactor DuplicateUserFinderService to return multiple matches

Created by: adipasquale

encore une PR préliminaire à #1284

Refacto pur, pas de changement de comportement

désolé @n-b je change un peu de direction par rapport à hier :

  • J'avais introduit un param only mais en fait ça ne m'arrange pas par rapport à ce que je veux faire dans la PR successive
  • C'est en fait plus pratique que le service renvoie tous les doublons et qu'on les filtre a posteriori si besoin
  • C'est un peu moins optimisé pour l'instant puisqu'on utilise le service puis on filtre les résultats pour n'afficher que les doublons sur base du numéro de téléphone dans le users/show. Il n'y a en fait probablement pas de raison de faire ça, on devrait par cohérence afficher tous types de doublons qui se produisent je pense (par ex ceux sur base de l'identité). Je n'ai pas changé ça dans la PR mais on pourra le faire plus tard.

Merge request reports