[17.0][MIG] account_analytic_required: Migration to 17.0 #669

peluko00 · 2024-06-18T10:13:51Z

Module migrated to version 17.0

cc https://github.com/APSL 156840
@miquelalzanillas @lbarry-apsl @javierobcn @mpascuall please review

[Issue] Migration to version 17.0 #607

…-analytic

…bit=0.0 This is useful in some situations where the tax generates move lines with credit=debit=0.0 with a non-zero tax amount (for instance with tax code VAT-IN-V82-21-EU-S in l10n_be). Without this patch the transaction would be rejected when analytic policy is 'required', since analytic accounts are not propagated through taxes in 6.1.

[RFR] Line length of affected code [UPD] Update pot file with newly generated code strings [ADD] Dutch translations

… two lines before new class.

…ugh when changing analytic account

Update import path.

…lytic policy

It has a default value, so this is fine.

For compatibility with account_constraints.

… credit

Plus: use lambda trick in the selection field for readability

In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.

these cause Warnings during the tests

…od to avoid warning

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/

Currently translated at 100.0% (12 of 12 strings) Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/fr_FR/

Currently translated at 100.0% (12 of 12 strings) Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/pt_BR/

Currently translated at 100.0% (12 of 12 strings) Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/es/

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/

Currently translated at 100.0% (12 of 12 strings) Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/pt_BR/

Co-authored-by: Alexis de Lattre <alexis@via.ecp.fr>

Currently translated at 100.0% (12 of 12 strings) Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/it/

Currently translated at 100.0% (12 of 12 strings) Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/sv/

Currently translated at 91.6% (11 of 12 strings) Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_required Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_required/hr/

mpascuall

LGTM

Alexis de Lattre and others added 30 commits June 18, 2024 12:12

Move my module account_analytic_required from extra-addons to account…

03eab6a

…-analytic

[MIGR] Set installable = False in modules to setup the 7.0 series

afaf753

account_analytic_required: migration V7

bdbbe1c

[FIX] Move substitution arguments out of _() call

64e75a1

[RFR] Line length of affected code [UPD] Update pot file with newly generated code strings [ADD] Dutch translations

account_analytic_required: in openerp.py replace *_xml by data, leave…

3de70be

… two lines before new class.

[IMP] account_analytic_required/tests

841c7c9

[FIX] account_analytic_required allowed policy violation to slip thro…

71478d6

…ugh when changing analytic account

Validate with flake8.

3243d0c

Update import path.

Add policy for analytic accounting in tree view.

dafd432

[IMP] comments in test suite + long lines __openerp__.py

92bfe98

Launchpad automatic translations update.

4885efc

lint the whole repo

e52520d

delete # noqa in __init__.py imports

09fc9fa

make all files not installable

8a16133

move all modules to __unported__

cde20fc

[MOD] Move account_analytic_required out of __unported__

29cb956

[MOD] Change installable and active status

9b283fc

[IMP] Replace active by auto_install key and set to False

9a11fef

[IMP] refactor account_analytic_required with constraints for robustness

8da8a8f

[IMP] account_analytic_required: better mechanism to extend policies

c1319f9

[IMP] life would be boring without flake8

d5f2c36

[IMP] account_analytic_required: add an extension point to obtain ana…

247f1ec

…lytic policy

[IMP] analytic policy must be required otherwise the checks will break

b31c4d7

It has a default value, so this is fine.

[IMP] account_analytic_required: make test use [ids] in write

1d9e736

For compatibility with account_constraints.

[FIX] account_analytic_required: add missing field in constraint

0554142

[FIX] account_analytic_required: constraint also depends on debit and…

bcf61d7

… credit

[FIX] account_analytic_(plan)_required: make policies translatable

de4c08e

Plus: use lambda trick in the selection field for readability

Add OCA as author of OCA addons

af7f40c

In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.

remove deprecated test suite declarations

63b0f57

these cause Warnings during the tests

OCA-git-bot and others added 27 commits June 18, 2024 12:12

account_analytic_required 14.0.1.0.1

5c4ed08

[IMP] account_analytic_required: black, isort, prettier

dbef835

[MIG] account_analytic_required: Migration to 15.0

4777d91

[15.0][mig] account_analytic_required

89d4295

[UPD] Update account_analytic_required.pot

0962641

[UPD] README.rst

9234c0f

[FIX] account_analytic_required: action_post() instead of post() meth…

18405bd

…od to avoid warning

account_analytic_required 15.0.1.0.2

222013a

[16.0][MIG] account_analytic_required: Migrate to version 16.0

e81086e

[UPD] Update account_analytic_required.pot

36fda94

[UPD] README.rst

21652fc

Added translation using Weblate (French (France) (fr_FR))

b0bf6c4

[UPD] README.rst

82b8102

[TRN] remove fr_FR.po

083008d

Co-authored-by: Alexis de Lattre <alexis@via.ecp.fr>

Added translation using Weblate (Italian)

ceab27e

Added translation using Weblate (Croatian)

576a59a

[IMP] account_analytic_required: pre-commit auto fixes

dfdd4f9

peluko00 mentioned this pull request Jun 18, 2024

Migration to version 17.0 #607

Open

13 tasks

[MIG] account_analytic_required: Migration to 17.0

9c618e0

mpascuall approved these changes Jun 18, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[17.0][MIG] account_analytic_required: Migration to 17.0 #669

[17.0][MIG] account_analytic_required: Migration to 17.0 #669

peluko00 commented Jun 18, 2024 •

edited

mpascuall left a comment

[17.0][MIG] account_analytic_required: Migration to 17.0 #669

Are you sure you want to change the base?

[17.0][MIG] account_analytic_required: Migration to 17.0 #669

Conversation

peluko00 commented Jun 18, 2024 • edited

mpascuall left a comment

Choose a reason for hiding this comment

peluko00 commented Jun 18, 2024 •

edited