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

Project refactoring - part I #3855

Merged
merged 7 commits into from Dec 11, 2016
Merged

Conversation

@wonder-sk
Copy link
Member

@wonder-sk wonder-sk commented Dec 11, 2016

Large amount of changes here, but nothing super exciting here yet. Interesting bits:

  • QgsMapLayerRegistry merged into QgsProject
  • QgsMapSettings, QgsComposerMap switched from layer IDs to weak pointers with QPointer
  • started to remove QgsProject::instance()->addMapLayers(...) where not necessary in tests
  • QgsMapThemeCollection simplified the theme storage and API and switched to weak pointers
@nyalldawson

This comment has been minimized.

Copy link

@nyalldawson nyalldawson commented on src/core/qgsmaplayerlistutils.h in 5fc10d6 Dec 11, 2016

I'd like to see unit tests for these 3 methods

@nyalldawson
Copy link
Collaborator

@nyalldawson nyalldawson commented Dec 11, 2016

Looks great! Love it

@wonder-sk
Copy link
Member Author

@wonder-sk wonder-sk commented Dec 11, 2016

Thanks for the review! I will add the unit tests and merge...

@wonder-sk wonder-sk merged commit 20cdd4f into qgis:master Dec 11, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@nyalldawson
Copy link
Collaborator

@nyalldawson nyalldawson commented Dec 11, 2016

Really exciting stuff... Qgis 3 is going to be fantastic!

@m-kuhn
Copy link
Member

@m-kuhn m-kuhn commented Dec 12, 2016

Nice job, @wonder-sk !
Thanks for tackling this 🎉

@mhugo
Copy link

@mhugo mhugo commented Dec 13, 2016

Great ! Thanks !

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

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.