[Kmymoney-devel] Review Request: Missing Payee in Category Split

[Allan Anderson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6820/
-----------------------------------------------------------

Review request for kmymoney.


Description
-------

When imported via QIF, the 'Pay To' is missing in category splits.

In the code, in MyMoneyStatementReader::processTransactionEntry(), in 'process
splits', I noticed 
if (acc.isAssetLiability()) {
        s2.setPayeeId(s1.payeeId());

I'm uncertain of the original purpose of the 'if' statement, but removing it fixes the problem.


This addresses bug 283339.
    https://bugs.kde.org/show_bug.cgi?id=283339


Diffs
-----


Diff: http://svn.reviewboard.kde.org/r/6820/diff/diff


Testing
-------

Tested with the original failing QIF file, and with several other files.  No apparent adverse effects.


Thanks,

Allan Anderson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20111104/353a3b50/attachment.html>