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

[16.0] [IMP] shopinvader_api_cart: don't use self.env.norecompute #1500

Merged

Conversation

ivantodorovich
Copy link
Contributor

This context manager has been deprecated since many versions now, and it's not doing absolutely anything.

It was originally used to prevent computed fields to be recomputed when their dependencies are updated, but that's not the case anymore since the apparition of flush, where the recomputation is delayed.

See: https://github.com/odoo/odoo/blob/be4c1502/odoo/api.py#L823-L826
Also: odoo/odoo@9920f20e

This context manager has been deprecated since many versions now, and it's not
doing absolutely anything.

It was originally used to prevent computed fields to be recomputed when their
dependencies are updated, but that's not the case anymore since the apparition
of `flush`, where the recomputation is delayed.

See: https://github.com/odoo/odoo/blob/be4c1502/odoo/api.py#L823-L826
Also: odoo/odoo@9920f20e
@simahawk
Copy link
Contributor

/ocabot merge patch

@shopinvader-git-bot
Copy link

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1500-by-simahawk-bump-patch, awaiting test results.

@shopinvader-git-bot shopinvader-git-bot merged commit 9ce393e into shopinvader:16.0 Feb 13, 2024
3 checks passed
@shopinvader-git-bot
Copy link

Congratulations, your PR was merged at f833d61. Thanks a lot for contributing to shopinvader. ❤️

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