----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6688/#review10143 -----------------------------------------------------------
Ship it! It looks alright, and afterall you are the QIF expert :). - Cristian On May 25, 2011, 12:10 a.m., Allan Anderson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/6688/ > ----------------------------------------------------------- > > (Updated May 25, 2011, 12:10 a.m.) > > > 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
