KO | EN

GitLite

Minimalist Open Source Repository Search Engine

gitlite — search

#dsh #dsh-plugin #typescript #developer-tools #claude-code #ai-agent #codex #deepseek-harness #ai-agents #cli #react #mcp

pylint-odoo

★ 169 Open GitHub ↗

Odoo plugin for Pylint

// repository documentation

Was this content helpful?

★ 0 (0 ratings)

[//]: # (start-badges)

[![Build Status](https://github.com/OCA/pylint-odoo/actions/workflows/test.yml/badge.svg?branch=main)](https://github.com/OCA/pylint-odoo/actions/workflows/test.yml?query=branch%3Amain)
[![codecov](https://codecov.io/gh/OCA/pylint-odoo/branch/main/graph/badge.svg)](https://codecov.io/gh/OCA/pylint-odoo)
[![code-style-black](https://img.shields.io/badge/code%20style-black-000000.svg)](https://github.com/psf/black)
[![version](https://img.shields.io/pypi/v/pylint-odoo.svg)](https://pypi.org/project/pylint-odoo)
[![pypi-downloads-monthly](https://img.shields.io/pypi/dm/pylint-odoo.svg?style=flat)](https://pypi.python.org/pypi/pylint-odoo)
[![supported-versions](https://img.shields.io/pypi/pyversions/pylint-odoo.svg)](https://pypi.org/project/pylint-odoo)
[![wheel](https://img.shields.io/pypi/wheel/pylint-odoo.svg)](https://pypi.org/project/pylint-odoo)
[![commits-since](https://img.shields.io/github/commits-since/OCA/pylint-odoo/v10.0.9.svg)](https://github.com/OCA/pylint-odoo/compare/v10.0.9...main)

[//]: # (end-badges)

# Pylint Odoo plugin

Enable custom checks for Odoo modules.

[//]: # (start-checks)

Short Name | Description | Code
--- | --- | ---
attribute-deprecated | attribute "%s" deprecated | W8105
attribute-string-redundant | The attribute string is redundant. String parameter equal to name of variable | W8113
bad-builtin-groupby | Used builtin function `itertools.groupby`. Prefer `odoo.tools.groupby` instead. More info about https://github.com/odoo/odoo/issues/105376 | W8155
category-allowed | Category "%s" not allowed in manifest file. | C8114
category-allowed-app | Category "%s" not allowed in manifest file for modules with price. | C8117
consider-merging-classes-inherited | Consider merging classes inherited to "%s" from %s. | R8180
context-overridden | Context overridden using dict. Better using kwargs `with_context(**%s)` or `with_context(key=value)` | W8121
deprecated-inselect-operator | The domain operator %r is deprecated in Odoo 18.0+. Use 'in' SQL or 'not in' SQL instead. More info at https://github.com/odoo/odoo/pull/171371 | E8149
deprecated-name-get | 'name_get' is deprecated. Use '_compute_display_name' instead. More info at https://github.com/odoo/odoo/pull/122085. | E8146
deprecated-odoo-model-method | %s has been deprecated by Odoo. Please look for alternatives. | W8160
deprecated-self-cr | Use "self.env.cr" instead of "self._cr" (deprecated since 19.0) | W8165
development-status-allowed | Manifest key development_status "%s" not allowed. Use one of: %s. | C8111
except-pass | pass into block except. If you really need to use the pass consider logging that exception | W8138
external-request-timeout | Use of external request method `%s` without timeout. It could wait for a long time | E8106
inheritable-method-lambda | Use `%s=lambda self: self.%s()` to preserve inheritability. More info at https://github.com/OCA/odoo-pre-commit-hooks/issues/126 | E8148
inheritable-method-string | Use string method name `"%s"` to preserve inheritability. More info at https://github.com/OCA/odoo-pre-commit-hooks/issues/126 | E8147
invalid-commit | Use of cr.commit() directly - More info https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#never-commit-the-transaction | E8102
invalid-email | Invalid email "%s" | R8181
license-allowed | License "%s" not allowed in manifest file. | C8105
manifest-author-string | The author key in the manifest file must be a string (with comma separated values) | E8101
manifest-behind-migrations | Manifest version (%s) is lower than migration scripts (%s) | E8145
manifest-data-duplicated | The file "%s" is duplicated in lines %s from manifest key "%s" | W8125
manifest-deprecated-key | Deprecated key "%s" in manifest file | C8103
manifest-external-assets | Asset %s should be distributed with module's source code. More info at https://httptoolkit.com/blog/public-cdn-risks/ | W8162
manifest-maintainers-list | The maintainers key in the manifest file must be a list of strings | E8104
manifest-required-author | One of the following authors must be present in manifest: %s | C8101
manifest-required-key | Missing required key "%s" in manifest file | C8102
manifest-required-key-app | Missing required key "%s" in manifest file for modules with price. | C8119
manifest-summary-multiline | Summary in manifest file should be a one-line short description, found newline character | C8120
manifest-superfluous-key | Manifest superfluous key "%s". It is the same as the default value: %s. Better remove it | C8116
manifest-version-format | Wrong Version Format "%s" in manifest file. Regex to match: "%s" | C8106
method-compute | Name of compute method should start with "_compute_" | C8108
method-inverse | Name of inverse method should start with "_inverse_" | C8110
method-required-super | Missing `super` call in "%s" method. | W8106
method-search | Name of search method should start with "_search_" | C8109
missing-odoo-file | Missing %s file | C8115
missing-odoo-file-app | Missing %s file for modules with price | C8118
missing-readme | Missing ./README.rst file. Template here: %s | C8112
missing-return | Missing `return` (`super` is used) in method %s. | W8110
no-raise-unlink | No exceptions should be raised inside unlink() functions | E8140
no-search-all | Using an empty domain `%s([])` without a `limit` will load all records, may impact performance. | W8163
no-wizard-in-models | No wizard class for model directory. See the complete structure https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#complete-structure | C8113
no-write-in-compute | Compute method calling `write`. Use `update` instead. | E8135
odoo-addons-relative-import | Same Odoo module absolute import. You should use relative import with "." instead of "odoo.addons.%s" | W8150
odoo-exception-warning | `odoo.exceptions.Warning` is a deprecated alias to `odoo.exceptions.UserError` use `from odoo.exceptions import UserError` | R8101
prefer-env-translation | Better using self.env._ More info at https://github.com/odoo/odoo/pull/174844 | W8161
print-used | Print used. Use `logger` instead. | W8116
prohibited-method-override | Prohibited override of "%s" method. | W8107
renamed-field-parameter | Field parameter "%s" is no longer supported. Use "%s" instead. | W8111
resource-not-exist | File "%s": "%s" not found. | F8101
sql-injection | SQL injection risk. Use parameters if you can. - More info https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#no-sql-injection | E8103
super-method-mismatch | `super().%s` mismatch but defined method is `%s` | W8164
test-folder-imported | Test folder imported in module %s | E8130
translation-contains-variable | Translatable term in "%s" contains variables. Use %s instead | W8115
translation-field | Translation method _("string") in fields is not necessary. | W8103
translation-format-interpolation | Use %s formatting in odoo._ functions | W8302
translation-format-truncated | Logging format string ends in middle of conversion specifier | E8301
translation-fstring-interpolation | Use %s formatting in odoo._ functions | W8303
translation-injection | Do not use str.format on translation methods. Use placeholders instead. Reference: https://lucumr.pocoo.org/2016/12/29/careful-with-str-format/ | E8151
translation-not-lazy | Use %s formatting in odoo._ functions | W8301
translation-positional-used | Translation method _(%s) is using positional string printf formatting with multiple arguments. Use named placeholder `_("%%(placeholder)s")` instead. | W8120
translation-required | String parameter on "%s" requires translation. Use %s%s(%s) | C8107
translation-too-few-args | Not enough arguments for odoo._ format string | E8306
translation-too-many-args | Too many arguments for odoo._ format string | E8305
translation-unsupported-format | Unsupported odoo._ format character %r (%#02x) at index %d | E8300
use-vim-comment | Use of vim comment | W8202
website-manifest-key-not-valid-uri | Website "%s" in manifest key is not a valid URI. %s | W8114

[//]: # (end-checks)

# Install

You do not need to install manually if you use pre-commit-config

But if you even need to install it

pip install pylint-odoo

# Usage pre-commit-config.yaml

Add to your ".pre-commit-config.yaml" configuration file the following input

```yaml
    - repo: https://github.com/OCA/pylint-odoo
        rev: v10.0.9 # may be a tag or commit hash
        hooks:
        # Add to your .pylintrc file:
        # [MASTER]
        # load-plugins=pylint_odoo
        - id: pylint_odoo
```

# Usage

pylint --load-plugins=pylint_odoo -e odoolint path/to/test

or use configuration file you can generate the OCA one using the following template repository:

https://github.com/OCA/oca-addons-repo-template

Then running

pylint --rcfile=.pylintrc path/to/test

Example to test only pylint_odoo checks:

pylint --load-plugins=pylint_odoo -d all -e odoolint {ADDONS-PATH}/*

There are checks only valid for a particular Odoo version
To know what version of odoo are you running pylint needs the parameter

pylint --load-plugins=pylint_odoo --valid-odoo-versions={YOUR_ODOO_VERSION}

with particular odoo version e.g. `"16.0"`

Check valid only for odoo >= 18.0

prefer-env-translation

Checks valid only for odoo >= 14.0

translation-format-interpolation
    translation-format-truncated
    translation-fstring-interpolation
    translation-not-lazy
    translation-too-few-args
    translation-too-many-args
    translation-unsupported-format

Checks valid only for odoo <= 13.0

translation-contains-variable

[//]: # (start-example)

# Examples

* attribute-deprecated

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L200 attribute "_columns" deprecated
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L201 attribute "_defaults" deprecated
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L202 attribute "length" deprecated

* attribute-string-redundant

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L276 The attribute string is redundant. String parameter equal to name of variable
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L280 The attribute string is redundant. String parameter equal to name of variable
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L284 The attribute string is redundant. String parameter equal to name of variable

* bad-builtin-groupby

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L205 Used builtin function `itertools.groupby`. Prefer `odoo.tools.groupby` instead. More info about https://github.com/odoo/odoo/issues/105376
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L206 Used builtin function `itertools.groupby`. Prefer `odoo.tools.groupby` instead. More info about https://github.com/odoo/odoo/issues/105376

* category-allowed-app

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L6 Category "No valid for odoo.com/apps" not allowed in manifest file for modules with price.

* consider-merging-classes-inherited

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/model_inhe2.py#L11 Consider merging classes inherited to "res.company" from testing/resources/test_repo/broken_module/models/model_inhe1.py:8:4, testing/resources/test_repo/broken_module/models/model_inhe2.py:7:4.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/model_inhe2.py#L19 Consider merging classes inherited to "res.partner" from testing/resources/test_repo/broken_module/models/model_inhe2.py:15:4.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/model_inhe2.py#L56 Consider merging classes inherited to "stock.warehouse.orderpoint" from testing/resources/test_repo/broken_module/models/model_inhe1.py:19:4.

* context-overridden

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L314 Context overridden using dict. Better using kwargs `with_context(**{'overwrite_context': True})` or `with_context(key=value)`
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L316 Context overridden using dict. Better using kwargs `with_context(**ctx)` or `with_context(key=value)`
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L318 Context overridden using dict. Better using kwargs `with_context(**ctx2)` or `with_context(key=value)`

* deprecated-inselect-operator

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L1151 The domain operator 'inselect' is deprecated in Odoo 18.0+. Use 'in' SQL or 'not in' SQL instead. More info at https://github.com/odoo/odoo/pull/171371
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L1152 The domain operator 'not inselect' is deprecated in Odoo 18.0+. Use 'in' SQL or 'not in' SQL instead. More info at https://github.com/odoo/odoo/pull/171371
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L1153 The domain operator 'INSELECT' is deprecated in Odoo 18.0+. Use 'in' SQL or 'not in' SQL instead. More info at https://github.com/odoo/odoo/pull/171371

* deprecated-name-get

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/twelve_module/models.py#L7 'name_get' is deprecated. Use '_compute_display_name' instead. More info at https://github.com/odoo/odoo/pull/122085.

* deprecated-odoo-model-method

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L186 fields_view_get has been deprecated by Odoo. Please look for alternatives.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/eleven_module/models.py#L17 fields_view_get has been deprecated by Odoo. Please look for alternatives.

* deprecated-self-cr

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L665 Use "self.env.cr" instead of "self._cr" (deprecated since 19.0)
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L819 Use "self.env.cr" instead of "self._cr" (deprecated since 19.0)
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L830 Use "self.env.cr" instead of "self._cr" (deprecated since 19.0)

* development-status-allowed

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module2/__openerp__.py#L6 Manifest key development_status "InvalidDevStatus" not allowed. Use one of: Alpha, Beta, Mature, Production/Stable.

* except-pass

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/test_module/except_pass.py#L11 pass into block except. If you really need to use the pass consider logging that exception
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/test_module/except_pass.py#L53 pass into block except. If you really need to use the pass consider logging that exception
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/test_module/except_pass.py#L62 pass into block except. If you really need to use the pass consider logging that exception

* external-request-timeout

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L1010 Use of external request method `requests.delete` without timeout. It could wait for a long time
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L1011 Use of external request method `requests.get` without timeout. It could wait for a long time
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L1012 Use of external request method `requests.head` without timeout. It could wait for a long time

* inheritable-method-lambda

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L176 Use `default=lambda self: self._default()` to preserve inheritability. More info at https://github.com/OCA/odoo-pre-commit-hooks/issues/126
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L179 Use `domain=lambda self: self._domain()` to preserve inheritability. More info at https://github.com/OCA/odoo-pre-commit-hooks/issues/126

* inheritable-method-string

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L253 Use string method name `"_compute_name"` to preserve inheritability. More info at https://github.com/OCA/odoo-pre-commit-hooks/issues/126
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L254 Use string method name `"_search_name"` to preserve inheritability. More info at https://github.com/OCA/odoo-pre-commit-hooks/issues/126
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L255 Use string method name `"_inverse_name"` to preserve inheritability. More info at https://github.com/OCA/odoo-pre-commit-hooks/issues/126

* invalid-commit

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L663 Use of cr.commit() directly - More info https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#never-commit-the-transaction
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L664 Use of cr.commit() directly - More info https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#never-commit-the-transaction
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L665 Use of cr.commit() directly - More info https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#never-commit-the-transaction

* invalid-email

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module2/__openerp__.py#L15 Invalid email "invalidmail.com"

* license-allowed

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module2/__openerp__.py#L4 License "unknow license" not allowed in manifest file.

* manifest-author-string

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module3/__openerp__.py#L5 The author key in the manifest file must be a string (with comma separated values)

* manifest-behind-migrations

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module2/__openerp__.py#L2 Manifest version (1.0) is lower than migration scripts (2.0)
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/eleven_module/__manifest__.py#L1 Manifest version (11.0.1.0.0) is lower than migration scripts (11.0.1.0.1)
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/test_module/__openerp__.py#L2 Manifest version (10.0.1.0.0) is lower than migration scripts (11.0.1.0.0)

* manifest-data-duplicated

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L20 The file "duplicated.xml" is duplicated in lines 21 from manifest key "data"

* manifest-deprecated-key

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L9 Deprecated key "description" in manifest file

* manifest-external-assets

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/twelve_module/__manifest__.py#L15 Asset https://shady.cdn.com/somefile.js should be distributed with module's source code. More info at https://httptoolkit.com/blog/public-cdn-risks/
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/twelve_module/__manifest__.py#L19 Asset https://bad.idea.com/cool.css should be distributed with module's source code. More info at https://httptoolkit.com/blog/public-cdn-risks/
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/twelve_module/__manifest__.py#L20 Asset http://insecure.and.bad.idea.com/kiwi.js should be distributed with module's source code. More info at https://httptoolkit.com/blog/public-cdn-risks/

* manifest-maintainers-list

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module3/__openerp__.py#L6 The maintainers key in the manifest file must be a list of strings

* manifest-required-author

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L5 One of the following authors must be present in manifest: 'Odoo Community Association (OCA)'

* manifest-required-key

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L2 Missing required key "license" in manifest file

* manifest-required-key-app

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/app_module/__manifest__.py#L1 Missing required key "currency" in manifest file for modules with price.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/app_module/__manifest__.py#L1 Missing required key "images" in manifest file for modules with price.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L2 Missing required key "currency" in manifest file for modules with price.

* manifest-summary-multiline

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L8 Summary in manifest file should be a one-line short description, found newline character
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module2/__openerp__.py#L8 Summary in manifest file should be a one-line short description, found newline character

* manifest-superfluous-key

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L27 Manifest superfluous key "installable". It is the same as the default value: True. Better remove it
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L29 Manifest superfluous key "active". It is the same as the default value: True. Better remove it
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L30 Manifest superfluous key "auto_install". It is the same as the default value: False. Better remove it

* manifest-version-format

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L10 Wrong Version Format "8_0.1.0.0" in manifest file. Regex to match: "(4\.2|5\.0|6\.0|6\.1|7\.0|8\.0|9\.0|10\.0|11\.0|12\.0|13\.0|14\.0|15\.0|16\.0|17\.0|18\.0|19\.0|20\.0)\.\d+\.\d+\.\d+$"
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module2/__openerp__.py#L10 Wrong Version Format "1.0" in manifest file. Regex to match: "(4\.2|5\.0|6\.0|6\.1|7\.0|8\.0|9\.0|10\.0|11\.0|12\.0|13\.0|14\.0|15\.0|16\.0|17\.0|18\.0|19\.0|20\.0)\.\d+\.\d+\.\d+$"
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module3/__openerp__.py#L8 Wrong Version Format "8.0.1.0.0foo" in manifest file. Regex to match: "(4\.2|5\.0|6\.0|6\.1|7\.0|8\.0|9\.0|10\.0|11\.0|12\.0|13\.0|14\.0|15\.0|16\.0|17\.0|18\.0|19\.0|20\.0)\.\d+\.\d+\.\d+$"

* method-compute

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L262 Name of compute method should start with "_compute_"
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L270 Name of compute method should start with "_compute_"

* method-inverse

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L264 Name of inverse method should start with "_inverse_"
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L272 Name of inverse method should start with "_inverse_"

* method-required-super

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/pylint_oca_broken.py#L40 Missing `super` call in "copy" method.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/pylint_oca_broken.py#L44 Missing `super` call in "create" method.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/pylint_oca_broken.py#L48 Missing `super` call in "write" method.

* method-search

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L263 Name of search method should start with "_search_"
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L271 Name of search method should start with "_search_"

* missing-odoo-file-app

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L2 Missing broken_module/static/description/index.html file for modules with price

* missing-readme

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L2 Missing ./README.rst file. Template here: https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst

* missing-return

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/pylint_oca_broken.py#L24 Missing `return` (`super` is used) in method inherited_method.

* no-raise-unlink

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/test_module/res_partner_unlink.py#L9 No exceptions should be raised inside unlink() functions
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/test_module/sale_order_unlink.py#L14 No exceptions should be raised inside unlink() functions

* no-search-all

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L100 Using an empty domain `search([])` without a `limit` will load all records, may impact performance.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L102 Using an empty domain `search([])` without a `limit` will load all records, may impact performance.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L103 Using an empty domain `search([])` without a `limit` will load all records, may impact performance.

* no-wizard-in-models

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L1167 No wizard class for model directory. See the complete structure https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#complete-structure

* no-write-in-compute

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L209 Compute method calling `write`. Use `update` instead.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L211 Compute method calling `write`. Use `update` instead.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L214 Compute method calling `write`. Use `update` instead.

* odoo-addons-relative-import

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L56 Same Odoo module absolute import. You should use relative import with "." instead of "odoo.addons.broken_module"
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L57 Same Odoo module absolute import. You should use relative import with "." instead of "odoo.addons.broken_module"
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L58 Same Odoo module absolute import. You should use relative import with "." instead of "odoo.addons.broken_module"

* odoo-exception-warning

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/pylint_oca_broken.py#L10 `odoo.exceptions.Warning` is a deprecated alias to `odoo.exceptions.UserError` use `from odoo.exceptions import UserError`
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/pylint_oca_broken.py#L11 `odoo.exceptions.Warning` is a deprecated alias to `odoo.exceptions.UserError` use `from odoo.exceptions import UserError`
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/pylint_oca_broken.py#L8 `odoo.exceptions.Warning` is a deprecated alias to `odoo.exceptions.UserError` use `from odoo.exceptions import UserError`

* prefer-env-translation

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L244 Better using self.env._ More info at https://github.com/odoo/odoo/pull/174844
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L260 Better using self.env._ More info at https://github.com/odoo/odoo/pull/174844
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L268 Better using self.env._ More info at https://github.com/odoo/odoo/pull/174844

* print-used

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/test_module/except_pass.py#L20 Print used. Use `logger` instead.

* renamed-field-parameter

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L335 Field parameter "digits_compute" is no longer supported. Use "digits" instead.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L335 Field parameter "select" is no longer supported. Use "index" instead.

* resource-not-exist

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L16 File "data": "file_no_exist.xml" not found.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L20 File "data": "duplicated.xml" not found.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__openerp__.py#L25 File "demo": "file_no_exist.xml" not found.

* sql-injection

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L896 SQL injection risk. Use parameters if you can. - More info https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#no-sql-injection
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L898 SQL injection risk. Use parameters if you can. - More info https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#no-sql-injection
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L900 SQL injection risk. Use parameters if you can. - More info https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#no-sql-injection

* super-method-mismatch

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/tests/test_model.py#L19 `super().test_base_method_2` mismatch but defined method is `test_base_method_1`
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/tests/test_model.py#L20 `super().test_base_method_3` mismatch but defined method is `test_base_method_1`
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/tests/test_model.py#L24 `super().test_base_method_1` mismatch but defined method is `test_base_method_2`

* test-folder-imported

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/__init__.py#L5 Test folder imported in module broken_module
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module2/__init__.py#L3 Test folder imported in module broken_module2
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/eleven_module/__init__.py#L3 Test folder imported in module eleven_module

* translation-contains-variable

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L497 Translatable term in "'Variable not translatable: %s' % variable1" contains variables. Use _('Variable not translatable: %s') % variable1 instead
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L498 Translatable term in "'Variables not translatable: %s, %s' % (variable1, variable2)" contains variables. Use _('Variables not translatable: %s, %s') % (variable1, variable2) instead
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L500 Translatable term in "'Variable not translatable: %s' % variable1" contains variables. Use _('Variable not translatable: %s') % variable1 instead

* translation-field

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L244 Translation method _("string") in fields is not necessary.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L260 Translation method _("string") in fields is not necessary.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L268 Translation method _("string") in fields is not necessary.

* translation-format-interpolation

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L473 Use lazy % or .format() or % formatting in odoo._ functions
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L481 Use lazy % or .format() or % formatting in odoo._ functions
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L506 Use lazy % or .format() or % formatting in odoo._ functions

* translation-format-truncated

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L774 Logging format string ends in middle of conversion specifier
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L789 Logging format string ends in middle of conversion specifier

* translation-fstring-interpolation

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L268 Use lazy % or .format() or % formatting in odoo._ functions
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L772 Use lazy % or .format() or % formatting in odoo._ functions
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L787 Use lazy % or .format() or % formatting in odoo._ functions

* translation-injection

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L473 Do not use str.format on translation methods. Use placeholders instead. Reference: https://lucumr.pocoo.org/2016/12/29/careful-with-str-format/
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L481 Do not use str.format on translation methods. Use placeholders instead. Reference: https://lucumr.pocoo.org/2016/12/29/careful-with-str-format/
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L514 Do not use str.format on translation methods. Use placeholders instead. Reference: https://lucumr.pocoo.org/2016/12/29/careful-with-str-format/

* translation-not-lazy

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L471 Use lazy % or .format() or % formatting in odoo._ functions
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L472 Use lazy % or .format() or % formatting in odoo._ functions
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L474 Use lazy % or .format() or % formatting in odoo._ functions

* translation-positional-used

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L498 Translation method _('Variables not translatable: %s, %s' % (variable1, variable2)) is using positional string printf formatting with multiple arguments. Use named placeholder `_("%(placeholder)s")` instead.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L501 Translation method _('Variables not translatable: %s %s' % (variable1, variable2)) is using positional string printf formatting with multiple arguments. Use named placeholder `_("%(placeholder)s")` instead.
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L504 Translation method _('Variables not translatable: %s, %s' % (variable1, variable2)) is using positional string printf formatting with multiple arguments. Use named placeholder `_("%(placeholder)s")` instead.

* translation-required

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L448 String parameter on "message_post" requires translation. Use body=self.env._('Body not translatable %s')
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L448 String parameter on "message_post" requires translation. Use subject=self.env._('Subject not translatable')
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L450 String parameter on "message_post" requires translation. Use body=self.env._('Body not translatable {}')

* translation-too-few-args

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L770 Not enough arguments for odoo._ format string
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L785 Not enough arguments for odoo._ format string

* translation-too-many-args

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L775 Too many arguments for odoo._ format string
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L790 Too many arguments for odoo._ format string

* translation-unsupported-format

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L773 Unsupported odoo._ format character 'y' (0x79) at index 30
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/models/broken_model.py#L788 Unsupported odoo._ format character 'y' (0x79) at index 30

* use-vim-comment

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module/pylint_oca_broken.py#L108 Use of vim comment

* website-manifest-key-not-valid-uri

- https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module2/__openerp__.py#L7 Website "https://odoo-community.org,https://odoo.com" in manifest key is not a valid URI. Domain 'odoo-community.org,https:' contains invalid characters
    - https://github.com/OCA/pylint-odoo/blob/v10.0.9/testing/resources/test_repo/broken_module3/__openerp__.py#L7 Website "htt://odoo-community.com" in manifest key is not a valid URI. URL needs to start with 'http[s]://'

[//]: # (end-example)

# Development

To run all the tests run:

tox

Use extra parameters to change the test behaviour

e.g. particular python version

tox -e py310

e.g. particular unittest method

tox -e py310 -- -k test_20_expected_errors

e.g. all the tests at the same time in parallel

tox -p auto

## Licenses

This repository is licensed under [AGPL-3.0](LICENSE).

----
OCA, or the [Odoo Community Association](http://odoo-community.org/), is a nonprofit
organization whose mission is to support the collaborative development of Odoo features
and promote its widespread use.

pylint-odoo

Explore Similar Repositories

Was this content helpful?

Recent Feedback

Download README