Admin : Correction d'un bug empêchant la fusion d'utilisateurs

[leo-naeka]

🤔 Pourquoi ?

https://gip-inclusion.slack.com/archives/C01181Y04LT/p1755510327609539

🍰 Comment ?

• Utilisation de l'introspection des apps pour récupérer toutes les FK concrètes vers users_user, en incluant les relations cachées (sans inverse rel)
• Dans l'absolu ça pourrait être caché, mais vu le faible usage je n'en vois pas l'intérêt…
• Permet d'éviter de maintenir à jour manuellement la liste des FK sans reverse (ce qui a été récemment oublié pour les demandes de changement de NIR) tant dans notre code que dans des apps tierces

🚨 À vérifier

• Mettre à jour le CHANGELOG_breaking_changes.md ?
• Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

[francoisfreitag]

Un petit test serait utile :)

[rsebille]

Je ne comprends pas pourquoi ça n'a pas été fait de base, n'en vois pas la raison

Je pense que c'est pour justement que ça casse afin qu'on se pose la question de si c'est légitime ou pas, et ainsi d'ajuster MODEL_MAPPING pour gérer la fusion adéquatement et MODEL_REPR_MAPPING pour avoir une trace réutilisable de ce qui est déplacé.

[leo-naeka]

Je pense que c'est pour justement que ça casse afin qu'on se pose la question de si c'est légitime ou pas

Le fait est que ça ne cassait que lorsqu'il n'y a pas de relation inverse de créée. Là ça cassait car il y avait bien une entrée référençant User dans une autre table, mais comme pas de relation inverse, elle n'était pas retournée par le get_users_relations actuel.

Sinon pour toute relation on transfère la data, sauf si flag en noop dans MODEL_MAPPING

[leo-naeka]

Un petit test serait utile :)

La fusion des utilisateurs est déjà testée. Tu voudrais tester unitairement get_users_relations ?
Quoi spécifiquement ? La version que je propose n'est ni plus ni moins qu'un appel à l'introspection Django, qui filtre sur des FK depuis un seul côté et retire les relations à ignorer (statiquement flag en "noop" dans MODEL_MAPPING)

[xavfernandez]

Comme test supplémentaire, je m'attendrais à voir l'équivalent du cas de Koria qui planterait sans la modif et passerait avec.

[francoisfreitag]

Autrement très bien 👍