Skip to content

Commit add240c

Browse files
committed
[IMP] core: make author field mandatory in manifests
External contributors (and namely Odoo PS) forget to set the author field in their manifest file. The result is that the module is wrongly registered with `'author': 'Odoo S.A.'`. This work makes the field mandatory, with a warning when not set. It also offers an upgrade-code script to mass-rewrite all the manifests in order to set the author. task-4485983
1 parent 606b56c commit add240c

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

content/developer/howtos/accounting_localization.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -277,6 +277,7 @@ Basic :file:`__manifest__.py` file for such a module looks as following:
277277
"name": "COUNTRY - Accounting Reports",
278278
"category": "Accounting/Localizations/Reporting",
279279
"version": "1.0.0",
280+
"author": "Odoo S.A.",
280281
"license": "OEEL-1",
281282
"depends": [
282283
"l10n_XX", "account_reports"

content/developer/tutorials/server_framework_101/02_newapp.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ On the other hand, the ``__manifest__.py`` file must describe our module and can
6363
Its only required field is the ``name``, but it usually contains much more information.
6464

6565
Take a look at the
66-
`CRM file <https://github.com/odoo/odoo/blob/fc92728fb2aa306bf0e01a7f9ae1cfa3c1df0e10/addons/crm/__manifest__.py#L1-L67>`__
66+
`CRM file <https://github.com/odoo/odoo/blob/master/addons/crm/__manifest__.py#L1-L67>`__
6767
as an example. In addition to providing the description of the module (``name``, ``category``,
6868
``summary``, ``website``...), it lists its dependencies (``depends``). A dependency means that the
6969
Odoo framework will ensure that these modules are installed before our module is installed. Moreover, if

0 commit comments

Comments
 (0)