om_account_asset: Thoughts on converting float fields to monetary?
kwoychesko opened this issue · 3 comments
kwoychesko commented
The asset module uses float fields for all currency fields and I'm wondering if we should change them to "monetary" fields?
Reasons:
- The current float fields are not rounded, and as a result the database stores values with many more decimal places than the company currency (see image below). Maybe this is okay & normal in the Odoo development world?
- A lot of work is done using float_compare and decimal precision, and I wonder if we made the fields monetary would some of this extra work go away?
Would love your thoughts.
Ken
odoomates commented
Thanks for bring the attention and yes we can make it Monetary for better working of the system
kwoychesko commented
kwoychesko commented
I'll close this--fix looks great!