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.

Sign up for GitHub

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

Jump to bottom

[16.0]ir_config_parameter_multi_company: impossible to install the module #754

Open
flotho opened this issue Jan 14, 2025 · 3 comments
Open

[16.0]ir_config_parameter_multi_company: impossible to install the module #754

flotho opened this issue Jan 14, 2025 · 3 comments
Labels
bug

Comments

@flotho
Copy link
Member

flotho commented Jan 14, 2025

Module

ir_config_parameter_multi_company

Describe the bug

Impossible to install the module

ile "/home/odoo_16/additionnal_repositories/OCA-multi-company/ir_config_parameter_multi_company/models/ir_config_parameter.py", line 19, in _get_param
    self.env.cr.execute(
  File "/home/odoo_16/server/odoo/sql_db.py", line 321, in execute
    res = self._obj.execute(query, params)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
psycopg2.errors.UndefinedColumn: ERREUR:  la colonne « company_id » n'existe pas
LIGNE 1 : ...onfig_parameter WHERE key = 'crm.pls_fields' AND (company_id...

To Reproduce

Affected versions:
16.0

Steps to reproduce the behavior:

  1. install crm
  2. try to install the module
@flotho flotho added the bug label Jan 14, 2025
@flotho flotho changed the title [16.0]ir_config_parameter_multi_company: impossible to install yhe module [16.0]ir_config_parameter_multi_company: impossible to install the module Jan 15, 2025
@flotho
Copy link
Member Author

flotho commented Jan 15, 2025

reproducted there : http://oca-multi-company-16-0-5c0a6cf80fa7.runboat.odoo-community.org/web#action=36&model=ir.module.module&view_type=kanban&cids=1&menu_id=5

@flotho
Copy link
Member Author

flotho commented Jan 15, 2025

ping @eLBati @antonioburic

@flotho
Copy link
Member Author

flotho commented Jan 15, 2025

AFAICS, we need a preinithook to create the field before loading the parameter.
Maybe we coud also make a dependency with base_multi_company ?

flotho added a commit to MindAndGo/multi-company that referenced this issue Jan 15, 2025
@flotho
[FIX]ir_config_parameter_multi_company: defensive for installation
6fc99ee 
during the installation process, the table parameter could be queried by other modules, for exemple CRM.
at this moment the filed is not available in the table causing OCA#754
This patch check the presence of the field as a defensive patch.
@flotho flotho mentioned this issue Jan 15, 2025
Open
flotho added a commit to MindAndGo/multi-company that referenced this issue Feb 12, 2025
@flotho
[FIX]ir_config_parameter_multi_company: defensive for installation
826ac59 
during the installation process, the table parameter could be queried by other modules, for exemple CRM.
at this moment the filed is not available in the table causing OCA#754
This patch check the presence of the field as a defensive patch.
flotho added a commit to MindAndGo/multi-company that referenced this issue Feb 12, 2025
@flotho
[FIX]ir_config_parameter_multi_company: defensive for installation
bea663d 
during the installation process, the table parameter could be queried by other modules, for exemple CRM.
at this moment the filed is not available in the table causing OCA#754
This patch check the presence of the field as a defensive patch.
flotho added a commit to MindAndGo/multi-company that referenced this issue Feb 12, 2025
@flotho
[FIX]ir_config_parameter_multi_company: defensive for installation
f295459 
during the installation process, the table parameter could be queried by other modules, for exemple CRM.
at this moment the filed is not available in the table causing OCA#754
This patch check the presence of the field as a defensive patch.

LINT

[FIX]ir_config_parameter_multi_company: defensive for installation
during the installation process, the table parameter could be queried by other modules, for exemple CRM.
at this moment the filed is not available in the table causing OCA#754
This patch check the presence of the field as a defensive patch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@flotho