search

geo2france / idg-qgis-plugin

Plugin QGIS pour la consultation des données des différentes Infrastructure de Données Géographiques en France
GNU General Public License v2.0
14 stars 4 forks source link

Téléchargement du fichier de config par défaut uniquement si besoin … #73

Closed bchartier closed 2 months ago

bchartier commented 3 months ago

Pull request associée à #71 :

jbdesbas commented 3 months ago

Est-ce que cela peut impacter #68 (url du fichier de config dans les settings pour faciliter les test et ré-utilisation du plugin) ?

bchartier commented 3 months ago

Est-ce que cela peut impacter #68 (url du fichier de config dans les settings pour faciliter les test et ré-utilisation du plugin) ?

Oui et non. Oui : j'ai touché à la ligne de code indiquée en déplaçant le code dans PluginGlobals et en définissant des constantes :

    DEFAULT_CONFIG_FILE_NAME = "default_idg.json"
    DEFAULT_CONFIG_FILE_URL = (
        "https://raw.githubusercontent.com/geo2france/idg-qgis-plugin/dev/plugin/"
        f"idg/config/{DEFAULT_CONFIG_FILE_NAME}"
    )

Non : cela ne résout pas #68 car l'URL et le nom du fichier sont toujours en dur dans le code. Pour résoudre ce ticket il faudrait initialiser ces constantes par une information qui viendrait des settings du plugin.

En tout cas ça aide à résoudre #68 car le nom du fichier était défini à plusieurs endroit dans le code avec deux noms différents. Maintenant on a qu'un seul endroit à toucher.

jbdesbas commented 3 months ago

Salut @bchartier j'ai testé, aucun problème de mon côté. :+1: Petit point de vigilance, CONFIG_FILES_DOWNLOAD_AT_STARTUP ne contrôle que le téléchargement ou non du fichier json unique qui liste les plateformes (le pluriel est donc peut-être trompeur ?). (task1) Par contre les fichiers qgz/qgs des plateformes non-masqués sont téléchargées dans tous les cas (task2).

Merci pour la PR :smiley:

bchartier commented 3 months ago

Salut @bchartier j'ai testé, aucun problème de mon côté. 👍 Petit point de vigilance, CONFIG_FILES_DOWNLOAD_AT_STARTUP ne contrôle que le téléchargement ou non du fichier json unique qui liste les plateformes (le pluriel est donc peut-être trompeur ?). (task1) Par contre les fichiers qgz/qgs des plateformes non-masqués sont téléchargées dans tous les cas (task2).

Ok, je vais renommer cette variable et faire en sorte qu'on télécharge à la fois le fichier JSON et les projets QGIS (cf. #71). On mergera une fois que j'aurai mis à jour la PR si ça te va.

bchartier commented 3 months ago

Salut @bchartier j'ai testé, aucun problème de mon côté. 👍 Petit point de vigilance, CONFIG_FILES_DOWNLOAD_AT_STARTUP ne contrôle que le téléchargement ou non du fichier json unique qui liste les plateformes (le pluriel est donc peut-être trompeur ?). (task1) Par contre les fichiers qgz/qgs des plateformes non-masqués sont téléchargées dans tous les cas (task2).

Ok, je vais renommer cette variable et faire en sorte qu'on télécharge à la fois le fichier JSON et les projets QGIS (cf. #71). On mergera une fois que j'aurai mis à jour la PR si ça te va.

Voilà j'en est profité pour renommer DownloadAllConfigFilesAsync en DownloadAllIdgFilesAsync et pour changer la valeur par défaut du paramètre de téléchargement à True. J'espère que j'ai bien fait.

bchartier commented 2 months ago

Je m'aperçois que ma pull-request ne corrige pas grand-chose. 🫤 Le code n'est super clean:

bchartier commented 2 months ago

Tickets créés :