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

WMS Legend Image is flawed: WMS getLegendGraphic url's get double names/keys #22998

Closed
qgib opened this issue Jun 17, 2016 · 4 comments
Closed
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!

Comments

@qgib
Copy link
Contributor

qgib commented Jun 17, 2016

Author Name: Richard Duivenvoorde (@rduivenvoorde)
Original Redmine Issue: 15055
Affected QGIS version: master
Redmine category:unknown


IF a WMS service provides GetLegendGraphic url's in it's capabilities, QGIS tries to retrieve the legendimage and shows it in the layermanager.

Some services provide full url's, others only parts, that is why in current implementation there is a check IF certain query-names are available in the url-query and else are added:
https://github.com/qgis/QGIS/blob/master/src/providers/wms/qgswmsprovider.cpp#L2993

BUT certain services provide the url in lowercase key/value pairs, resulting in double key/value pairs in the url, resulting in error messages instead of images.

To test: add a layer from this service:
http://geoservices.knmi.nl/cgi-bin/RADNL_OPER_R___25PCPRR_L3.cgi?

I have already created a patch for it, and will do a pull request


@qgib
Copy link
Contributor Author

qgib commented Jun 17, 2016

Author Name: Richard Duivenvoorde (@rduivenvoorde)


  • 10031 was configured as legendflawed.png

  • legendflawed.png (Richard Duivenvoorde) - Legend Flawed message in layermanager

@qgib
Copy link
Contributor Author

qgib commented Jun 20, 2016

Author Name: Richard Duivenvoorde (@rduivenvoorde)


Fixed in changeset "543d5211332a885960a19b4a7ca7ff3296838a27".


  • status_id was changed from Open to Closed

@qgib
Copy link
Contributor Author

qgib commented Sep 22, 2017

Author Name: Jürgen Fischer (@jef-n)


  • category_id was configured as Unknown

@qgib qgib added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label May 25, 2019
@qgib qgib closed this as completed May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!
Projects
None yet
Development

No branches or pull requests

1 participant