3liz / qgis-openads-plugin

Extension QGIS pour une utilisation d'openADS
0 stars 2 forks source link

Paramètre "URL du dossier OpenADS" obligatoire #19

Open arno974 opened 2 years ago

arno974 commented 2 years ago

Lors du chargement des couches depuis la base, le paramètre "URL du dossier OpenADS" est spécifié comme optionnel. Mais, l'import des données sans que ce paramètre soit spécifié génére une erreur :

Votre projet ne contient pas la variable openads_url_ads, vous devez donc renseigner la valeur pour l'URL
L’exécution a échoué après 0.09 secondes

Capture d’écran du 2022-09-09 13-12-11

Gustry commented 2 years ago

Soit tu donnes l'URL dans les entrées de l'algorithme. L'entrée est en effet optionnel. Soit il doit y avoir une variable dans le projet QGIS openads_url_ads.

Si aucun des deux, alors en effet tu as le message d'erreur. Dans ton cas, tu as ni de variable de projet, ni d'URL lors du lancement de l'algo.

Il faut bien relire le message d'erreur ;-) Je suis d'accord, le message peut-être amélioré.

arno974 commented 2 years ago

Hello,

Si j'exécute le script de chargement des couches depuis la base sans spécifier de valeur dans URL du dossier OpenADS [Optionnel] (openads_url_ads) j'obtiens le message d'erreur ci-dessus. En sortie, les couches ne se chargent pas et la variable projet openads_url_ads n'est pas créée.

Si maintenant je mets une valeur (même un simple espace) dans URL du dossier OpenADS [Optionnel] tout se charge correctement et une variable projet openads_url_ads est bien créée.

À mon sens, l'exécution du traitement nécessite de spécifier une valeur pour URL du dossier OpenADS [Optionnel], cela même s'il est marqué comme Optionnel. Il doit y avoir probablement quelque part dans l'exécution du script une erreur qui est levée et qui bloque l'exécution si ce paramètre est manquant (alors même qu'il est spécifié comme optionnel).

Gustry commented 2 years ago

La variable openads_url_ads dans ton projet peut exister avant l’exécution du script. Maintenant que tu as la variable par exemple, tu peux relancer le script sans URL.

arno974 commented 2 years ago

Oui si je la relance maintenant que la variable existe tout fonctionne.

Ce que je veux dire c'est que dans un nouveau projet, si je lance le chargement des couches depuis la base sans avoir spécifié d'URL du dossier OpenADS j'obtiens forcément une erreur et le process n'est pas exécuté (pas de chargement des couches ni de création de variable). Pourtant ce paramètre est spécifié comme étant optionnel.

Tu es d'accord avec moi qu'il y a une erreur non ? Soit c'est optionnel soit ça ne l'est pas.