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

[BUG] test_testing_utilities: Recursive o2m/m2o fields #31458

Closed
smetl opened this issue Feb 27, 2019 · 2 comments
Closed

[BUG] test_testing_utilities: Recursive o2m/m2o fields #31458

smetl opened this issue Feb 27, 2019 · 2 comments
Assignees
Labels
RD research & development, internal work

Comments

@smetl
Copy link
Contributor

smetl commented Feb 27, 2019

Steps to reproduce:

Make a model having both a one2many & many2one to itself.

Current behavior:

RecursionError when using the Form view emulator even both fields are readonly.

branch adding a test depicting this issue:
master...odoo-dev:master-form-recursive-o2m-las

@smetl smetl added the RD research & development, internal work label Feb 27, 2019
@xmo-odoo
Copy link
Collaborator

xmo-odoo commented Mar 7, 2019

ping @svs-odoo

xmo-odoo added a commit to odoo-dev/odoo that referenced this issue Apr 3, 2019
The extra setup probably affects any o2m whose edition view itself
contains an o2m, but most likely to blow up entirely on models with
some sort of tree structure (parent/child relationship): the SSF
eagerly loads and setups the o2m's view, and the o2m's o2m's, ... ad
infinitam.

A better / cleaner fix would be to set up the subview on-demand (and
possibly cache it), but the rest of the o2m stuff is unlikely to work
correctly recursively so just don't recurse the o2m view setup at all
for now.

fixes odoo#31458
@robodoo robodoo closed this as completed in 922d77a Apr 3, 2019
@smetl
Copy link
Contributor Author

smetl commented Apr 3, 2019

@xmo-odoo Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

No branches or pull requests

2 participants