Daily updates from Odoo
Friday, August 15, 2025
7 changes
2 changes
Resolved issues and error corrections
This fix makes an internal bus testing helper more reliable by preventing a timing issue that could cause automated tests to fail unexpectedly. It also corrects how test errors are reported, helping teams identify real problems more clearly and keeping release validation stable.
Original PR description
This change addresses two issues in the `waitForChannels` test utility: - Fixes a race condition. The `failTimeout` variable was sometimes accessed before its `setTimeout` declaration had been executed. This happens when a fast bus event triggers the `onWebsocketEvent` handler, resulting in a reference error. The variable is now declared at the beginning of the function, making it available before any access (fixes runbot-223252). - Corrects error in def.reject. `message` only takes one param, and a string does not have a join method. Forward-Port-Of: odoo/odoo#222967
French tax reports now compare calculated totals with proper rounding, avoiding false mismatches caused by hidden decimal precision. This helps prevent incorrect warnings or validation issues when displayed amounts are effectively the same.
Original PR description
Currently, the code was checking for a strict equality between two sums. However, it may happen that there are many decimals that aren't displayed on the UI, hence the strict equality check fails. For instance `30201.63 != 30201.629999999997`. Instead, we now use the `float_compare` which takes care of rounding issues. opw-5000147 Forward-Port-Of: odoo/enterprise#92405 Forward-Port-Of: odoo/enterprise#92356
2 changes
Resolved issues and error corrections
This fix makes an internal bus testing helper more reliable by preventing timing-related failures when events arrive very quickly. It also corrects how errors are reported in the helper, reducing false test failures and improving development stability.
Original PR description
This change addresses two issues in the `waitForChannels` test utility: - Fixes a race condition. The `failTimeout` variable was sometimes accessed before its `setTimeout` declaration had been executed. This happens when a fast bus event triggers the `onWebsocketEvent` handler, resulting in a reference error. The variable is now declared at the beginning of the function, making it available before any access (fixes runbot-223252). - Corrects error in def.reject. `message` only takes one param, and a string does not have a join method. Forward-Port-Of: odoo/odoo#222967
This update reverses a recent accounting tax rounding change. It helps restore the previous behavior for tax total calculations, reducing the risk of unexpected invoice or tax amount differences.
Original PR description
As per https://github.com/odoo/odoo/pull/222759#issuecomment-3188749767 --- task-none --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#223053
1 change
Resolved issues and error corrections
Tax reports for France and Belgium now compare totals in a way that accounts for normal rounding differences. This prevents false mismatches when displayed amounts are effectively the same, improving reliability for users reviewing reports.
Original PR description
Currently, the code was checking for a strict equality between two sums. However, it may happen that there are many decimals that aren't displayed on the UI, hence the strict equality check fails. For instance `30201.63 != 30201.629999999997`. Instead, we now use the `float_compare` which takes care of rounding issues. opw-5000147 Forward-Port-Of: odoo/enterprise#92405 Forward-Port-Of: odoo/enterprise#92356
2 changes
Resolved issues and error corrections
This fix makes point-of-sale loyalty tests more reliable by ensuring the test customer appears near the top of the customer list. It helps prevent random test failures that can block automated validation, without changing customer-facing functionality.
Original PR description
Tours in POS rely on clicking on partners. But the POS interface does not load ALL records, meaning that depending on the test environment, the tested partner "Test Partner 2" can be too far down the list and not loaded properly. This commit fixes this by adding "AAA" in front of the name to move it up un the list order. It's a temporary fix to unblock the runbot and will be investigated more in details. Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This change reverses a recent accounting tax rounding adjustment. It helps restore the previous behavior for tax total calculations, reducing the risk of unexpected invoice or accounting totals.
Original PR description
As per https://github.com/odoo/odoo/pull/222759#issuecomment-3188749767 --- task-none --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr