-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
[FIX] restore_registry in Odoo 16 #22
[FIX] restore_registry in Odoo 16 #22
Conversation
737d8f5
to
ec1a87f
Compare
8bafdff
to
a9ee140
Compare
a9ee140
to
e3e662b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review
I can confirm this solves problems like OCA/stock-logistics-barcode#450 |
@OCA/core-maintainers |
Sorry to be late to the party. I'm sick and missed this change 😓 |
@simahawk take care! |
@StefanRijnhart Could you check this as it seems to fail now: https://github.com/OCA/stock-logistics-workflow/actions/runs/4132016101/jobs/7140224436#step:8:244 |
It seems that this line is causing trouble: I'm wondering how if an attribute exists in a model, the following line that tries to delete it fails on retrieving it... |
Thanks for noting, I'll have a look. |
@rousseldenis #24 should fix it, I think. |
See odoo/odoo@cd12293
This new attribute is the source of truth for the base classes, and in setup_models (called further down in the modified code in this PR), the model's base classes are reset from it: https://github.com/odoo/odoo/blob/e1f06479a526c703ccabc441b1e194646206b966/odoo/models.py#L2728-L2730.
The test failure fixed by this PR can be inspected in https://app.travis-ci.com/github/OCA/odoo-test-helper/builds/258453331