> On May 27, 2011, 6:21 a.m., Thomas Baumgart wrote: > > I don't see a problem with it. I know that there is a method in MyMoneyFile > > that takes care of the colon separated account hierarchy and one that > > doesn't. It looks like you fixed the right places. > > > > This whole QIF thingy needs an overhaul anyway ... where is the time to do > > it?
Good. Thanks. Amen on both those comments! Skrooge seems a bit more forgiving, from my very limited experience. - Allan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6688/#review10149 ----------------------------------------------------------- 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
