Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TMS- le style au format QML n'est pas reconnu comme valide #428

Closed
IGNF-Xavier opened this issue Jun 13, 2024 · 9 comments · Fixed by #672
Closed

TMS- le style au format QML n'est pas reconnu comme valide #428

IGNF-Xavier opened this issue Jun 13, 2024 · 9 comments · Fixed by #672
Assignees
Labels
blocked Nécessite une évolution d'API, de config d'environnement, ou la fourniture de contenu bug Anomalie d'une fonctionnalité existante

Comments

@IGNF-Xavier
Copy link
Contributor

IGNF-Xavier commented Jun 13, 2024

LE qml fonctionne sous qgis
Style_OCSGE_Couverture.zip

mais une erreur est relevée

Image

entrepôt : a1e47cbf-b212-4a46-9b39-3fb3bb8f46d8/service/5bf8f1e0-c395-411d-9d3b-7ed31e185153
datasheetName=OCS-GE Données intermédiaires

@pprev94
Copy link
Contributor

pprev94 commented Jun 14, 2024

C'est le parser QML de geostyler qui plante ...

fonction QGISStyleParser.prototype.qmlSymbolizerLayerPropsToObject, qmlSymbolizer.prop n'existe pas

@IGNF-Xavier
Copy link
Contributor Author

Je suggère donc de retirer l'option qml et de concentrer plutôt les efforts sur le Sld qui bloque.

@IGNF-Xavier
Copy link
Contributor Author

@slafayIGN Vu que la gestion du QML n'est pas assuré pour des versions de QGIS supérieure à 3.28, peut-on faire sauter l'option QML
image

Puis réaffecter sur MVP-Future

@slafayIGN
Copy link
Contributor

Je trouve ça dommage de se priver de l'option alors que ça marche dans beaucoup de cas.

@IGNF-Xavier
Copy link
Contributor Author

Laissons mais il faudra s'attendre à des retours utilisateurs

@slafayIGN slafayIGN added the bug Anomalie d'une fonctionnalité existante label Nov 27, 2024
@slafayIGN
Copy link
Contributor

@cbrousseau1

Dans les 2 boites modales du site ou on peut sélectionner des fichiers de style au format QML, rajouter l'avertissement suivant en dessous de la ligne "Ajouter un fichier qml par couche présente dans votre service" uniquement lorsque c'est l'option QML qui est sélectionnée :
"Attention : les fichier QML générés avec une version de QGIS > 3.28 ne sont pas encore compatibles."

@slafayIGN slafayIGN added the blocked Nécessite une évolution d'API, de config d'environnement, ou la fourniture de contenu label Dec 11, 2024
@slafayIGN
Copy link
Contributor

Je repasse l'issue en blocked car une évolution indépendante sur geostyler-qgis-parser est maintenant nécessaire à la résolution complète du problème.

@slafayIGN
Copy link
Contributor

Les travaux sont en cours (par d'autres) depuis 2 semaines sur geostyler-qgis-parser. Quand une nouvelle release sera publiée nous pourrons tester des formats QML plus récents.

@slafayIGN
Copy link
Contributor

Une nouvelle version de geostyler-qgis-parser (v4.0.0 du 13/01/2025) règle des problèmes de compatibilité avec les QML générés par des versions de QGIS > 3.28 : https://github.com/geostyler/geostyler-qgis-parser/releases/tag/v4.0.0

Attention au changement cassant : le parser prend un paramètre qgisVersion, facultatif, en paramètre.

Idéalement l'utilisateur qui téléverse un QML ne devrait pas avoir à le fournir car la première balise contient généralement la version (<qgis version="3.28.7-Firenze" ...>). Mais il faudra des tests plus poussés dans le cas où on ne parvient pas à lire cette version du fichier d'origine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Nécessite une évolution d'API, de config d'environnement, ou la fourniture de contenu bug Anomalie d'une fonctionnalité existante
Projects
None yet
4 participants