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

--- Comment #5 from Dawid Wróbel <m...@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 watching all bug changes.

Reply via email to