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

Add geometryColumnName() method for WFS and OAPIF providers #60055

Closed
kannes opened this issue Jan 6, 2025 · 1 comment · Fixed by #60191
Closed

Add geometryColumnName() method for WFS and OAPIF providers #60055

kannes opened this issue Jan 6, 2025 · 1 comment · Fixed by #60191

Comments

@kannes
Copy link
Contributor

kannes commented Jan 6, 2025

Feature description

#59909 added https://qgis.org/pyqgis/master/core/QgsVectorDataProvider.html#qgis.core.QgsVectorDataProvider.geometryColumnName for some providers (ogr, hana, mssql, oracle, postgres, spatialite).

Would it be possible to extent this to the WFS and OAPIF providers too? (cc @nyalldawson)

For QGIS expression filters one can use $geometry. But when constructing SQL layer filters in Python that refer to the geometry, one needs to know the geometry column's name so this would greatly simply there.

Additional context

No response

@rouault
Copy link
Contributor

rouault commented Jan 20, 2025

for WFS, in #60191
for OAPIF, I don't think this makes sense currently. Nothing in the driver AFAICS and AFAIR knows about the geometry column name. This is somewhat implicit. Maybe some latest OGC API Features Part XXX have an explicit geometry column name, but it isn't yet there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants