Differential D5474

Use security instead of account in KForecastView
ClosedPublic
Actions

Authored by wojnilowicz on Apr 16 2017, 1:15 PM.

Details

Reviewers
tbaumgart
Group Reviewers
KMyMoney
Commits
R261:61f43d5f3e29: Use security instead of account in KForecastView
Summary

Amount is monetary and account's fraction could be quantitative, so
amount could be displayed wrong, if the fractions aren't equal.

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 created this revision.Apr 16 2017, 1:15 PM
tbaumgart accepted this revision as: tbaumgart.Apr 17 2017, 9:59 AM
tbaumgart added a subscriber: tbaumgart.

Never stumbled across that one, but on the other hand I don't use the forecast with investments not at all. Code wise it looks logical to me.

This revision is now accepted and ready to land.Apr 17 2017, 9:59 AM
Closed by commit R261:61f43d5f3e29: Use security instead of account in KForecastView (authored by wojnilowicz). · Explain WhyApr 17 2017, 1:00 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkmymoney/views/kforecastview.cpp (2 lines)

Diff 13532

View Options

kmymoney/views/kforecastview.cpp

Loading...