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
1 stars 1 forks source link

Masquage provisoire des options de gestion des modèles #127

Closed alhyss closed 1 year ago

alhyss commented 1 year ago

J'ai l'impression que le paramètre qui décide le masquage des options de gestion des modèles n'est pas systématiquement pris en compte.

Il l'est pour l'écran d'accueil : masquage_gestion_modeles_ok

Il ne l'est semble-t-il plus dès qu'une fiche de métadonnées est affichée : masquage_gestion_modeles_pas_ok

Si je reviens sur l'écran d'accueil, le menu disparaît de nouveau.

Dans la fenêtre de configuration, ça a l'air plus compliqué que ça. Lorsque Plume est activé sur l'écran d'accueil, l'onglet Modèles n'est (comme il se doit) pas là. Si je charge une fiche, il apparaît. Si je reviens sur l'écran d'accueil, il reste, pour autant que je puisse voir.

masquage_gestion_modeles_pas_ok_onglet

WREATCHED commented 1 year ago

Le problème vient du fait que je n'ai pas désactivé (c'est pas bien de ma part) la ligne 81 du code dans bibli_plume.py qui permet quelque soit le profil utilisé, de retourner False et donc de ne pas avoir accès à la gestion des modèles

#==========================
def ifActivateRightsToManageModels(_self) : #Retourne un booléen pour activer le bouton pour la gestion des modèles
    mKeySql = queries.query_is_template_admin()
    r, zMessError_Code, zMessError_Erreur, zMessError_Diag = executeSql(_self, _self.mConnectEnCours, mKeySql, optionRetour = "fetchone")
    #r = False # for Test A supprimer
    return r
alhyss commented 1 year ago

@WREATCHED J'allais clore l'issue, mais est-ce que ce serait envisageable, tant que les développements sont en cours, de gérer ça avec un paramètre de configuration qui vaudrait False par défaut chez tout le monde et que toi seul basculerait sur True dans ton QGIS3.ini ? Ça éviterait d'avoir à y penser à chaque fois que tu pousses des modifications sur le Git.

Ce ne serait pas pour la v1.0, mais pour les prochaines versions jusqu'à ce que la gestion des modèles soit prête pour la diffusion.

WREATCHED commented 1 year ago

Complétement d'accord.

Solution

image

#==========================
def ifActivateRightsToManageModels(_self) : #Retourne un booléen pour activer le bouton pour la gestion des modèles
    mKeySql = queries.query_is_template_admin()
    r, zMessError_Code, zMessError_Erreur, zMessError_Diag = executeSql(_self, _self.mConnectEnCours, mKeySql, optionRetour = "fetchone")
    if not _self.ifActivateRightsToManageModelsFlag : r = False # Permet de chinter le controle des droits et dans tous les cas = False
    return r

et

        self.mIfActivateRightsToManageModels = False if not self.instalMetadata  else ifActivateRightsToManageModels(self)