----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120153/ -----------------------------------------------------------
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