Import investment splits in order matching manual entering
ClosedPublic

Authored by wojnilowicz on Feb 22 2017, 5:54 PM.

Details

Summary

Commit "Prevent overwrite of assetAccountSplit" uncovered bug in
detecting right category for fee split when importing investment
statement. Bug is caused because of some assumptions and different
approach to splits order between manual transaction entering and
transaction importing.
MyMoneyFile::addTransaction adds buy transaction with splits in
following order:

  1. manual entering

brokerage split
fee split
stock split

  1. importing

fee split
stock split
brokerage split

This patch unifies two approaches and makes it working for already
imported transactions.

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.Feb 22 2017, 5:54 PM
wojnilowicz abandoned this revision.Mar 4 2017, 8:58 AM

Submited.

This revision was automatically updated to reflect the committed changes.