From 5dd4c51f8ae66da2256587a53a91be9749846538 Mon Sep 17 00:00:00 2001 From: "Pedro M. Baeza" Date: Tue, 9 Jul 2024 11:21:58 +0200 Subject: [PATCH 1/2] [IMP] *: disable auto-install Odoo chooses to auto-install some modules that are debatable, so here in OCB we can revert this decission and let the Odoo admin to decide about them: - account_qr_code_sepa: this is just some tooling with no UI, so only modules using this tooling should depend and install it. - base_install_request: Feature for asking to install a module that doesn't make too much sense to be auto-installed in on premise maintained installations for avoiding false expectations. - iap: Odoo in-app purchases services should be optional, and more having a lot of dependant modules also being auto-installed (sms, snailmail, partner_autocomplete...). This way, we stop the chain from the beginning. - l10n_es_edi_facturae: Not everyone in Spain requires Facturae, and even if so, the OCA implementation is more suitable. - l10n_es_pos: The implementation of this need is faulty (creating one journal entry per PoS order), so the OCA one should be used instead. - mail_bot: It doesn't bring any real feature per se (only the annoying "Write an emoji" initial conversation). - pos_epson_printer: Not everyone using PoS has an Epson printer. - privacy_lookup: Privacy tool that not everyone uses. - sale_pdf_quote_builder: Optional feature. --- addons/account_qr_code_sepa/__manifest__.py | 2 +- addons/base_install_request/__manifest__.py | 2 +- addons/iap/__manifest__.py | 2 +- addons/l10n_es_edi_facturae/__manifest__.py | 2 +- addons/mail_bot/__manifest__.py | 2 +- addons/pos_epson_printer/__manifest__.py | 2 +- addons/privacy_lookup/__manifest__.py | 2 +- addons/sale_pdf_quote_builder/__manifest__.py | 2 +- 8 files changed, 8 insertions(+), 8 deletions(-) diff --git a/addons/account_qr_code_sepa/__manifest__.py b/addons/account_qr_code_sepa/__manifest__.py index 7d229cf32d04c0..85a6a817d77487 100644 --- a/addons/account_qr_code_sepa/__manifest__.py +++ b/addons/account_qr_code_sepa/__manifest__.py @@ -12,6 +12,6 @@ # any module necessary for this one to work correctly 'depends': ['account', 'base_iban'], - 'auto_install': True, + 'auto_install': False, 'license': 'LGPL-3', } diff --git a/addons/base_install_request/__manifest__.py b/addons/base_install_request/__manifest__.py index 89fa1dba16e49c..48da847f9ec4ac 100644 --- a/addons/base_install_request/__manifest__.py +++ b/addons/base_install_request/__manifest__.py @@ -9,7 +9,7 @@ Allow internal users requesting a module installation ===================================================== """, - 'auto_install': True, + 'auto_install': False, 'data':[ 'security/ir.model.access.csv', 'wizard/base_module_install_request_views.xml', diff --git a/addons/iap/__manifest__.py b/addons/iap/__manifest__.py index 893435895285cd..f61a24148d371c 100644 --- a/addons/iap/__manifest__.py +++ b/addons/iap/__manifest__.py @@ -20,7 +20,7 @@ 'views/iap_views.xml', 'views/res_config_settings.xml', ], - 'auto_install': True, + 'auto_install': False, 'assets': { 'web.assets_backend': [ 'iap/static/src/**/*.js', diff --git a/addons/l10n_es_edi_facturae/__manifest__.py b/addons/l10n_es_edi_facturae/__manifest__.py index fd5d9ec5e0bc82..dfc6f35f8db41c 100644 --- a/addons/l10n_es_edi_facturae/__manifest__.py +++ b/addons/l10n_es_edi_facturae/__manifest__.py @@ -38,6 +38,6 @@ ], 'post_init_hook': '_l10n_es_edi_facturae_post_init_hook', 'installable': True, - 'auto_install': ['l10n_es'], + 'auto_install': False, 'license': 'LGPL-3', } diff --git a/addons/mail_bot/__manifest__.py b/addons/mail_bot/__manifest__.py index 9b30c984c0de35..d6105955303e8d 100644 --- a/addons/mail_bot/__manifest__.py +++ b/addons/mail_bot/__manifest__.py @@ -8,7 +8,7 @@ 'summary': 'Add OdooBot in discussions', 'website': 'https://www.odoo.com/app/discuss', 'depends': ['mail'], - 'auto_install': True, + 'auto_install': False, 'installable': True, 'data': [ 'views/res_users_views.xml', diff --git a/addons/pos_epson_printer/__manifest__.py b/addons/pos_epson_printer/__manifest__.py index 58dc5464165ab6..32ddac2d5a8e58 100644 --- a/addons/pos_epson_printer/__manifest__.py +++ b/addons/pos_epson_printer/__manifest__.py @@ -19,7 +19,7 @@ 'views/pos_printer_views.xml', ], 'installable': True, - 'auto_install': True, + 'auto_install': False, 'assets': { 'point_of_sale._assets_pos': [ 'pos_epson_printer/static/src/**/*', diff --git a/addons/privacy_lookup/__manifest__.py b/addons/privacy_lookup/__manifest__.py index 97aec4de4073b9..d247f54afb8b6f 100644 --- a/addons/privacy_lookup/__manifest__.py +++ b/addons/privacy_lookup/__manifest__.py @@ -12,6 +12,6 @@ 'security/ir.model.access.csv', 'data/ir_actions_server_data.xml', ], - 'auto_install': True, + 'auto_install': False, 'license': 'LGPL-3', } diff --git a/addons/sale_pdf_quote_builder/__manifest__.py b/addons/sale_pdf_quote_builder/__manifest__.py index b639e4bb54b487..4db4031499f5d8 100644 --- a/addons/sale_pdf_quote_builder/__manifest__.py +++ b/addons/sale_pdf_quote_builder/__manifest__.py @@ -25,7 +25,7 @@ 'demo': [ 'data/sale_pdf_quote_builder_demo.xml', ], - 'auto_install': True, + 'auto_install': False, 'assets': { 'web.assets_backend': [ 'sale_pdf_quote_builder/static/src/js/**/*', From 5956fe7d8a3f017124b314e2318aa61e25c93b76 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9mi=20-=20Le=20Filament?= <30716308+remi-filament@users.noreply.github.com> Date: Fri, 12 Jul 2024 12:15:28 +0200 Subject: [PATCH 2/2] [IMP] google_gmail, web_unsplash: disable auto_install --- addons/google_gmail/__manifest__.py | 2 +- addons/web_unsplash/__manifest__.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/google_gmail/__manifest__.py b/addons/google_gmail/__manifest__.py index 9e4a8fa6ea6ab4..91ced8af2ee778 100644 --- a/addons/google_gmail/__manifest__.py +++ b/addons/google_gmail/__manifest__.py @@ -14,7 +14,7 @@ "views/ir_mail_server_views.xml", "views/res_config_settings_views.xml", ], - "auto_install": True, + "auto_install": False, "license": "LGPL-3", "assets": { "web.assets_backend": [ diff --git a/addons/web_unsplash/__manifest__.py b/addons/web_unsplash/__manifest__.py index 1dbcdd01392d64..d4a140202e79c4 100644 --- a/addons/web_unsplash/__manifest__.py +++ b/addons/web_unsplash/__manifest__.py @@ -10,7 +10,7 @@ 'data': [ 'views/res_config_settings_view.xml', ], - 'auto_install': True, + 'auto_install': False, 'assets': { 'web.assets_frontend': [ 'web_unsplash/static/src/frontend/unsplash_beacon.js',