----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120153/#review71206 -----------------------------------------------------------
I've been drawn back to this because of another bug I'm looking at - "[Bug 322381] Fees(taxes) are deducted twice from dividend amount after QIF import". As mymoneystatementreader.cpp is involve also in QIF and CSV importing, I suspect there may be a side effect of this change. In mymoneyqifreader.cpp, I see the following - "// For historic reasons (coming from the OFX importer) the statement reader // expects the dividend with a reverse sign. So we just do that. tr.m_amount = -(amount - tr.m_fees);" Will not a corresponding change be required here? I suspect this may be in part contributing to Bug 322381. I've removed the -ve sign and it does partially correct this problem, but I need to look deeper because I suspect there is an earlier bug too. - Allan Anderson On Sept. 21, 2014, 9:33 a.m., Cristian Oneț wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120153/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2014, 9:33 a.m.) > > > Review request for KMymoney and Allan Anderson. > > > Bugs: 333522 > http://bugs.kde.org/show_bug.cgi?id=333522 > > > Repository: kmymoney > > > Description > ------- > > As the author of the hack states he's not sure why this was necessary. > This hack also caused a lot of workarounds in MyMoneyStatementReader > so it definitely should be removed. I've tried to keep the current > behaviour in MyMoneyStatementReader but since I'm not familiar with > investment transaction these should be double checked. > > BUG: 333522 > > > Diffs > ----- > > kmymoney/converter/mymoneystatementreader.cpp > 766d2151e17f07891f43e6d4d50861d40cbffe17 > kmymoney/plugins/ofximport/ofximporterplugin.cpp > 556cb4195ef1c5680ac50ce394f8bd9893fdcbdb > > Diff: https://git.reviewboard.kde.org/r/120153/diff/ > > > Testing > ------- > > Imported the OFX file attached to BUG 333522 in a checking account and in an > investemnt account. Allan please take a look if the investment transactions > part in MyMoneyStatementReader is OK. > > > Thanks, > > Cristian Oneț > >
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel