Differential D11407

Move online actions to KAccountsView
ClosedPublic
Actions

Authored by wojnilowicz on Mar 17 2018, 5:57 AM.

Details

Reviewers

tbaumgart

Group Reviewers

KMyMoney

Commits

R261:56f95eff8089: Move online actions to KAccountsView

Summary

Switching actions for online accounts is already handled in KAccountsView, so handle the actions there as well. This allows greater encapsulation, as KMyMoneyApp doesn't have to know which account is currently selected and KMyMoneyView doesn't need to call KMyMoneyApp.

Diff Detail

Repository

R261 KMyMoney

Lint

Automatic diff as part of commit; lint not applicable.

Unit

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

wojnilowicz requested review of this revision.Mar 17 2018, 5:57 AM

wojnilowicz created this revision.

Looks good to me.

This revision is now accepted and ready to land.Mar 17 2018, 8:51 PM

Closed by commit R261:56f95eff8089: Move online actions to KAccountsView (authored by wojnilowicz). · Explain WhyMar 18 2018, 9:42 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			kmymoney/kmymoney.h (17 lines)
M			kmymoney/kmymoney.cpp (264 lines)
M			kmymoney/views/kaccountsview.h (4 lines)
M			kmymoney/views/kaccountsview.cpp (197 lines)
M			kmymoney/views/kmymoneyview.cpp (3 lines)