Remove ArtifactEditorModel dependencies
ClosedPublic
Actions

Authored by ervin on Nov 12 2015, 2:13 PM.

Details

Reviewers

bensi
franckarrecot

Commits

R4:94b5f992a0f6: Remove ArtifactEditorModel dependencies

Summary

This way it doesn't depend for a repository in particular. Instead we do
the integration via a couple of injected functions which poke the right
repository depending on the application we started.

Diff Detail

Repository

R4 Zanshin

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

ervin updated this revision to Diff 1266.Nov 12 2015, 2:13 PM

ervin retitled this revision from to Remove ArtifactEditorModel dependencies.

ervin updated this object.

ervin edited the test plan for this revision. (Show Details)

ervin added reviewers: bensi, franckarrecot.

bensi accepted this revision.Nov 12 2015, 2:21 PM

bensi edited edge metadata.

This revision is now accepted and ready to land.Nov 12 2015, 2:21 PM

Closed by commit R4:94b5f992a0f6: Remove ArtifactEditorModel dependencies (authored by ervin). · Explain WhyNov 12 2015, 2:24 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			src/presentation/artifacteditormodel.h (23 lines)
M			src/presentation/artifacteditormodel.cpp (40 lines)
M			src/renku/app/dependencies.cpp (13 lines)
M			src/zanshin/app/dependencies.cpp (16 lines)
M			tests/units/presentation/applicationmodeltest.cpp (3 lines)
M			tests/units/presentation/artifacteditormodeltest.cpp (180 lines)