https://bugs.kde.org/show_bug.cgi?id=303026

Cristian Oneț <onet.crist...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|http://commits.kde.org/kmym |http://commits.kde.org/kmym
                   |oney/1506b0ccacb9d4efa2b6aa |oney/1a76540fbc936dd6e80cfe
                   |eb725c2e73f5507d83          |9faaeef7b35d0c166b

--- Comment #6 from Cristian Oneț <onet.crist...@gmail.com> ---
Git commit 1a76540fbc936dd6e80cfe9faaeef7b35d0c166b by Cristian Oneț.
Committed on 18/10/2014 at 12:57.
Pushed by conet into branch '4.7'.

Fix the validation of investment transactions.

Don't ajust the values to the smallest account fraction of the
currency. This will cause roundings that will lead to the reported
validation error.

Because at 0.5 AlkValue::RoundRound can round up/down depending on
the result it could be that one split it rounded up while the other
is rounded down leading to a 0.1 error while summing them.

Having a value which is more precise than the smallest account
fraction should not cause any issues since the UI should render
the value according to this fraction anyway.

If rounding is necessary a fix rounding method should be applied
for all splits which are going to be summed up.
(cherry picked from commit 1506b0ccacb9d4efa2b6aaeb725c2e73f5507d83)

M  +0    -8    kmymoney/dialogs/investtransactioneditor.cpp

http://commits.kde.org/kmymoney/1a76540fbc936dd6e80cfe9faaeef7b35d0c166b

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Reply via email to