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

[DB_Manager] allows to specify multiple columns as the primary for a query to PostGIS #2382

Merged
merged 7 commits into from
Oct 22, 2015

Conversation

SebDieBln
Copy link
Contributor

Since a4124da it is possible to specify a primary key spanning multiple columns for PostGIS layers based on a query.

This PR modifies the DB_Manager to support this new feature for queries to PostGIS. When building a query to SpatiaLite still only one column can be selected.

This PR also fixes the odd behavior of setting the selected columns back to the default when rerunning the query or reloading the fields, even if the selected columns still exist.

@brushtyler
Copy link
Contributor

@jef-n should this be considered a bug fix (not new feature) as it follows the merge you've done few days ago? Or should we wait until the end of feature freeze?

@m-kuhn m-kuhn assigned slarosa and brushtyler and unassigned slarosa Oct 18, 2015
brushtyler added a commit that referenced this pull request Oct 22, 2015
[DB_Manager] allows to specify multiple columns as the primary for a query to PostGIS (follow a4124da)
@brushtyler brushtyler merged commit ef8f3d3 into qgis:master Oct 22, 2015
@brushtyler
Copy link
Contributor

Merged, so to be consistent with provider behaviour. Thanks.

@SebDieBln SebDieBln deleted the MultiColumnPK_DbManagerQuery branch October 22, 2015 20:41
@SebDieBln SebDieBln mentioned this pull request Dec 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants