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

when adding WMS/WFS layers from Browser use project CRS if it is supported by the server (fix #44325) #60476

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

alexbruy
Copy link
Contributor

@alexbruy alexbruy commented Feb 6, 2025

Description

When WMS and WFS layers added from Browser dock they use the first CRS supported by the server even if the project CRS is also available. This results in unnecessary on-the-fly reprojection. It is better to use project CRS if it is supported by the server and fallback to the first available CRS otherwise.

Fixes #44325.

@github-actions github-actions bot added this to the 3.42.0 milestone Feb 6, 2025
@alexbruy alexbruy force-pushed the browser-wms-wfs-default-crs branch 2 times, most recently from dc3cf1f to 927e779 Compare February 6, 2025 09:23
Copy link

github-actions bot commented Feb 6, 2025

🪟 Windows builds

Download Windows builds of this PR for testing.
Debug symbols for this build are available here.
(Built from commit f6afd7d)

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
(Built from commit f6afd7d)

src/providers/wfs/qgswfsdataitems.cpp Outdated Show resolved Hide resolved
src/providers/wms/qgswmsdataitems.cpp Outdated Show resolved Hide resolved
@alexbruy alexbruy force-pushed the browser-wms-wfs-default-crs branch from 927e779 to f6afd7d Compare February 6, 2025 10:17
@nyalldawson nyalldawson merged commit bf6268f into qgis:master Feb 7, 2025
31 checks passed
@alexbruy alexbruy deleted the browser-wms-wfs-default-crs branch February 7, 2025 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WFS GetFeature request is not using project CRS despite it is offered by WFS
3 participants