Skip to content

[IMP] Add error messages for deprecated '_check_recursion' and '_chec…#112

Open
lef-adhoc wants to merge 1 commit intoOCA:masterfrom
adhoc-dev:17.0-t-45018-lef2
Open

[IMP] Add error messages for deprecated '_check_recursion' and '_chec…#112
lef-adhoc wants to merge 1 commit intoOCA:masterfrom
adhoc-dev:17.0-t-45018-lef2

Conversation

@lef-adhoc
Copy link
Copy Markdown
Contributor

…k_m2m_recursion' methods, replaced by '_has_cycle'

More info here: odoo/odoo#162656

@LucasTran380381
Copy link
Copy Markdown

@@ -0,0 +1,6 @@
.py:
\b\w+\._check_recursion\(\):
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not work with _check_recursion with parameter

@@ -0,0 +1,6 @@
.py:
\b\w+\._check_recursion\(\):
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
\b\w+\._check_recursion\(\):
\b\w+\._check_recursion\([^)]*\):

…k_m2m_recursion' methods, replaced by '_has_cycle'

More info here: odoo/odoo#162656
@lef-adhoc
Copy link
Copy Markdown
Contributor Author

@LucasTran380381 There it is corrected

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants