dbarzin/mercator

Erreur sur l'écran d'édition de flux pour les sources et destinations de type Database

j-ova opened this issue · 6 comments

j-ova commented

Bonjour,

Lorsqu'on manipule des Flux, si la source ou la destination sont de type Base de donnée, la valeur n'est pas correctement chargée dans l'écran d'édition du flux.

En creusant les sources, j'ai détecté que l'objet Flux renvoie un identifiant nommé "DATABASE_XXX" et que l'écran d'édition tente de le mapper avec une valeur nommée "DB_XXX" :

Le chargement de la liste ne met donc jamais aucun élément en "selected" par défaut lorsqu'il s'agit d'une base de données :

Ce bug oblige à devoir re-sélectionner systématiquement la source et la destination d'un flux lorsqu'on l'ouvre en édition sous peine de perdre la donnée lorsqu'on sauvegarde la page.

Au-delà de ça, merci pour cet outil très fonctionnel qui va beaucoup m'aider à formaliser et maintenir la cartographie de notre SI !

dbarzin commented

Merci, c'est corrigé ici : 716478e

j-ova commented

Super réactivité, merci :)

  • 👍1
j-ova commented

@dbarzin je ne sais pas si j'ai mal fait quelque chose mais la mise à jour a l'air d'avoir appliqué un patch de db qui n'a pas de traduction dans le code.

Le fichier /database/migrations/2024_03_19_195927_contracts.php a été appliqué à mon redémarrage d'instance mais le code php cherchait toujours la colonne "importance" qui a été renommée dans ce patch.

J'ai un peu galéré à rollback car je ne connais pas les mécaniques php/Laravel.
J'ai tenté des choses manuellement et j'ai fini par y arriver. J'ai découvert au milieu du chemin qu'un "php artisan migrate:rollback --path=./database/migrations/2024_03_19_195927_contracts.php" aurait pu suffire mais j'avais cassé trop de choses déjà pour que ça marche...

dbarzin commented

Cette modification n'aurait pas dû passer déjà passer en production.

dbarzin commented

Corrigé ici : 4f3f17c
Vous pouvez réappliquer la migration

j-ova commented

Migration réappliquée avec succès. Merci !