Review Request 128061: Fix precision of split value

[Thomas Baumgart]

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

(Updated June 13, 2016, 12:11 p.m.)


Status
------

This change has been marked as submitted.


Review request for KMymoney.


Changes
-------

Submitted with commit 88040d6c3f43ee6781911087dd12801e890fd8b1 by Thomas Baumgart to branch master.


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


Repository: kmymoney


Description
-------

The problem described in bug 345655 is caused by a fraction of the splits value that is larger than the fraction of the accounts currency. This leads to the effect, that values are displayed with correct value but the underlying value used for calculations is not equal to the amount displayed. This will end up in KMyMoney presenting the user strange messages like 'Unassigned value of 0.00' which do not make sense. The difference is in reality between 0 and 0.01, e.g. 0.008.

The attached patch resolves this problem by rounding the numbers to the correct fraction of the referrenced account.


Diffs
-----

  kmymoney/mymoney/mymoneyfile.h 0fd558b 
  kmymoney/mymoney/mymoneyfile.cpp 568675c 
  kmymoney/mymoney/mymoneyfiletest.h 657ed39 
  kmymoney/mymoney/mymoneyfiletest.cpp 9903ed4 

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


Testing
-------

Test case is part of the patch. It uses the values provided in the sample attached to the bug 345655 entry.


Thanks,

Thomas Baumgart

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