odoo/odoo#232186
Created by fw-bot
Merged
at c0c013044ea83f3ee424cf720e6589e97732398f
Statuses:
- legal/cla: Contributor License Agreement check
- ci/runbot: Odoo Test Suite
- ci/upgrade_enterprise: Test upgrades for enterprise master
- ci/template: Contact runbot team on discord for help.
- ci/style: Optional style check. Ignore it only if strictly necessary.
- ci/security: Required security check. Can only be ignored by security team.
- ci/l10n: (runtime 26s)
Linked pull requests
- label
- odoo-dev:19.0-18.0-fix-tax-rounding-las-471137-fw
- head
- f84bace61b1afb796b6242f89a904eec7f2fd591
- merged
- 1 month ago by Accounting, Laurent Smet (las)
| odoo/odoo | odoo/enterprise | |
|---|---|---|
| 18.0 | #228437 | #96809 |
| saas-18.2 | #231621 | #97207 |
| saas-18.3 | #231640 | #97220 |
| saas-18.4 | #232025 | #97443 |
| 19.0 | #232186 | #97553 |
| master | #232207 | #97567 |
[FIX] account: Ref rounding of tax_details for better handling of price-included
== Fix bug price-included ==
Suppose a line of 24.99 with a 20% tax price-included.
base: 24.99 / 1.2 = 20.825
tax: 20.825 * 0.2 = 4.165
If we round both, we get 20.83 + 4.17 = 25.0 != 24.99
== Split and simplify round_base_line_tax_details ==
Easier implementation of this method to be easier to understand and
easier to be customized (see PT override). Also, we now use the
aggregate methods to aggregate the amounts instead of doing that by hand.
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr