fix(server): 🐛 remove administrationName from added sections if null

[matthieu-fesselier]

Contexte

Une fonctionnalité a été ajoutée récemment, un champ administrationName peut être défini sur les démarches (optionnel).

Toutes les démarches créées avant l’ajout de cette fonctionnalité n’ont pas ce champ en base.

Description du problème

Lors de la modification d’une démarche, certaines manipulation peuvent créer le champ à null (par exemple, j’ouvre la modal de l’administration puis la ferme sans rien changer).

Lorsque la nouvelle version est publiée, un calcul est effectué pour trouver la différence avec l’ancienne version. En ce qui concerne les éléments ajoutés, on compare simplement la liste des clés de la version originale avec la version actuelle.

Cas problématique → administrationName présent dans la nouvelle (à null) et pas dans l’ancienne. Le système ajoute donc cet élément “à revoir”. La traduction ne peut pas être publié tant qu’il n’est pas revu. L’élément ne peut pas être revu car étant null, il est masqué dans l’interface de traduction.

Solution proposée

1. Améliorer la méthode de détection des ajouts d’éléments entre versions : si l’élément ajouté est null, on ne le compte pas
2. Créer une migration qui va retirer l’ administrationName des éléments à revoir s’il est null dans la version originale

[linear]

RI-405 Les fiches traduites ne passent pas en "Publiées"

[kaaloo]

Love the detective work ! 👏🏼

P.S. Should we check for undefined too ?