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.

Sign up for GitHub

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

Jump to bottom

[Bugfix][Server] WFS: enhancing the way DescribeFeatureType response is build #8159

Merged
merged 1 commit into from
Oct 12, 2018
Merged

[Bugfix][Server] WFS: enhancing the way DescribeFeatureType response is build #8159

merged 1 commit into from
Oct 12, 2018

Conversation

rldhont
Copy link
Contributor

@rldhont rldhont commented Oct 10, 2018 •
edited
Loading

Description

Even if the TypeName parameter is set, all the vector layer was build in DescribeFeatureType request. The commit fix it by using QgsWFSProjectParser::mapLayerFromTypeName

Checklist

  • Commit messages are descriptive and explain the rationale for changes
  • I have read the QGIS Coding Standards and this PR complies with them
  • This PR passes all existing unit tests (test results will be reported by travis-ci after opening this PR)
  • I have run the scripts/prepare-commit.sh script before each commit

@rldhont rldhont added the Bugfix label Oct 10, 2018
@rldhont rldhont added this to the 2.18.25 milestone Oct 10, 2018
@rldhont rldhont force-pushed the server-wfs-enhance-describeFeatureType-218 branch from a02e889 to 7892df4 Compare October 10, 2018 14:09
@rldhont rldhont force-pushed the server-wfs-enhance-describeFeatureType-218 branch from 7892df4 to fde6d93 Compare October 11, 2018 08:51
@rldhont
Copy link
Contributor Author

rldhont commented Oct 11, 2018

@sbrunner @pblottiere @elpaso gentle ping
needed for next 2.18 release

@rldhont rldhont merged commit 85fdf4e into qgis:release-2_18 Oct 12, 2018
@rldhont
Copy link
Contributor Author

rldhont commented Oct 12, 2018

Thanks @elpaso

@rldhont rldhont deleted the server-wfs-enhance-describeFeatureType-218 branch January 2, 2020 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elpaso elpaso elpaso approved these changes

@sbrunner sbrunner Awaiting requested review from sbrunner

@pblottiere pblottiere Awaiting requested review from pblottiere

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.18.25
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rldhont @elpaso