-----------------------------------------------------------
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

Reply via email to