Remove ApplicationModel dependencies
ClosedPublic
Actions

Authored by ervin on Nov 12 2015, 1:36 PM.

Details

Reviewers

bensi
franckarrecot

Commits

R4:8e23f4e42ac7: Remove ApplicationModel dependencies

Summary

It's not doing anything interesting with them anymore anyway. Also means
we can create ApplicationModel instances directly, no need to go through
the dependency manager for it anymore.

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 1251.Nov 12 2015, 1:36 PM

ervin retitled this revision from to Remove ApplicationModel 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, 1:41 PM

bensi edited edge metadata.

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

Closed by commit R4:8e23f4e42ac7: Remove ApplicationModel dependencies (authored by ervin). · Explain WhyNov 12 2015, 1:44 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			src/presentation/applicationmodel.h (13 lines)
M			src/presentation/applicationmodel.cpp (13 lines)
M			src/renku/app/dependencies.cpp (7 lines)
M			src/renku/app/main.cpp (2 lines)
M			src/renku/kontact/part.cpp (2 lines)
M			src/zanshin/app/dependencies.cpp (7 lines)
M			src/zanshin/app/main.cpp (2 lines)
M			src/zanshin/kontact/part.cpp (2 lines)
M			tests/features/cuke-steps.cpp (2 lines)
M			tests/units/presentation/applicationmodeltest.cpp (46 lines)