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] fields: check for inconsistent many2many definitions #29323

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@rco-odoo
Member

rco-odoo commented Dec 6, 2018

Prevent two many2many fields from using the same config (table, column1, column2).

Fixes #24927.

@robodoo robodoo added the CI 🤖 label Dec 6, 2018

@C3POdoo C3POdoo added the RD label Dec 6, 2018

@rco-odoo rco-odoo force-pushed the odoo-dev:master-inherit-m2m-rco branch from 60553f7 to abd84dc Dec 7, 2018

@robodoo robodoo removed the CI 🤖 label Dec 7, 2018

@rco-odoo rco-odoo requested a review from KangOl Dec 7, 2018

@rco-odoo

This comment has been minimized.

Member

rco-odoo commented Dec 7, 2018

@robodoo rebase-ff

@KangOl

This comment has been minimized.

Contributor

KangOl commented Dec 11, 2018

Blocked by #29349 (should be forward-ported in following days)

rco-odoo added some commits Dec 4, 2018

[FIX] account: two m2m fields using the same table
The fields `analytic_tag_ids` and `second_analytic_tag_ids` on model
'account.reconcile.model' were accidentally using the same table and columns,
because of the automatic generation of the table name.  Make them explicitly
use distinct table names.

@rco-odoo rco-odoo force-pushed the odoo-dev:master-inherit-m2m-rco branch from abd84dc to d182ee3 Dec 17, 2018

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