isogeo / isogeo-plugin-qgis

Isogeo plugin for QGIS
http://help.isogeo.com/qgis/
GNU General Public License v3.0
16 stars 5 forks source link

Gestion des layers de type "table" #462

Closed SimonSAMPERE closed 5 months ago

SimonSAMPERE commented 1 year ago

Adaptation à ce ticket de l'API : https://github.com/isogeo/isogeo-api/issues/107

A voir si on a des modifications à effectuer mais globalement ça implique de prendre en compte le nouvel attribut type des ServiceLayers.

L'ajout d'une table de WFS entraîne une erreur Python :

modules\layer\geo_service.py", line 676, in build_wfs_url
              srs_id = [
             IndexError: list index out of range

Pour les EFS ça semble pas fonctionne non plus (même dans QGSI 3.22) : image

Apparemment QGIS ne permet pas d'ajouter les tables d'EMS ou d'EFS, il faudrait donc ne pas les proposer dans la liste déroulante.

SimonSAMPERE commented 1 year ago

@leodarengosse j'ai commencé à me préparer des métadonnées pour le dev donc j'ai créé 4 fiches de services pour l'EFS, l'EMS, le WFS et le WMS. J'ai utilisé le service EMS_EFS_WMS_WFS à chaque fois parce qu'il me semble qu'il a des groupe de couches et des "table" de service.

Je me suis rendu compte que les tables ont été créées uniquement dans la fiche EMS.

Quand je lis le ticket de l'API je trouve pas à quel endroit on a spécifié les formats de services pouvant diffuser des tables.

Tu peux m'indiquer si c'est un bug, si on veut effectivement gérer les tables de service uniquement pour les services EMS ou si c'est juste implémenté dans l'API seulement pour les EMS ? Sachant que côté Scan AGS on l'a implémenté pour les EFS, EMS et WFS

SimonSAMPERE commented 1 year ago

Recette

Vérifier qu'on peut ajouter les tables WFS et EFS avec le plugin Isogeo dans QGIS 3.22 et 3.28.

jgrosmaire commented 1 year ago

Recette

SimonSAMPERE commented 1 year ago

@jgrosmaire au temps pour moi, je me suis emmêlé les bâtons dans les roues, apparemment QGIS ne propose ni les tables d'EFS ni celles d'EMS.

Le nouveau comportement à tester est donc le suivant :

Désolé pour le dérangement.

jgrosmaire commented 1 year ago

Recette

SimonSAMPERE commented 5 months ago

https://github.com/isogeo/isogeo-plugin-qgis/releases/tag/3.5.0