odoo-pre-commit-hooks to use in pre-commit-config.yml files
Project description
odoo-pre-commit-hooks
OCA's custom pre-commit hooks for Odoo modules
Installation
You don't need to install it directly only configure your ".pre-commit-config.yaml" file
You even can install it directly:
-
Installing from pypi:
pip install -U oca-odoo-pre-commit-hooks
-
Installing from github:
pip install --force-reinstall -U git+https://github.com/OCA/odoo-pre-commit-hooks.git@main
Usage pre-commit-config.yaml
Add to your ".pre-commit-config.yaml" configuration file the following input
- repo: https://github.com/OCA/odoo-pre-commit-hooks
rev: v0.0.32
hooks:
- id: oca-checks-odoo-module
- id: oca-checks-po
args: ["--fix"]
Usage directly the entry points
If you install directly the package use the entry point:
oca-checks-odoo-module --help
oca-checks-po --help
Skip one xml-check for only one file
If you need to skip one check in one particular XML file you can use the follow comment
<?xml version="1.0" encoding="utf-8"?>
<!-- oca-hooks:disable=xml-check-to-skip -->
<odoo>
...
</odoo>
<?xml version="1.0" encoding="utf-8"?>
<!-- oca-hooks:disable=xml-check-to-skip,
xml-check-to-skip2 -->
<odoo>
...
</odoo>
The position of the comment it is not relative to the line that throw the check
It disable the entire file
Configuration
Behavior can be configured through several methods and as of now only consists of enabling/disabling checks.
Enabling or Disabling Checks
Each available hook consists of multiple checks which can be enabled/disabled using any of the following methods (ordered by priority):
- As an argument e.g.,
oca-checks-odoo --enable=check-to-enable --disable=check-to-disable1,check-to-disable2
- Using environment variables
OCA_HOOKS_ENABLE
orOCA_HOOKS_DISABLE
e.g.,export OCA_HOOKS_ENABLE=check1,check2
- A configuration file. The path to it can be specified with the argument
--config
. Alternatively a file named.oca_hooks.cfg
will be looked for (by default) in the following locations (in order):- Current working directory
- Repo's root
- User's home
Using a Configuration File
To enable or disable checks using a configuration file, add a disable
or enable
key under the MESSAGES_CONTROL
section.
For example:
[MESSAGES_CONTROL]
enable=check-enable1,check-enable2
disable=check-to-disable
As stated before, each source has a certain priority. This means that if the environment variable OCA_HOOKS_ENABLE=check1
exists, the configuration file above would not have any effect when it comes to enabling checks, and the only enabled
check will be check1
.
However, if OCA_HOOKS_DISABLE
is not set, the configuration file will still have an effect and check-to-disable
will
be disabled.
Checks
-
Check manifest-syntax-error Check if the manifest file has syntax error
-
Check csv-duplicate-record-id duplicate CSV "id" AKA xmlid but for CSV files
-
Check csv-syntax-error Check syntax error for CSV files declared in the manifest
-
Check xml-deprecated-data-node Deprecated node inside xml node
-
Check xml-deprecated-openerp-node deprecated xml node
-
Check xml-deprecated-qweb-directive for use of deprecated QWeb directives t-*-options
-
Check xml-not-valid-char-link The resource in in src/href contains a not valid character.
-
Check xml-oe-structure-missing-id
Ensure all tags with class 'oe_structure' have an ID. For more information on the rationale, see: https://github.com/OCA/odoo-pre-commit-hooks/issues/27
-
Check xml-redundant-module-name
If the module is called "module_a" and the xmlid is `<record id="module_a.xmlid_name1" ...` The "module_a." is redundant it could be replaced to only `<record id="xmlid_name1" ...`
-
Check xml-dangerous-filter-wo-user Check dangerous filter without a user assigned.
-
Check xml-create-user-wo-reset-password records of user without
context="{'no_reset_password': True}"
This context avoid send email and mail log warning -
Check xml-view-dangerous-replace-low-priority in ir.ui.view
<field name="priority" eval="10"/> ... <field name="name" position="replace"/>
-
Check xml-deprecated-tree-attribute The tree-view declaration is using a deprecated attribute.
-
Check xml-record-missing-id Generated when a tag has no id.
-
Check xml-duplicate-record-id
If a module has duplicated record_id AKA xml_ids file1.xml <record id="xmlid_name1" file2.xml <record id="xmlid_name1"
-
Check xml-duplicate-fields in all record nodes <record id="xmlid_name1"... <field name="field_name1"... <field name="field_name1"...
-
Check xml-syntax-error Check syntax of XML files declared in the Odoo manifest
-
Check xml-dangerous-qweb-replace-low-priority Dangerous qweb view defined with low priority
-
Check xml-duplicate-template-id Triggered when two templates share the same ID
-
Check xml-xpath-translatable-item check
xpath
nodes usingcontains(text(), 'Text translatable')
Since that the text could be translated so it is a mutable value. It could raiseValueError
exception if the language is changed.
Checks PO
-
Check po-requires-module Translation entry requires comment
#. module: MODULE
-
Check po-python-parse-printf Check if
msgid
is usingstr
variables like%s
So translationmsgstr
must be the same number of variables too -
Check po-python-parse-format Check if
msgid
is usingstr
variables like{}
So translationmsgstr
must be the same number of variables too -
Check po-duplicate-message-definition (message-id) in all entries of PO files
-
Check po-duplicate-model-definition Verify that no entries share the same 'model:' tag
-
Check po-pretty-format Check the following: 1. Entries sorted alphabetically 2. Lines are wrapped at 78 columns (same as Odoo) 3. Clear msgstr when it is the same as msgid
-
Check po-syntax-error Check syntax of PO files from i18n* folders
Help
usage: oca-checks-odoo-module [-h] [--no-verbose] [--no-exit] [--disable DISABLE] [--enable ENABLE] [--config CONFIG] [--list-msgs] [--fix] [files_or_modules ...]
positional arguments:
files_or_modules Odoo __manifest__.py paths or Odoo module paths.
options:
-h, --help show this help message and exit
--no-verbose If enabled so disable verbose mode.
--no-exit If enabled so it will not call exit.
--disable DISABLE, -d DISABLE Disable the checker with the given 'check-name', separated by commas.
--enable ENABLE, -e ENABLE Enable the checker with the given 'check-name', separated by commas. Default: All checks are enabled by default
--config CONFIG, -c CONFIG Path to a configuration file (default: .oca_hooks.cfg)
--list-msgs List all currently enabled messages.
--fix Automatically fix files when possible
Help PO
usage: oca-checks-po [-h] [--no-verbose] [--no-exit] [--disable DISABLE] [--enable ENABLE] [--config CONFIG] [--list-msgs] [--fix] [po_files ...]
positional arguments:
po_files PO files.
options:
-h, --help show this help message and exit
--no-verbose If enabled so disable verbose mode.
--no-exit If enabled so it will not call exit.
--disable DISABLE, -d DISABLE Disable the checker with the given 'check-name', separated by commas.
--enable ENABLE, -e ENABLE Enable the checker with the given 'check-name', separated by commas. Default: All checks are enabled by default
--config CONFIG, -c CONFIG Path to a configuration file (default: .oca_hooks.cfg)
--list-msgs List all currently enabled messages.
--fix Automatically fix files when possible
Examples
-
csv-duplicate-record-id
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/ir.model.access.csv#L5 Duplicate CSV record id "data/access_account_account_type" in test_repo/broken_module/ir.model.access.csv:6
-
csv-syntax-error
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/syntax_err_module/ir.model.access.csv#L1 'utf-8' codec can't decode byte 0xf1 in position 47: invalid continuation byte
-
manifest-syntax-error
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/manifest_werror/__manifest__.py#L1 could not be loaded manifest malformed
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/woinit_module/__manifest__.py#L1 could not be loaded
-
xml-create-user-wo-reset-password
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/res_users.xml#L10 record res.users without
context="{'no_reset_password': True}"
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/res_users.xml#L10 record res.users without
-
xml-dangerous-filter-wo-user
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo.xml#L60 Dangerous filter without explicit
user_id
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo.xml#L60 Dangerous filter without explicit
-
xml-dangerous-qweb-replace-low-priority
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L18 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L4 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L7 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1_copy.xml#L18 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1_copy.xml#L4 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1_copy.xml#L7 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1_copy2.xml#L18 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1_copy2.xml#L4 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1_copy2.xml#L7 Dangerous use of "replace" from view with priority
0 < 99
. Only replace as a last resort. Try position="attributes", position="move" or t-if="False" first
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L18 Dangerous use of "replace" from view with priority
-
xml-deprecated-data-node
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/demo/duplicated_id_demo.xml#L3 Use
<odoo>
instead of<odoo><data>
or use<odoo noupdate="1">
instead of<odoo><data noupdate="1">
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view.xml#L3 Use
<odoo>
instead of<odoo><data>
or use<odoo noupdate="1">
instead of<odoo><data noupdate="1">
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L3 Use
<odoo>
instead of<odoo><data>
or use<odoo noupdate="1">
instead of<odoo><data noupdate="1">
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo.xml#L3 Use
<odoo>
instead of<odoo><data>
or use<odoo noupdate="1">
instead of<odoo><data noupdate="1">
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo2.xml#L3 Use
<odoo>
instead of<odoo><data>
or use<odoo noupdate="1">
instead of<odoo><data noupdate="1">
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/skip_xml_check.xml#L5 Use
<odoo>
instead of<odoo><data>
or use<odoo noupdate="1">
instead of<odoo><data noupdate="1">
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/skip_xml_check_2.xml#L3 Use
<odoo>
instead of<odoo><data>
or use<odoo noupdate="1">
instead of<odoo><data noupdate="1">
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/model_view.xml#L3 Use
<odoo>
instead of<odoo><data>
or use<odoo noupdate="1">
instead of<odoo><data noupdate="1">
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/demo/duplicated_id_demo.xml#L3 Use
-
xml-deprecated-openerp-node
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view.xml#L2 Deprecated xml node
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L2 Deprecated xml node
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/model_view.xml#L2 Deprecated xml node
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/res_users.xml#L2 Deprecated xml node
-
xml-deprecated-qweb-directive
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L20 Deprecated QWeb directive
"t-field-options"
. Use"t-options"
instead - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L7 Deprecated QWeb directive
"t-esc-options"
. Use"t-options"
instead
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L20 Deprecated QWeb directive
-
xml-deprecated-tree-attribute
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo.xml#L31 Deprecated "<tree string=..."
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo.xml#L42 Deprecated "<tree colors=..."
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo.xml#L53 Deprecated "<tree fonts=..."
-
xml-duplicate-fields
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo2.xml#L19 Duplicate xml field "model" in lines 20
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo2.xml#L6 Duplicate xml field "name" in lines 13
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view_odoo2.xml#L74 Duplicate xml field "arch" in lines 77
-
xml-duplicate-record-id
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view.xml#L5 Duplicate xml record id "data/view_model_form_noupdate_0" in test_repo/broken_module/model_view_odoo.xml:5, test_repo/broken_module/model_view_odoo2.xml:5
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L5 Duplicate xml record id "data/view_model_form2_noupdate_0" in test_repo/broken_module/model_view_odoo2.xml:17
-
xml-duplicate-template-id
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L10 Duplicate xml template id "qweb/my_template2_noupdate_0" in test_repo/broken_module/template1_copy2.xml:10
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L17 Duplicate xml template id "qweb/my_template3_noupdate_0" in test_repo/broken_module/template1_copy2.xml:17
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L21 Duplicate xml template id "qweb/my_template4_noupdate_0" in test_repo/broken_module/template1_copy2.xml:21
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L25 Duplicate xml template id "qweb/my_template5_noupdate_0" in test_repo/broken_module/template1_copy2.xml:25
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L29 Duplicate xml template id "qweb/my_duplicate_template_noupdate_0" in test_repo/broken_module/template1.xml:33
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L3 Duplicate xml template id "qweb/my_template1_noupdate_0" in test_repo/broken_module/template1_copy2.xml:3
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L18 Duplicate xml template id "data/test_template_2_noupdate_0" in test_repo/test_module/website_templates_disable.xml:14
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L36 Duplicate xml template id "data/assets_backend_noupdate_0" in test_repo/test_module/website_templates_disable.xml:24
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L5 Duplicate xml template id "data/test_template_1_noupdate_0" in test_repo/test_module/website_templates_disable.xml:7
-
xml-not-valid-char-link
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L39 The resource in in src/href contains a not valid character
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L41 The resource in in src/href contains a not valid character
-
xml-oe-structure-missing-id
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L13 Consider removing the class 'oe_structure' or adding a proper id to the tag. The id must contain 'oe_structure'
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L24 Consider removing the class 'oe_structure' or adding a proper id to the tag. The id must contain 'oe_structure'
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L27 Consider removing the class 'oe_structure' or adding a proper id to the tag. The id must contain 'oe_structure'
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L30 Consider removing the class 'oe_structure' or adding a proper id to the tag. The id must contain 'oe_structure'
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates.xml#L9 Consider removing the class 'oe_structure' or adding a proper id to the tag. The id must contain 'oe_structure'
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/website_templates_disable.xml#L21 Consider removing the class 'oe_structure' or adding a proper id to the tag. The id must contain 'oe_structure'
-
xml-record-missing-id
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view.xml#L21 Record has no id, add a unique one to create a new record, use an existing one to update it
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view.xml#L24 Record has no id, add a unique one to create a new record, use an existing one to update it
-
xml-redundant-module-name
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L15 Redundant module name
<record id="broken_module.view_model_form2"
better using only<record id="view_model_form2"
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L15 Redundant module name
-
xml-syntax-error
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/file_no_exist.xml#L1 [Errno 2] No such file or directory: ''
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/file_no_exist.xml#L1 [Errno 2] No such file or directory: ''
-
xml-view-dangerous-replace-low-priority
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L108 Dangerous use of "replace" from view with priority 10 < 99. Only replace as a last resort. Try position="attributes", position="move" or invisible="1" first
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L25 Dangerous use of "replace" from view with priority 0 < 99. Only replace as a last resort. Try position="attributes", position="move" or invisible="1" first
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L37 Dangerous use of "replace" from view with priority 0 < 99. Only replace as a last resort. Try position="attributes", position="move" or invisible="1" first
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L47 Dangerous use of "replace" from view with priority 0 < 99. Only replace as a last resort. Try position="attributes", position="move" or invisible="1" first
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L70 Dangerous use of "replace" from view with priority 10 < 99. Only replace as a last resort. Try position="attributes", position="move" or invisible="1" first
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view2.xml#L92 Dangerous use of "replace" from view with priority 10 < 99. Only replace as a last resort. Try position="attributes", position="move" or invisible="1" first
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/skip_xml_check_3.xml#L15 Dangerous use of "replace" from view with priority 0 < 99. Only replace as a last resort. Try position="attributes", position="move" or invisible="1" first
-
xml-xpath-translatable-item
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view.xml#L11 Use of translatable xpath
text()
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1.xml#L39 Use of translatable xpath
text()
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1_copy.xml#L31 Use of translatable xpath
text()
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/template1_copy2.xml#L31 Use of translatable xpath
text()
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/model_view.xml#L11 Use of translatable xpath
Examples PO
-
po-duplicate-message-definition
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L17 Duplicate PO message definition "Branch" in lines 23, 29
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L35 Duplicate PO message definition "Message id toooooooooooooooooooooooooooo..." in lines 41
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L65 Duplicate PO message definition "One variable {variable1}" in lines 71
-
po-duplicate-model-definition
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L17 Translation for model:ir.model.fields,field_description:broken_module.field_wizard_description has been defined more than once in line(s) 29
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L35 Translation for model:ir.model.fields,field_description2:broken_module.field_wizard_description2 has been defined more than once in line(s) 41
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L59 Translation for model:ir.model.fields,field_description5:broken_module.field_wizard_description5 has been defined more than once in line(s) 65
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/i18n/fr.po#L24 Translation for model:ir.model.fields,field_description2:test_module.field_description2 has been defined more than once in line(s) 24
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/i18n/fr.po#L31 Translation for model:ir.model.fields,field_description5:test_module.field_description5 has been defined more than once in line(s) 31
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/i18n/fr.po#L38 Translation for model:ir.model.fields,field_description3:test_module.field_description3 has been defined more than once in line(s) 38
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/i18n/fr.po#L45 Translation for model:ir.model.fields,field_description4:test_module.field_description4 has been defined more than once in line(s) 45
-
po-pretty-format
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/ar_unicode.po is not formatted correctly
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/broken_module.pot is not formatted correctly
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po is not formatted correctly
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/xml_semi_empty.po is not formatted correctly
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/eleven_module/i18n/ugly.po is not formatted correctly
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/test_module/i18n/fr.po is not formatted correctly
-
po-python-parse-format
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L53 Translation string couldn't be parsed correctly using str.format IndexError('Replacement index 1 out of range for positional args tuple')
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L59 Translation string couldn't be parsed correctly using str.format IndexError('Replacement index 1 out of range for positional args tuple')
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L65 Translation string couldn't be parsed correctly using str.format KeyError('variable2')
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L71 Translation string couldn't be parsed correctly using str.format KeyError('variable2')
-
po-python-parse-printf
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L47 Translation string couldn't be parsed correctly using str%variables TypeError('not all arguments converted during string formatting')
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/es.po#L83 Translation string couldn't be parsed correctly using str%variables TypeError('%d format: a real number is required, not str')
-
po-requires-module
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/broken_module.pot#L14 Translation entry requires comment
#. module: MODULE
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module/i18n/broken_module.pot#L14 Translation entry requires comment
-
po-syntax-error
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/broken_module2/i18n/en.po#L1 Syntax error in po file (line 1)
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.0.32/test_repo/syntax_err_module/i18n/es.po#L1 Syntax error in po file (line 19)
Licenses
This repository is licensed under AGPL-3.0.
OCA, or the Odoo Community Association, is a nonprofit organization whose mission is to support the collaborative development of Odoo features and promote its widespread use.
CHANGES
v0.0.32
- [FIX] unwanted execution of globally disabled checks (#105)
- [REF] github-actions: Use exclude macosx-latest for py old (#104)
- [REF] github-actions: Add arch in cache-key to use macosx m1 and intel compatibility (#103)
- [REF] setup: Add setuptools deps to build Related to https://github.com/pypa/build/issues/98
- [REF] github-actions: Use macosx-latest only for py-latest and macosx-14 for older Related to https://github.com/actions/setup-python/issues/825\#issuecomment-2096792396
v0.0.31
- [REF] setup: Add py3.12 support info (#98)
- [REF] .github: Update actions version, fix cache and update lints (#97)
- [REF] tests: Fixing windows po autofix unitest (#96)
v0.0.30
- CI improvements (#93)
- [REF] setup: Enable py311 classifier (#87)
v0.0.29
- IMP] add xml-duplicate-template-id message (#82)
- [FIX] lints (#86)
- [ADD] po-duplicate-model-definition check (#85)
- [REF] Don't filter out messages, move is_message_enabled (#83)
- [IMP] add xml-record-missing-id (#84)
v0.0.28
- [FIX] oe-structure-missing-id: change constraints, add docs (#81)
v0.0.27
- [REF] improve xpath expressions (#80)
v0.0.26
- [IMP] Improve XPath expressions, compile them only once, fix false negative, add profiling tests (#79)
- [ADD] pretty-format-po check (#76)
- [IMP] add xml-oe-structure-missing-id check (#78)
- [REF] tests: Generate immutable readme (without extra spaces)
- [IMP] README: Added a section on configuration
v0.0.25
- [REF] requirements.txt: Update polib to fix issues and improves (#72)
v0.0.24
- [FIX] pre-commit: Fix isort hook - RuntimeError The Poetry configuration is invalid (#71)
v0.0.23
- [FIX] Correct documentation (#69)
v0.0.22
- [IMP] Help msg for "xml-view-dangerous-replace-low-priority" (#68)
- Wrote documentation for proposed refactoring (#66)
- [FIX] ci: update flake8 repo URL, tox.ini and gh actions .yml (#65)
- [REF] hooks: Add support for `` (#62)
v0.0.21
- [REF] duplicate-xml-fields: Only applies to direct record's children (#58)
v0.0.20
- [FIX] xml-xpath-translatable-item: Fix false negative calling a method o.amount_to_text() (#57)
v0.0.19
- [FIX] setup: Add missing "requirements.txt" file to package (#441) (#55)
v0.0.18
- [FIX] checks_odoo_module: Support glob expression for "qweb" manifest data section (#54)
- [IMP] oca_pre_commit_hooks: Add --list-msgs option (#53)
- [REF] oca_pre_commit_hooks: Silent subprocess git error (#52)
v0.0.17
- [IMP] oca_pre_commit_hooks: Support enable/disable from ENV (#38) (#49)
v0.0.16
- [ADD] xml-xpath-translatable-item: Add new check to identify mutable item used from xpath (#50)
v0.0.15
- [FIX] oca_pre_commit_hooks: Fix exit_status false error (#48)
v0.0.14
- [IMP] oca_pre_commit_hooks: Support for config file (#41)
v0.0.13
- [REF] pylint_odoo: Remove TODO comments (#47)
- [REF] requirements: Remove unused libraries and pre-commit hooks (#45)
- [REF] xml-deprecated-data-node: Use the data node sourceline instead of odoo node (#43)
- [IMP] oca_pre_commit_hooks: Support disable XML checks from disable comments (#42)
- [REF] README: Fix name of pypi package (#35)
- [REF] oca_pre_commit_hooks: cli_po, cli: Add cli_po and cli global (#36)
- [REF] setup: Match requires-python = ">=3.7" from pre-commit
- [REF] CI: Enable py3.11
v0.0.12
- [REF] checks_odoo_module_xml: Allow duplicated fields with 'groups' (#32)
- [REF] CI: Enable tox with parallels + codecov compatibility (#33)
v0.0.11
- [REF] codecov: Enable codecov again
- [REM] missing-readme: Check add to pylint-odoo
- [REF] TOX: Strip names, remove no-cover, compatible with parallels option (#24)
- [REF] lint: Adapt lints for py packages instead of odoo module (#23)
v0.0.10
- [REF] README: Update README and fix mutable results auto-generating it (#22)
- [REF] checks_odoo_module_po: Run on modified PO files only (#21)
- [REF] manifest: Skip duplicated data files (#20)
v0.0.9
- [FIX] README: Fix regex to replace autogenerated content (#18)
- [REF] README: Detect from CI if README needs to be updated (#17)
v0.0.8
- [REF] README: Add examples of checks section and script to auto-generate it (#16)
- [REF] README: Add help section and script to auto-generate it (#15)
v0.0.7
- [REF] checks: name_checks changed to name-checks (#14)
- [REF] oca_pre_commit_hooks: Use shorter path in messages (#13)
- [FIX] pre-commit-hooks.yaml: Run checks even if the manifest was not changed
- [REF] test: Remove TODO comments already fixed
v0.0.6
- [FIX] oca_pre_commit_hooks: Fix checks if it has syntax error (#8)
v0.0.5
- [REF] oca_pre_commit_hooks: Small improvements (#7)
v0.0.4
- [IMP] oca_pre_commit_hooks: Enable feature to 'enable' checks (#6)
- [IMP] oca_pre_commit_hooks: Add entrypoint arguments to disable checks (#5)
v0.0.3
- [REF] README: Remove deprecated CI build
v0.0.2
- [REF] CI: Re-use tox to run pre-commit in the CI with same caches and packages
- [ADD] .pre-commit-hooks.yaml: Compatible with pre-commit-hooks to use as hook id directly
v0.0.1
- [ADD] Migrate pylint-odoo checks (not related to python) to pre-commit hooks (#2)
- Initialize
Project details
Release history Release notifications | RSS feed
Download files
Download the file for your platform. If you're not sure which to choose, learn more about installing packages.
Source Distribution
Built Distribution
Hashes for oca_odoo_pre_commit_hooks-0.0.32.tar.gz
Algorithm | Hash digest | |
---|---|---|
SHA256 | 2aa5b234502edb9127f7b56bf98385a871b9e183695bc79f1147caf4f56436ca |
|
MD5 | 29cf5a948abc256ee074c54276d08f59 |
|
BLAKE2b-256 | 94b0216aa54aaa2cc0b751d67cda1faaffe77217bf3df75e9c254269f3d99410 |
Hashes for oca_odoo_pre_commit_hooks-0.0.32-py3-none-any.whl
Algorithm | Hash digest | |
---|---|---|
SHA256 | 63e9e6cb64bb245a44e0036fdd82ae4ac11cb98ad2890bd26e368833d100b1be |
|
MD5 | ae03725d28a7f1c93d609e6a5b847106 |
|
BLAKE2b-256 | c84247a58dd134ff16104a932a4b9e3fdd34662ad02839b00a4eeb0c3d7c7374 |