Differential D9561

Don't use memo as payee for investment transactions
ClosedPublic
Actions

Authored by tbaumgart on Dec 30 2017, 11:07 AM.

Details

Reviewers
ostroffjh
Commits
R261:e469782e605a: Don't use memo as payee for investment transactions
Summary

During the import of an OFX investment transaction don't use the memo field as payee in case the payee information is not present. Use the PAYEEID or NAME field in that order only. In other cases leave the payee empty.

Test Plan

Compile
Run 'make test'
Import OFX download statement for investment account.
Check that payee field remains empty for newly imported transactions.

Diff Detail

Repository
R261 KMyMoney
Branch
fix-ofx-invest-import
Lint
No Linters Available
Unit
No Unit Test Coverage
tbaumgart requested review of this revision.Dec 30 2017, 11:07 AM
tbaumgart created this revision.
ostroffjh accepted this revision.Dec 31 2017, 3:58 PM

This does what it is supposed to. Ship it.

This revision is now accepted and ready to land.Dec 31 2017, 3:58 PM
Closed by commit R261:e469782e605a: Don't use memo as payee for investment transactions (authored by tbaumgart). · Explain WhyDec 31 2017, 4:05 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 24479

View Options

kmymoney/plugins/ofximport/ofximporterplugin.cpp

Loading...