MTES-MCT / metadata-postgresql

Plume : gestion des métadonnées du patrimoine PostgreSQL
https://mtes-mct.github.io/metadata-postgresql/
GNU Affero General Public License v3.0
0 stars 1 forks source link

Import/export de modèles #109

Closed alhyss closed 7 months ago

alhyss commented 1 year ago

Fonctionnalité à ajouter, notamment pour faciliter la copie des modèles d'une base à une autre.

Principe :

alhyss commented 11 months ago

@WREATCHED J'ai implémenté les mécaniques d'import et d'export, tu dois pouvoir prendre la main là-dessus. Il y a essentiellement une fonction, plume.pg.template.dump_template_data, pour gérer l'export, et une classe du même module, TemplateQueryBuilder, pour construire les requêtes d'import. Tout ça est expliqué dans la documentation technique.

WREATCHED commented 8 months ago

Hello @alhyss Est-ce que tu peux me donner ton avis sur ça. image

alhyss commented 8 months ago

Pour l'export, est-ce que ça n'aurait pas été plus naturel d'utiliser l'arborescence Liste des modèles de l'onglet Ressources ? Au fond, elle contient exactement la même liste. Il pourrait y avoir une action Exporter les modèles sélectionnés dans le menu contextuel, et éventuellement une action Exporter tous les modèles pour tout exporter sans avoir à sélectionner ?

Je ne sais pas si l'import mérite un onglet non plus. Ça pourrait être une juste une action supplémentaire dans le menu Configuration, qui ouvre une boîte de dialogue avec la case à cocher + la sélection du fichier source ? Accessoirement, procéder comme ça aurait aussi l'avantage de t'éviter d'avoir à te préoccuper de rafraîchir quoi que ce soit dans la boîte de dialogue Gestion des modèles.

Juste des suggestions, bien sûr. Je ne sais pas ce qui est le plus simple à implémenter.

WREATCHED commented 8 months ago

Ce n'est pas une question d'implémentation, mais d'ergonomie. Le nombre d'information dans ressources est déjà assez riche et je ne trouve pas cela intuitif. Alors qu'un onglet supplémentaire me parait plus clair

WREATCHED commented 8 months ago

Après concertation Pour l'import

Pour l'export

WREATCHED commented 8 months ago

Problème détecté à l'import L'import des modèles pose des problèmes et plante Qgis (Plume) lorsque l'on a dans les champs jsonb, des valeurs décrites sous la forme de dictionnaire. psycopg2 ne parvient pas à les convertir. voir sql.py

Quelques captures d'écran image image

WREATCHED commented 8 months ago

Coté Qgis Tous les menus ont été implémentés

image

Coté bibliothèque Il ne reste que la partie @alhyss pour les champs jsonb