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 cascading: fix unreported issue wih missing fids #42100

Merged
merged 3 commits into from
Mar 10, 2021

Conversation

elpaso
Copy link
Contributor

@elpaso elpaso commented Mar 8, 2021

  1. in WMS provider try to set fid from external getFeatureInfo JSON response
  2. in WMS server, make sure we transform integer FID to string

This fixes an unreported issue where getFeatureInfo from cascaded WMS in
JSON format have no feature IDs.

Funded by Gis3W https://www.gis3w.it

1. in WMS provider try to set fid from external getFeatureInfo JSON response
2. in WMS server, make sure we transform integer FID to string

This fixes an unreported issue where getFeatureInfo from cascaded WMS in
JSON format have no feature IDs.

Funded by Gis3W https://www.gis3w.it
@elpaso elpaso added Bug Either a bug report, or a bug fix. Let's hope for the latter! Server Related to QGIS server backport release-3_16 WMS data provider labels Mar 8, 2021
@github-actions github-actions bot added this to the 3.20.0 milestone Mar 8, 2021
src/providers/wms/qgswmsprovider.cpp Outdated Show resolved Hide resolved
src/server/services/wms/qgswmsrenderer.cpp Outdated Show resolved Hide resolved
elpaso and others added 2 commits March 9, 2021 08:10
Co-authored-by: Nyall Dawson <nyall.dawson@gmail.com>
@elpaso elpaso merged commit d140311 into qgis:master Mar 10, 2021
@elpaso elpaso deleted the bugfix-unreported-wms-cascading-fid branch March 10, 2021 07:57
github-actions bot pushed a commit that referenced this pull request Mar 10, 2021
…-fid

WMS cascading: fix unreported issue wih missing fids
github-actions bot pushed a commit that referenced this pull request Mar 10, 2021
…-fid

WMS cascading: fix unreported issue wih missing fids
nyalldawson pushed a commit that referenced this pull request Mar 11, 2021
…-fid

WMS cascading: fix unreported issue wih missing fids
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! Server Related to QGIS server WMS data provider
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants