search

3liz / qgis-veloroutes_voies_vertes-plugin

Standard Covadis Véloroutes et Voies Vertes (3V)
5 stars 5 forks source link

Forcer le schéma ou non #59

Open Gustry opened 4 years ago

Gustry commented 4 years ago

Dans certains algos, on demande un schéma. Dans d'autres algos, on ne demande pas le schéma, donc il doit se nommer obligatoirement veloroute.

Gustry commented 3 years ago

Lié à #93, il y a 3 algos qui demandent le schéma : https://github.com/3liz/qgis-veloroutes_voies_vertes-plugin/search?q=SchemaWidgetWrapper

Ils utilisent SchemaWidgetWrapper qui n'existent plus dans QGIS >= 3.14.:

Afin de rendre le plugin compatible QGIS 3, je dois modifier ces wrapper, pour utiliser le widget Processing natif QGIS >= 3.14. QgsProcessingParameterDatabaseSchema https://qgis.org/api/classQgsProcessingParameterDatabaseSchema.html

La question, doit-on vraiment garder ce param ? Étant donné que l'installation, les migrations etc se font dans le schéma veloroutes.

Je suis plutôt pour supprimer. @rldhont ?

rldhont commented 3 years ago

Je pensais au contraire l'utiliser fortement ce paramètre schema. Je l'ai ajouter dans l'algo d'application des styles et je pensais l'ajouter à l'algo de mise à jour de la structure de la base.

Gustry commented 3 years ago

Ok, ca marche aussi évidement ;-)