Saturday, September 13, 2025
2 changes · saas-18.4
Resolved issues and error corrections
The journal report test was updated so it no longer depends on a payment reference staying blank in certain country-specific accounting setups. This helps keep automated builds stable when Czech localization is installed, reducing false test failures without changing user-facing accounting behavior.
Original PR description
test_document_data_basic was failing in builds with l10n_cz installed because - we set move_sales_2.payment_reference = '' in setUpClass and without l10n_cz it stays empty - but with l10n_cz installed it gets recomputed because of precompute=True on taxable_supply_date (which a stored computed field that triggers an extra write on account.move when the company is in CZ, and that write causes the compute graph to run again, and _compute_payment_reference fills the value back in) this commit solves this issue by not making assumptions about the payment_reference value and would use it as is in the generated data validation build_error-231479 Forward-Port-Of: odoo/enterprise#94554
This fixes a timing issue that could cause the two-step authentication flow to behave unreliably when the web client starts. It updates the related test synchronization so the problem is properly covered and less likely to reappear.
Original PR description
Should actually fix what #224161 tried to: I didn't notice that the events had been renamed (#220852) and since the test was still disabled (and the fixing PR was set to the master one) all the CIs ran without the test, thus not testing the fix in any way. It could have passed anyway as it's a non-deterministic issue, but it's at best a 50/50 that it succeeds so over 3 stagings I'd most likely have seen it... https://runbot.odoo.com/odoo/error/231316 https://runbot.odoo.com/odoo/error/181862 (the original of the same) Forward-Port-Of: odoo/odoo#226857