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

Try to use the project CRS when adding a layer from the browser #1310

Merged
merged 2 commits into from
Dec 23, 2014
Merged

Try to use the project CRS when adding a layer from the browser #1310

merged 2 commits into from
Dec 23, 2014

Conversation

manisandro
Copy link
Member

Currently, when a layer is added from the browser, the first valid CRS is used. With this patch, the project CRS is used if possible. Also, the last selected image format is used if possible.

At the same time, proper escaping and unescaping routines are introduced, removing the need for the if ( parts[3] == "PG" ) workaround.

Funded by Sourcepole QGIS Enterprise.

@mhugent
Copy link
Contributor

mhugent commented Apr 10, 2014

assigned to @wonder-sk

@wonder-sk
Copy link
Member

Code looks good to me, but I haven't had time to test it.

Currently the PR cannot be automatically merged. When merging, please update SIP bindings and note that the new methods were added in 2.4. @mhugent ?

@wonder-sk wonder-sk assigned mhugent and unassigned wonder-sk May 23, 2014
@manisandro
Copy link
Member Author

Since qgisapp is not exposed in the python bindings, no updates to the sip files should be necessary.

@wonder-sk
Copy link
Member

There are also changes to QgsDataItem and QgsMimeDataUtils which belong to the core library and they are exposed to Python...

@manisandro
Copy link
Member Author

Ups yeah forgot about those. Fixed.

@NathanW2
Copy link
Member

@wonder-sk does this look good to you?

@manisandro
Copy link
Member Author

Any chance this can get merged?

mhugent added a commit that referenced this pull request Dec 23, 2014
Try to use the project CRS when adding a layer from the browser
@mhugent mhugent merged commit 1bce9cc into qgis:master Dec 23, 2014
@manisandro manisandro deleted the adjust_crs_drop_from_browser branch December 23, 2014 13:08
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.

4 participants