----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6688/ -----------------------------------------------------------
Review request for kmymoney. Summary ------- The account contained in the 'L' record gets passed correctly from kmymoney/kmymoney/converter/mymoneyqifreader.cpp to mymoneystatementreader.cpp, as 'brokerageaccount'. There, however, the 'brokerageactid' does not get set correctly. The attached patch appears to fix the problem. With the patch applied, an 'L' record of 'L[Bank-Accounts:chq]' will generate automatically those two accounts and correctly transfer money into/from 'Bank-Accounts:chq' . However the asset account automatically created is of type 'checking'. So, to achieve what the OP requires, an account of type 'asset' first needs to be created. Then, the required result is obtained. If both the 'Bank-Accounts' and 'chq' accounts are created manually, the same result is achieved. This addresses bug 273950. https://bugs.kde.org/show_bug.cgi?id=273950 Diffs ----- /trunk/extragear/office/kmymoney/kmymoney/converter/mymoneystatementreader.cpp 1233480 Diff: http://svn.reviewboard.kde.org/r/6688/diff Testing ------- Tested with buy, sell and dividend transactions. Both QIF and CSV imports now work correctly. Thanks, Allan
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel