Tuesday, March 14, 2023
10 changes · master
Resolved issues and error corrections
The German SKR03 accounting template now classifies account 1380 correctly in the balance sheet. This helps German companies rely on more accurate financial reporting and chart of accounts setup.
Original PR description
By checking the script checking if all the account supposed to be in the BS are. The account 1380 seems to be miss configured task-id: 3041738 --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
The Expense app was missing a required link to another component, which prevented businesses from installing it by itself. This fix adds the missing requirement so the app can be installed independently as expected.
Original PR description
Before this commit, it was not possible to install hr_expense alone. This commit enable the installation of standalone hr_expense by adding missing dependency. --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
This fixes an internal web test helper that could get stuck when a component was removed after an error. The change makes automated tests more reliable, helping developers catch issues without affecting regular users.
Original PR description
The `afternextrender` helper is meant to be used in some tests that want to do something after owl has completed all its work following a specific action. It is useful when the action may cause some…
The `afternextrender` helper is meant to be used in some tests that want to do something after owl has completed all its work following a specific action. It is useful when the action may cause some additional renders, so waiting for one nextTick is not enough. Unfortunately, it has a bug: it waits for Owl scheduler to be empty. But in some cases, when a component is destroyed because of an error, the scheduler task list will not be cleared until the next render completes. It is implemented that way in Owl to reduce the amount of work it does. But this means that in some testing scenarios, the afterNextRender promise will simply not be resolved, since there is no upcoming render to clear the tasklist. With this commit, we just avoid the issue by ignoring the fibers coming from destroyed component. Note that I think that it is not a good semantic for tests, as it is a little bit too "black-boxy". Ideally, we should rather wait for something in the DOM. There is a waitUntil helper somewhere that may be useful for this situation 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
The Mongolian balance sheet report has been updated to include accounts that were previously missing. This helps ensure the balance sheet balances correctly and provides more reliable financial reporting for Mongolian localization users.
Original PR description
When checking if the BS is balanced with a script, it seems that some accounts were missing from the BS. This PR correct that task-id: 3041738
The Italian financial reports now include accounts that were previously missing from the balance sheet report definitions. This helps companies using the Italian localization produce more complete and accurate balance sheet reporting.
Original PR description
By checking the script checking if all the account supposed to be in the BS are. Seems some account were present in the COA but not in the Balance sheet, this PR corrects that. task-id: 3041738
The Lithuanian balance sheet report has been adjusted so that all relevant accounts are included and amounts use the correct signs. This helps businesses get a balanced and more reliable financial statement for Lithuanian reporting.
Original PR description
When checking if the BS is balanced with a script,it seems that some accounts were missing in the BS and some sign were reverse, this pr corrects that task-id: 3041738
The Finnish balance sheet report has been adjusted so that missing accounts are included and reversed signs are corrected. This helps businesses produce more accurate balanced financial statements for Finnish reporting requirements.
Original PR description
When checking if the BS is balanced with a script,it seems that some accounts were missing in the BS and some sign were reverse, this pr corrects that task-id: 3041738
The Norwegian balance sheet report has been updated so all relevant accounts are included and signs are applied correctly. This helps ensure the balance sheet balances properly and provides more reliable statutory reporting.
Original PR description
When checking if the BS is balanced with a script,it seems that some accounts were missing in the BS and some sign were reverse, this pr corrects that task-id: 3041738
The Belgian balance sheet report now includes account 497, which was already part of the Belgian chart of accounts but missing from the report structure. This corrects an imbalance so Belgian financial statements present totals accurately.
Original PR description
Before this PR the belgian BS wasn't balanced since an account was missing in the BS (the account 497 was present in the COA but not in the BS). By adding the account the balance sheet is now correctly balanced. Task-id: 3041738
The German reporting setup now includes accounts that were previously present in the chart of accounts but missing from the balance sheet report. This helps ensure balance sheet reporting is more complete and accurate for German accounting users.
Original PR description
By checking the script checking if all the account supposed to be in the BS are. Seems some account were present in the COA but not in the Balance sheet, this PR corrects that. task-id: 3041738