hesperides icon indicating copy to clipboard operation
hesperides copied to clipboard

Ajouter un query param only_in_model=true pour exclure les clefs-valeurs inutilisées de GET /properties

Open lenaing opened this issue 6 years ago • 3 comments

Hello,

Lorsque l'on récupère les properties d'un module utilisé par une plateforme via l'API, on récupère toutes les properties avec leur valorisation.

Malheureusement, actuellement on récupère aussi des propriétés qui ont été valorisées à un instant mais qui ne sont pas/plus définies dans le modèle du module. Ces properties sont marquées comme "supprimées" dans l'IHM.

Les identifier comme "supprimées" directement lorsqu'elles sont retournées serait un plus et permettrait d'éviter de devoir faire la comparaison en faisant un appel au model associé et vérifier qu'une propriété y est encore définie ou non.

lenaing avatar Nov 06 '18 15:11 lenaing

C'est un peu en lien avec https://github.com/voyages-sncf-technologies/hesperides-gui/issues/81

Qu'entends-tu par "Les identifier comme "supprimées" directement lorsqu'elles sont retournées" ? Qu'elles aient une propriété delete: true où qu'elles soient dissociées des autres dans l'arborescence JSON, par exemple ?

Lucas-C avatar Nov 06 '18 16:11 Lucas-C

Hello Lucas, dans mon cas avoir une propriété deleted: true serait suffisant. Je ne sais pas comment fait actuellement le front pour dissocier propriétés existantes et propriétés supprimées. En tout cas, embarquer cette information épargnerait, je suppose, de nouveaux appels pour connaître l'état d'une propriété.

lenaing avatar Nov 27 '18 12:11 lenaing

Après discussion avec @lenaing sur son besoin initial, je reviens sur la suggestion de mon commentaire précédent.

Pour précision, il ne s'agit pas réellement de propriétés "supprimées", mais d'anciennes valorisations de propriétés pour lesquelles le modèle (c-à-d les templates) on changés et qui sont donc désormais inutilisées.

Plutôt que de rajouter un champ booléen indicatif à la sortie JSON, je pense qu'il serait plus pertinent de permettre d'exclure ces propriétés via un query param. Depuis l'introduction des PropertyVisitors, cela devrait être relativement simple. J'ai modifié le nom de cette issue en conséquence.

Lucas-C avatar Oct 08 '19 12:10 Lucas-C