Lien vers le fichier de configuration distant en dur

[jbdesbas]

Le lien vers le fichier de configuration distant default_idg.json est en dur dans le code :

idg-qgis-plugin/plugin/idg/toolbelt/tree_node_factory.py

Lines 18 to 19 in b8cce67

	url="https://raw.githubusercontent.com/geo2france/idg-qgis-plugin/dev/plugin/idg/config"
	"/default_idg.json",

Il s'agit du fichier qui contient la listes des plateformes intégrées au plugin.

L'URL de ce fichier devrait être placer dans les variables de configuration pour faciliter les tests et réutilisation du plugin.

[bchartier]

Qu'est-ce que tu entends par variables de configuration ?
Les préférences qui sont lues par "preferences.py" et qui peuvent être éditées via "dlg_settings" ou autre chose ?
Si c'est ça, je suis parfaitement d'accord.

[jbdesbas]

Oui c'est bien ça 👍

[bchartier]

Ok, je vais faire cela.

[bchartier]

Je suis ennuyé par ce ticket.
Le fait de rendre éditable le chemin du fichier JSON dans le panneau de config du plugin rend certaines choses incohérentes :

Si on modifie l'URL alors il faut mettre à jour la liste des IDG listées dans le panneau de config.
Pour faire cela il faut télécharger le fichier de config et en faisant cela on écrase le précédent.
Du coup, les boutons "Apply" et "Cancel" du panneau de config n'ont plus tellement de sens :

• si on appuie sur "Cancel" alors on est incapable d'annuler car on n'a plus l'ancien fichier
• si on appuie sur "Ok" c'est cool mais on a déjà fait le téléchargement donc une bonne partie des opérations réalisées dans le panneau de config ont déjà été appliquées de fait

Je me demande donc si c'est opportun de mettre ce paramètre dans le panneau de config ou s'il faut gérer cela dans un autre menu. Une autre manière de faire serait de mettre en mémoire toute la config précédente pour être en mesure de la remettre en place si on annule.

Dans un premier temps, je vais faire un truc crado en téléchargeant le fichier ce qui appliquera de nouveaux paramètres sans que l'utilisateur clique sur "Apply" et sans qu'il puisse annuler.

Fait avec ce commit : f3e0ce1

[bchartier]

Résultat de la discussion d'aujourd'hui avec @jbdesbas :

• on retire le paramètre de l'URl du fichier de config de la fenêtre dlg_settings pour le laisser dans l'édition des paramètres avancés de QGIS
• du coup, plus besoin de se poser la question du rechargement en temps réel et de la cohérence avec les boutons "Annuler" et "Ok".

Action de @bchartier : retirer le champ d'édition de la boîte de dialogue.