Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FW][FIX] account_group: Avoid parent_id recursion #156418

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Mar 4, 2024

Issue:

Infinite loop in Trial Balance.

Analyze:

The infinite loop is due to
account_reports.models.account_report.AccountReport.get_account_codes while group: loop if a recursion exist in group.parent_id there is an infinite loop.

Fix:

Ensure the no recursion constrains on parent_id in account_reports.

Related tasks:

opw-3665256
opw-3700368

Forward-Port-Of: #156321
Forward-Port-Of: #150171

Issue:
Infinite loop in Trial Balance.

Analyze:
The infinite loop is due to
account_reports.models.account_report.AccountReport.get_account_codes
`while group:` loop if a recursion exist in group.parent_id there is an
infinite loop.

Fix:
Ensure the no recursion constrains on parent_id in account_reports.

Related task:
opw-3665256

X-original-commit: 0ce8efb
@robodoo
Copy link
Contributor

robodoo commented Mar 4, 2024

@fw-bot
Copy link
Contributor Author

fw-bot commented Mar 4, 2024

This PR targets saas-17.1 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Mar 4, 2024
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Mar 4, 2024
robodoo pushed a commit that referenced this pull request Mar 5, 2024
Issue:
Infinite loop in Trial Balance.

Analyze:
The infinite loop is due to
account_reports.models.account_report.AccountReport.get_account_codes
`while group:` loop if a recursion exist in group.parent_id there is an
infinite loop.

Fix:
Ensure the no recursion constrains on parent_id in account_reports.

Related task:
opw-3665256

closes #156418

X-original-commit: 0ce8efb
Signed-off-by: Benjamin Hanquin (beha) <beha@odoo.com>
Signed-off-by: William André (wan) <wan@odoo.com>
@robodoo robodoo closed this Mar 5, 2024
@fw-bot fw-bot deleted the saas-17.1-15.0-account_account_report_cyclic_group_hierarchy-beha-COkU-fw branch March 19, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants