Review Request 127711: Bug 361021 Add fee for buy and sell operation in the right way

Łukasz Wojniłowicz lukasz.wojnilowicz at gmail.com
Sun May 8 07:12:01 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127711/
-----------------------------------------------------------

(Updated May 8, 2016, 9:12 a.m.)


Status
------

This change has been marked as submitted.


Review request for KMymoney.


Changes
-------

Submitted with commit 7d7e398cf180b21f3ab9e90c5184290aced161a0 by ?ukasz Wojni?owicz to branch master.


Bugs: 361021
    http://bugs.kde.org/show_bug.cgi?id=361021


Repository: kmymoney


Description
-------

Fixes bug #361021. Fee should be added to buy and substracted from sell
operations. Difference between balance for banking and investment
account should include fees.


Diffs
-----

  kmymoney/converter/mymoneystatementreader.cpp 1634bbb 
  kmymoney/plugins/csvimport/investprocessing.cpp 3879819 

Diff: https://git.reviewboard.kde.org/r/127711/diff/


Testing
-------

I imported:
1) CSV test file from bug #361021 with fees,
2) CSV file from Allan Anderson without fees.


Thanks,

Łukasz Wojniłowicz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20160508/cd11e772/attachment-0001.html>


More information about the KMyMoney-devel mailing list