oca-odoo-pre-commit-hooks 0.2.8__tar.gz → 0.2.9__tar.gz
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/PKG-INFO +114 -99
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/README.md +106 -98
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/setup.py +1 -1
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_odoo_pre_commit_hooks.egg-info/PKG-INFO +114 -99
- oca_odoo_pre_commit_hooks-0.2.9/src/oca_pre_commit_hooks/__init__.py +1 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module.py +100 -45
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module_fixit/prefer_env_translation.py +2 -1
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module_xml.py +5 -3
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/utils.py +30 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/tests/test_checks.py +32 -0
- oca_odoo_pre_commit_hooks-0.2.8/src/oca_pre_commit_hooks/__init__.py +0 -1
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/LICENSE +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/pyproject.toml +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/requirements.txt +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/setup.cfg +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_odoo_pre_commit_hooks.egg-info/SOURCES.txt +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_odoo_pre_commit_hooks.egg-info/dependency_links.txt +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_odoo_pre_commit_hooks.egg-info/entry_points.txt +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_odoo_pre_commit_hooks.egg-info/not-zip-safe +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_odoo_pre_commit_hooks.egg-info/requires.txt +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_odoo_pre_commit_hooks.egg-info/top_level.txt +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/__main__.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/base_checker.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module_csv.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module_fixit/__init__.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module_fixit/manifest_superfluous_key_rule.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module_fixit/unused_logger.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module_fixit_common.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/checks_odoo_module_po.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/cli.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/cli_po.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/src/oca_pre_commit_hooks/global_parser.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/tests/test_checks_po.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/tests/test_fixit.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/tests/test_pre_commit_hook.py +0 -0
- {oca_odoo_pre_commit_hooks-0.2.8 → oca_odoo_pre_commit_hooks-0.2.9}/tests/test_profiling.py +0 -0
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
Metadata-Version: 2.4
|
|
2
2
|
Name: oca-odoo-pre-commit-hooks
|
|
3
|
-
Version: 0.2.
|
|
3
|
+
Version: 0.2.9
|
|
4
4
|
Summary: odoo-pre-commit-hooks to use in pre-commit-config.yml files
|
|
5
5
|
Home-page: https://github.com/OCA/odoo-pre-commit-hooks
|
|
6
6
|
Author: Odoo Community Association (OCA)
|
|
@@ -51,7 +51,7 @@ Dynamic: summary
|
|
|
51
51
|
[](https://pypi.org/project/oca-odoo-pre-commit-hooks)
|
|
52
52
|
[](https://pypi.org/project/oca-odoo-pre-commit-hooks)
|
|
53
53
|
[](https://pypi.org/project/oca-odoo-pre-commit-hooks)
|
|
54
|
-
[](https://github.com/OCA/odoo-pre-commit-hooks/compare/v0.2.9...main)
|
|
55
55
|
[](https://github.com/psf/black)
|
|
56
56
|
|
|
57
57
|
[//]: # (end-badges)
|
|
@@ -80,7 +80,7 @@ Add to your ".pre-commit-config.yaml" configuration file the following input
|
|
|
80
80
|
|
|
81
81
|
```yaml
|
|
82
82
|
- repo: https://github.com/OCA/odoo-pre-commit-hooks
|
|
83
|
-
rev: v0.2.
|
|
83
|
+
rev: v0.2.9
|
|
84
84
|
hooks:
|
|
85
85
|
- id: oca-checks-odoo-module
|
|
86
86
|
- id: oca-checks-po
|
|
@@ -160,6 +160,10 @@ Check if there is a file created but not referenced from __manifest__.py
|
|
|
160
160
|
* Check manifest-syntax-error
|
|
161
161
|
Check if the manifest file has syntax error
|
|
162
162
|
|
|
163
|
+
* Check use-header-comments
|
|
164
|
+
Check if the py file has comments '# comment' only in the header of python files
|
|
165
|
+
Except valid comments e.g. pylint, flake8, shebang or comments in the middle (not header)
|
|
166
|
+
|
|
163
167
|
* Check prefer-readme-rst
|
|
164
168
|
Check if the module has README.md file to prefer README.rst file
|
|
165
169
|
|
|
@@ -386,167 +390,171 @@ options:
|
|
|
386
390
|
|
|
387
391
|
* csv-duplicate-record-id
|
|
388
392
|
|
|
389
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
393
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/ir.model.access.csv#L5 Duplicate CSV record `access_account_account_type`
|
|
390
394
|
|
|
391
395
|
* csv-syntax-error
|
|
392
396
|
|
|
393
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
397
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/syntax_err_module/ir.model.access.csv#L1 'utf-8' codec can't decode byte 0xf1 in position 47: invalid continuation byte
|
|
394
398
|
|
|
395
399
|
* file-not-used
|
|
396
400
|
|
|
397
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
401
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/__openerp__.py#L1 File "broken_module/report/test_report.xml" is not referenced in the manifest.
|
|
398
402
|
|
|
399
403
|
* manifest-superfluous-key
|
|
400
404
|
|
|
401
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
402
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
403
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
405
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/__openerp__.py#L32 Delete empty values.
|
|
406
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/__openerp__.py#L34 Delete empty values.
|
|
407
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/woversion_module/__manifest__.py#L8 Delete empty values.
|
|
404
408
|
|
|
405
409
|
* manifest-syntax-error
|
|
406
410
|
|
|
407
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
408
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
411
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/manifest_werror/__manifest__.py#L1 Manifest could not be loaded
|
|
412
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/woinit_module/__manifest__.py#L1 Manifest could not be loaded
|
|
409
413
|
|
|
410
414
|
* prefer-env-translation
|
|
411
415
|
|
|
412
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
413
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
414
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
416
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/models/broken_model.py#L76 Use self.env._(...) instead of _(…) directly inside Odoo model methods.
|
|
417
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/models/broken_model.py#L86 Use self.env._(...) instead of _(…) directly inside Odoo model methods.
|
|
418
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/models/broken_model.py#L243 Use self.env._(...) instead of _(…) directly inside Odoo model methods.
|
|
415
419
|
|
|
416
420
|
* prefer-readme-rst
|
|
417
421
|
|
|
418
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
422
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/README.md#L1 Prefer README.rst instead of README.md
|
|
419
423
|
|
|
420
424
|
* unused-logger
|
|
421
425
|
|
|
422
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
426
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/models/model_inhe1.py#L17 Unused `_logger` is not allowed in Odoo models. Remove it if not used.
|
|
427
|
+
|
|
428
|
+
* use-header-comments
|
|
429
|
+
|
|
430
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/eleven_module/models.py#L9 Use of header comments in lines 3, 5, 6, 7, 9
|
|
423
431
|
|
|
424
432
|
* xml-create-user-wo-reset-password
|
|
425
433
|
|
|
426
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
434
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/res_users.xml#L10 record res.users without `context="{'no_reset_password': True}"`
|
|
427
435
|
|
|
428
436
|
* xml-dangerous-qweb-replace-low-priority
|
|
429
437
|
|
|
430
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
431
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
432
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
438
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/template1.xml#L4 Dangerous use of `replace` from view with priority 0 < 99
|
|
439
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/template1.xml#L7 Dangerous use of `replace` from view with priority 0 < 99
|
|
440
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/template1.xml#L18 Dangerous use of `replace` from view with priority 0 < 99
|
|
433
441
|
|
|
434
442
|
* xml-deprecated-data-node
|
|
435
443
|
|
|
436
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
437
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
438
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
444
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/demo/duplicated_id_demo.xml#L3 Deprecated `<data>` node
|
|
445
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view.xml#L3 Deprecated `<data>` node
|
|
446
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view2.xml#L3 Deprecated `<data>` node
|
|
439
447
|
|
|
440
448
|
* xml-deprecated-oe-chatter
|
|
441
449
|
|
|
442
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
450
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/odoo18_module/views/deprecated_chatter.xml#L6 Please replace old style chatters with the new tag <chatter/>.
|
|
443
451
|
|
|
444
452
|
* xml-deprecated-openerp-node
|
|
445
453
|
|
|
446
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
447
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
448
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
454
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view.xml#L2 Deprecated `<openerp>` xml node
|
|
455
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view2.xml#L2 Deprecated `<openerp>` xml node
|
|
456
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/model_view.xml#L2 Deprecated `<openerp>` xml node
|
|
449
457
|
|
|
450
458
|
* xml-deprecated-qweb-directive
|
|
451
459
|
|
|
452
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
453
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
460
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/website_templates.xml#L7 Deprecated QWeb directive `t-esc-options`. Use `t-options` instead
|
|
461
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/website_templates.xml#L37 Deprecated QWeb directive `t-field-options`. Use `t-options` instead
|
|
454
462
|
|
|
455
463
|
* xml-deprecated-tree-attribute
|
|
456
464
|
|
|
457
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
458
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
459
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
465
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo.xml#L31 Deprecated "<tree string=..."
|
|
466
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo.xml#L42 Deprecated "<tree colors=..."
|
|
467
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo.xml#L53 Deprecated "<tree fonts=..."
|
|
460
468
|
|
|
461
469
|
* xml-duplicate-fields
|
|
462
470
|
|
|
463
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
464
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
465
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
471
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L5 Duplicate xml field `name`
|
|
472
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L22 Duplicate xml field `model`
|
|
473
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L80 Duplicate xml field `arch`
|
|
466
474
|
|
|
467
475
|
* xml-duplicate-record-id
|
|
468
476
|
|
|
469
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
470
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
477
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view.xml#L5 Duplicate xml record id `view_model_form`
|
|
478
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view2.xml#L5 Duplicate xml record id `view_model_form2`
|
|
471
479
|
|
|
472
480
|
* xml-duplicate-template-id
|
|
473
481
|
|
|
474
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
475
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
476
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
482
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/template1.xml#L3 Duplicate xml template id `qweb/my_template1_noupdate_0`
|
|
483
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/template1.xml#L10 Duplicate xml template id `qweb/my_template2_noupdate_0`
|
|
484
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/template1.xml#L17 Duplicate xml template id `qweb/my_template3_noupdate_0`
|
|
477
485
|
|
|
478
486
|
* xml-field-bool-without-eval
|
|
479
487
|
|
|
480
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
481
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
488
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/demo/duplicated_id_demo.xml#L18 Field `active` with boolean value without `eval` attribute
|
|
489
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L18 Field `active` with boolean value without `eval` attribute
|
|
482
490
|
|
|
483
491
|
* xml-field-numeric-without-eval
|
|
484
492
|
|
|
485
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
486
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
487
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
493
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/demo/duplicated_id_demo.xml#L8 Field `priority` with numeric value without `eval` attribute
|
|
494
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/demo/duplicated_id_demo.xml#L22 Field `sequence` with numeric value without `eval` attribute
|
|
495
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view2.xml#L62 Field `priority` with numeric value without `eval` attribute
|
|
488
496
|
|
|
489
497
|
* xml-header-missing
|
|
490
498
|
|
|
491
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
492
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
499
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L1 XML missing header
|
|
500
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/xml_wo_header.xml#L1 XML missing header
|
|
493
501
|
|
|
494
502
|
* xml-header-wrong
|
|
495
503
|
|
|
496
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
497
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
498
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
504
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/demo/duplicated_id_demo.xml#L1 XML header expected '<?xml version="1.0" encoding="UTF-8" ?>' but received '<?xml version="1.0" encoding="utf-8"?>'
|
|
505
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/deprecated_disable.xml#L1 XML header expected '<?xml version="1.0" encoding="UTF-8" ?>' but received '<?xml version="1.0" encoding="utf-8" ?>'
|
|
506
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view.xml#L1 XML header expected '<?xml version="1.0" encoding="UTF-8" ?>' but received '<?xml version="1.0" encoding="utf-8"?>'
|
|
499
507
|
|
|
500
508
|
* xml-id-position-first
|
|
501
509
|
|
|
502
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
503
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
504
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
510
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/deprecated_disable.xml#L4 The "id" attribute must be first `<record id="duplicate_record" model=...`
|
|
511
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L43 The "id" attribute must be first `<record id="view_ir_config_search" model=...`
|
|
512
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L68 The "id" attribute must be first `<record id="access_rule" model=...`
|
|
505
513
|
|
|
506
514
|
* xml-not-valid-char-link
|
|
507
515
|
|
|
508
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
509
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
516
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/website_templates.xml#L59 The resource in in src/href contains a not valid character
|
|
517
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/website_templates.xml#L61 The resource in in src/href contains a not valid character
|
|
510
518
|
|
|
511
519
|
* xml-oe-structure-missing-id
|
|
512
520
|
|
|
513
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
514
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
515
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
521
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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`
|
|
522
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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`
|
|
523
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/website_templates.xml#L41 Consider removing the class `oe_structure` or adding a proper id to the tag. The id must contain `oe_structure`
|
|
516
524
|
|
|
517
525
|
* xml-record-missing-id
|
|
518
526
|
|
|
519
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
520
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
527
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view.xml#L25 Record has no id, add a unique one to create a new record, use an existing one to update it
|
|
528
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view.xml#L28 Record has no id, add a unique one to create a new record, use an existing one to update it
|
|
521
529
|
|
|
522
530
|
* xml-redundant-module-name
|
|
523
531
|
|
|
524
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
525
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
526
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
532
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view2.xml#L15 Redundant module name `<record id="broken_module.view_model_form2"`
|
|
533
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L155 Redundant module name `<menuitem id="broken_module.menu_root"`
|
|
534
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view_odoo2.xml#L161 Redundant module name `<menuitem id="broken_module.menu_root2"`
|
|
527
535
|
|
|
528
536
|
* xml-syntax-error
|
|
529
537
|
|
|
530
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
531
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
538
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/file_no_exist.xml#L1 [Errno 2] No such file or directory: ''
|
|
539
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/file_no_exist.xml#L1 [Errno 2] No such file or directory: ''
|
|
532
540
|
|
|
533
541
|
* xml-template-prettier-incompatible
|
|
534
542
|
|
|
535
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
536
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
537
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
543
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/website_templates.xml#L22 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...`
|
|
544
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/website_templates.xml#L24 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...`
|
|
545
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/website_templates.xml#L26 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...`
|
|
538
546
|
|
|
539
547
|
* xml-view-dangerous-replace-low-priority
|
|
540
548
|
|
|
541
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
542
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
543
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
549
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view2.xml#L25 Dangerous use of `replace` from view with priority 0 < 99
|
|
550
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view2.xml#L37 Dangerous use of `replace` from view with priority 0 < 99
|
|
551
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view2.xml#L47 Dangerous use of `replace` from view with priority 0 < 99
|
|
544
552
|
|
|
545
553
|
* xml-xpath-translatable-item
|
|
546
554
|
|
|
547
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
548
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
549
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
555
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/model_view.xml#L11 Use of translatable xpath `text()`
|
|
556
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/template1.xml#L39 Use of translatable xpath `text()`
|
|
557
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/template1_copy.xml#L31 Use of translatable xpath `text()`
|
|
550
558
|
|
|
551
559
|
[//]: # (end-example)
|
|
552
560
|
|
|
@@ -558,49 +566,49 @@ options:
|
|
|
558
566
|
|
|
559
567
|
* po-duplicate-message-definition
|
|
560
568
|
|
|
561
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
562
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
563
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
569
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/es.po#L17 Duplicate PO message definition `Branch` in lines 23, 29. Odoo exports these items by msgid and delete one of them. Use the `i18n_extra` folder instead of `i18n` to ignore this message.
|
|
570
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/es.po#L35 Duplicate PO message definition `Message id toooooooooooooooooooooooooooo...` in lines 41. Odoo exports these items by msgid and delete one of them. Use the `i18n_extra` folder instead of `i18n` to ignore this message.
|
|
571
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/es.po#L65 Duplicate PO message definition `One variable {variable1}` in lines 71. Odoo exports these items by msgid and delete one of them. Use the `i18n_extra` folder instead of `i18n` to ignore this message.
|
|
564
572
|
|
|
565
573
|
* po-duplicate-model-definition
|
|
566
574
|
|
|
567
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
568
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
569
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
570
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
571
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
572
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
573
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
575
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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
|
|
576
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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
|
|
577
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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
|
|
578
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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
|
|
579
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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
|
|
580
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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
|
|
581
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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
|
|
574
582
|
|
|
575
583
|
* po-pretty-format
|
|
576
584
|
|
|
577
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
578
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
579
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
580
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
581
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
582
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
585
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/ar_unicode.po Wrong formatting
|
|
586
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/broken_module.pot Wrong formatting
|
|
587
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/es.po Wrong formatting
|
|
588
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/xml_semi_empty.po Wrong formatting
|
|
589
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/eleven_module/i18n/ugly.po Wrong formatting
|
|
590
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/test_module/i18n/fr.po Wrong formatting
|
|
583
591
|
|
|
584
592
|
* po-python-parse-format
|
|
585
593
|
|
|
586
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
587
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
588
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
589
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
594
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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')
|
|
595
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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')
|
|
596
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/es.po#L65 Translation string couldn't be parsed correctly using str.format KeyError('variable2')
|
|
597
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/es.po#L71 Translation string couldn't be parsed correctly using str.format KeyError('variable2')
|
|
590
598
|
|
|
591
599
|
* po-python-parse-printf
|
|
592
600
|
|
|
593
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
594
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
601
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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')
|
|
602
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/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')
|
|
595
603
|
|
|
596
604
|
* po-requires-module
|
|
597
605
|
|
|
598
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
606
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module/i18n/broken_module.pot#L14 Translation entry requires comment `#. module: MODULE`
|
|
599
607
|
|
|
600
608
|
* po-syntax-error
|
|
601
609
|
|
|
602
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
603
|
-
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.
|
|
610
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/broken_module2/i18n/en.po#L1 Syntax error in po file
|
|
611
|
+
- https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.9/test_repo/syntax_err_module/i18n/es.po#L19 Syntax error in po file
|
|
604
612
|
|
|
605
613
|
[//]: # (end-example-po)
|
|
606
614
|
|
|
@@ -616,6 +624,13 @@ and promote its widespread use.
|
|
|
616
624
|
CHANGES
|
|
617
625
|
=======
|
|
618
626
|
|
|
627
|
+
v0.2.9
|
|
628
|
+
------
|
|
629
|
+
|
|
630
|
+
* [ADD] use-header-comments: New check to remove comments in the py headers (#157)
|
|
631
|
+
* [REF] fixit: If the environment variable FIXIT\_ODOO\_VERSION is defined so it is used (#154)
|
|
632
|
+
* [REF] checks\_odoo\_module\_xml: Use filename\_short instead of full path (#156)
|
|
633
|
+
|
|
619
634
|
v0.2.8
|
|
620
635
|
------
|
|
621
636
|
|