[kmymoney] [Bug 420593] A sum of multiple rows selected is incorrect for securities with fraction > 100

[Dawid Wróbel]

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

--- Comment #5 from Dawid Wróbel <me at dawidwrobel.com> ---
Jack, yes, point 1. was supposed to say "investment account", not "brokerage".

> In the investment account, I don't think it makes any sense to show a sum of anything

This functionality was already in place and I find it useful. 

> unless the selected transactions are all for the same security. 

I think we can safely assume the users are smart enough to understand which
securities they select and what does the sum mean in this case. Besides, I
don't think you're right – I myself specifically had to sum the shares of
different securities after my brokerage automatically liquidated B class of
shares and repurchased C class. Either way, it's a different issue, unrelated
to this bug fix. 

> In that case, why not use use precision of the security, instead of just figuring what the transactions need. 

We don't need to figure out the precision at all when summing the values. The
precision only matters when converting the floating point value to a string for
displaying and the MyMoneyMoney::formatMoney() function can handle that
automatically – which my code change took advantage of.

-- 
You are receiving this mail because:
You are the assignee for the bug.