alhyss
commented
2 years ago Il faut maintenant utiliser plume.pg.queries.query_plume_pg_check au lieu de plume.pg.queries.query_exists_extension pour générer la requête qui détermine s'il faut tenter d'importer les modèles depuis la base ou se contenter des modèles locaux.
Ça concerne donc plume.plume_ui.Ui_Dialog_plume.displayToolBar.
plume.pg.queries.query_plume_pg_check ne renvoie plus seulement un booléen, mais un tuple de valeurs. La première est toujours un booléen. Comme avant, si c'est True on continue et on va chercher les modèles sur le serveur, si c'est False il faudra se contenter des modèles locaux. Les valeurs suivantes permettent d'expliquer d'où vient le problème et pourraient être utilisées pour l'infobulle sur le bouton de sélection du modèle.
Plus de détails dans la documentation technique.
WREATCHED
Réponse 2
Oui, je te l'ai dit ce matin, je n'avais pas fait le
La prochaine version de Plume va aller de paire avec une nouvelle version de PlumePg et ne sera pas compatible avec les versions antérieures.
Pour l'heure, Plume se contente de vérifier que PlumePg est installée avant d'aller récupérer ses modèles, mais il faudra dorénavant vérifier aussi que la version installée est compatible avec la version de Plume dont dispose l'utilisateur.
Il y a aura clairement une condition du type "version de PlumePg supérieure ou égale à la version de référence x.x.x", mais je pense aussi qu'on aurait intérêt à fixer une borne supérieure, car je ne suis pas sûre qu'on sera toujours capable de garantir la rétro-compatibilité.
Par ailleurs, il serait utile de vérifier également si l'utilisateur a bien les droits nécessaires sur les objets de l'extension :
USAGEsur le schéma etSELECTsur les tables et vues. Si Asgard était actif sur la base au moment de l'activation de PlumePg, c'est normalement le cas, mais Asgard n'est pas présent sur toutes les bases... Il me semblerait plus propre d'en rester aux modèles locaux quand l'utilisateur n'a pas accès aux données, plutôt que d'avoir le genre d'erreur que j'ai eu lors de la démonstration. D'autant que, en pratique, une telle erreur empêcherait l'utilisateur de se servir de Plume jusqu'à ce qu'une personne habilitée lui donne les droits.Bref, je vais préparer une requête qui réalise tous ces tests.