[19.0][MIG] l10n_es_intrastat_report: Migration to 19.0 by victoralmau · Pull Request #5056 · OCA/l10n-spain

victoralmau · 2026-06-18T09:24:03Z

Migración a 19.0

Por favor @pedrobaeza ¿puedes revisarlo?

@Tecnativa TT61941

HaraldPanten · 2026-06-18T10:33:42Z

/ocabot migration l10n_es_intrastat_report

OCA-git-bot · 2026-06-18T10:33:54Z

The migration issue (#4402) has not been updated to reference the current pull request because a previous pull request (#4985) is not closed.
Perhaps you should check that there is no duplicate work.
CC @miguelvazquez1

HaraldPanten · 2026-06-18T10:38:16Z

Se cierra el anterior PR por falta de seguimiento.

/ocabot migration l10n_es_intrastat_report

juancarlosonate-tecnativa

LGTM

…ral imps/fixes * Remove unneeded dependency to stock_picking_invoice_link. * Updated HS Codes source file. * Optimized compute/import methods. * Remove useless/obsolete code, data and fields. * Added Spanish translation.

Previous code was forcing the existence of linked purchase lines which may be not true for all. As we already have a fallback in declaration code, let's not force this at location level.

TT27865

Missed step from the migration to v13 Update multi-company security domains and provide migration scripts TT27865

Due to upstream change, now invoice date is required.

… is set Co-authored-by: Pedro M. Baeza <pedro.baeza@tecnativa.com> l10n_es_intrastat_report 14.0.1.0.1

…ording to documentation. TT30682 l10n_es_intrastat_report 14.0.1.1.0

… de 2022 Co-Authored-By: Pedro M. Baeza <pedro.baeza@tecnativa.com>

TT34295 l10n_es_intrastat_report 14.0.2.0.1

… and show error when missing product_origin_country_id in line. TT29332 l10n_es_intrastat_report 14.0.2.1.0

Since 81aac30dd2278e43a43cf8fd6cef31e1e8c60f3f, all tests depending on invoicing base test should be post-install.

TT34295

… and show error when missing product_origin_country_id in line. TT29332

Updated from https://sede.agenciatributaria.gob.es/Sede/estadisticas/estadisticas-comercio-exterior/nomenclatura-combinada-ano.html TT35871

…ady defined in intrastat_product. Transaction 70 is eliminated, it does not exist (officially). https://eur-lex.europa.eu/legal-content/ES/TXT/HTML/?uri=CELEX:32020R1197&from=EN#d1e32-7-1 company_id=False is not necessary in v16 because the field is no longer has a default

…or when fiscal position is B2C

…ns with Intrastat Since v17, ir.model.data records (external identifiers) are no longer created with the localization module (l10n_es). Account is defined in all of them, so the post_init_hook() method must be adjusted so that the appropriate fiscal positions have the appropriate Intrastat data.

…nit_hook Define the appropriate data (intrastat+vat_required) in the fiscal positions to be consistent with what is indicated in the _get_fiscal_position() method of account.chart.template

[UPD] Update l10n_es_intrastat_report.pot [BOT] post-merge updates

[BOT] post-merge updates

TT55417

[UPD] Update l10n_es_intrastat_report.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_intrastat_report Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_intrastat_report/

Related to OCA/intrastat-extrastat#335 [UPD] Update l10n_es_intrastat_report.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_intrastat_report Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_intrastat_report/

…ue of the Intrastat field) Related to OCA/intrastat-extrastat#333 [BOT] post-merge updates

Currently translated at 98.0% (51 of 52 strings) Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_intrastat_report Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_intrastat_report/es/ Translated using Weblate (Spanish) Currently translated at 100.0% (52 of 52 strings) Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_intrastat_report Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_intrastat_report/es/

Currently translated at 65.3% (34 of 52 strings) Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_intrastat_report Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_intrastat_report/ca/ [UPD] Update l10n_es_intrastat_report.pot Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_intrastat_report Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_intrastat_report/

Relacionado con OCA/intrastat-extrastat@ee34b6d TT61896 [BOT] post-merge updates

TT61941

pedrobaeza

He realizado ya los cambios para poder fusionar.

/ocabot merge nobump

pedrobaeza · 2026-07-08T16:56:40Z

                    else:
                        raise exceptions.UserError(
-                            self.env._("Unit not found: '%s'") % iu
+                            self.env._("Unit not found: '%(iu)s'", iu=iu)


No hay que cambiar la traducción, si no cómo cargar el argumento:

Suggested change

self.env._("Unit not found: '%(iu)s'", iu=iu)

self.env._("Unit not found: '%s'", iu)

pedrobaeza · 2026-07-08T16:57:13Z

        "views/l10n_es_intrastat_product.xml",
        "wizards/l10n_es_intrastat_code_import.xml",
    ],
+    "external_dependencies": {"python": ["openpyxl"]},


No hace falta declararlo, porque el propio Odoo ya lo tiene en su requirements.txt: https://github.com/odoo/odoo/blob/19.0/requirements.txt#L49

OCA-git-bot · 2026-07-08T16:59:57Z

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-5056-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot · 2026-07-08T17:00:03Z

What a great day to merge this nice PR. Let's do it!
Prepared branch 19.0-ocabot-merge-pr-5056-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot · 2026-07-08T17:19:35Z

Congratulations, your PR was merged at 2051667. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot added series:19.0 mod:l10n_es_intrastat_report Module l10n_es_intrastat_report labels Jun 18, 2026

victoralmau mentioned this pull request Jun 18, 2026

[19.0][IMP] intrastat_product: Add a method to enable compatibility OCA/intrastat-extrastat#354

Closed

OCA-git-bot added this to the 19.0 milestone Jun 18, 2026

HaraldPanten mentioned this pull request Jun 18, 2026

[19.0][MIG] l10n_es_intrastat_report: Migration to 19.0 #4985

Closed

OCA-git-bot mentioned this pull request Jun 18, 2026

Migration to version 19.0 #4402

Open

40 tasks

victoralmau force-pushed the 19.0-mig-l10n_es_intrastat_report branch from 673b16b to 50b088f Compare June 19, 2026 07:01

juancarlosonate-tecnativa approved these changes Jul 8, 2026

View reviewed changes

luc-demeyer and others added 19 commits July 8, 2026 18:59

add intrastat module

6ead51d

[11.0][ADD] l10n_es_intrastat_product_declaration

fe43747

[MIG] l10n_es_intrastat_product: Migration to 12.0

69c1eb2

[FIX] l10n_es_intrastat_report: Allow invoices with no linked purchases

1e2ff57

Previous code was forcing the existence of linked purchase lines which may be not true for all. As we already have a fallback in declaration code, let's not force this at location level.

[IMP] l10n_es_intrastat_report: black, isort, prettier

fefe023

[MIG] l10n_es_intrastat_report: Migration to 13.0

1afa94d

TT27865

[FIX] l10n_es_intrastat_report: Update multi-company rules

d6eaf8c

Missed step from the migration to v13 Update multi-company security domains and provide migration scripts TT27865

[MIG] l10n_es_intrastat_report: Migration to 14.0

d51b19b

[FIX] l10n_es_intrastat_report: Fix tests

2c550b0

Due to upstream change, now invoice date is required.

[FIX] l10n_es_intrastat_report: additional check if company's country…

b1d303b

… is set Co-authored-by: Pedro M. Baeza <pedro.baeza@tecnativa.com> l10n_es_intrastat_report 14.0.1.0.1

[IMP] l10n_es_intrastat_report: Update intrastat transaction data acc…

2f8b72f

…ording to documentation. TT30682 l10n_es_intrastat_report 14.0.1.1.0

[IMP] l10n_es_intrastat_report: adaptacion para la nueva presentacion…

0577b89

… de 2022 Co-Authored-By: Pedro M. Baeza <pedro.baeza@tecnativa.com>

[FIX] l10n_es_intrastat_report: Change the link to the codes.

dfd5f14

TT34295 l10n_es_intrastat_report 14.0.2.0.1

[IMP] l10n_es_intrastat_report: Add to notes when missing partner_vat…

08e30ea

… and show error when missing product_origin_country_id in line. TT29332 l10n_es_intrastat_report 14.0.2.1.0

[FIX] l10n_es_intrastat_report: Test at post-install

d1551ff

Since 81aac30dd2278e43a43cf8fd6cef31e1e8c60f3f, all tests depending on invoicing base test should be post-install.

[FIX] l10n_es_intrastat_report: Change the link to the codes.

0597430

TT34295

[IMP] l10n_es_intrastat_report: Add to notes when missing partner_vat…

ae0079d

… and show error when missing product_origin_country_id in line. TT29332

[IMP] l10n_es_intrastat_report: 2022 NC codes

4b92e2f

Updated from https://sede.agenciatributaria.gob.es/Sede/estadisticas/estadisticas-comercio-exterior/nomenclatura-combinada-ano.html TT35871

victoralmau and others added 19 commits July 8, 2026 18:59

[UPD] Update l10n_es_intrastat_report.pot

e050b5f

[FIX] l10n_es_intrastat_report: Do not display missing VAT number err…

102d85d

…or when fiscal position is B2C

[FIX] l10n_es_intrastat_report: Adapt to changes on confirmation

0b5b12f

[IMP] l10n_es_intrastat_report: pre-commit auto fixes

9384d36

[17.0][MIG] l10n_es_intrastat_report

057e585

[FIX] l10n_es_intrastat_report: Define the appropriate data in post_i…

3ffb378

…nit_hook Define the appropriate data (intrastat+vat_required) in the fiscal positions to be consistent with what is indicated in the _get_fiscal_position() method of account.chart.template

[IMP] l10n_es_intrastat_report: pre-commit auto-fixes

5b2fff1

[MIG] l10n_es_intrastat_report: Migration to 18.0

5454b60

[UPD] Update l10n_es_intrastat_report.pot [BOT] post-merge updates

[FIX] l10n_es_intrastat_report: Updated template name

b55ad4e

[BOT] post-merge updates

[FIX] l10n_es_intrastat_report: Incompatibility with other intrastat

b6f5efc

TT55417

[FIX] l10n_es_intrastat_report: Fix tests (related to the default val…

b8e0b5e

…ue of the Intrastat field) Related to OCA/intrastat-extrastat#333 [BOT] post-merge updates

[FIX] l10n_es_intrastat_report: Convertir valores a 2 decimales

65e4d93

Relacionado con OCA/intrastat-extrastat@ee34b6d TT61896 [BOT] post-merge updates

[MIG] l10n_es_intrastat_report: Migration to 19.0

1d9adb7

TT61941

pedrobaeza force-pushed the 19.0-mig-l10n_es_intrastat_report branch from 50b088f to 1d9adb7 Compare July 8, 2026 16:59

pedrobaeza approved these changes Jul 8, 2026

View reviewed changes

OCA-git-bot added the approved label Jul 8, 2026

OCA-git-bot added the bot is merging ⏳ label Jul 8, 2026

OCA-git-bot merged commit 1c670ac into OCA:19.0 Jul 8, 2026
5 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Jul 8, 2026

pedrobaeza deleted the 19.0-mig-l10n_es_intrastat_report branch July 8, 2026 17:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

[19.0][MIG] l10n_es_intrastat_report: Migration to 19.0 - #5056

[19.0][MIG] l10n_es_intrastat_report: Migration to 19.0#5056
OCA-git-bot merged 47 commits into
OCA:19.0from
Tecnativa:19.0-mig-l10n_es_intrastat_report

victoralmau commented Jun 18, 2026 •

edited

Loading

Uh oh!

HaraldPanten commented Jun 18, 2026

Uh oh!

OCA-git-bot commented Jun 18, 2026

Uh oh!

HaraldPanten commented Jun 18, 2026

Uh oh!

juancarlosonate-tecnativa left a comment

Uh oh!

pedrobaeza left a comment

Uh oh!

pedrobaeza Jul 8, 2026

Uh oh!

pedrobaeza Jul 8, 2026

Uh oh!

OCA-git-bot commented Jul 8, 2026

Uh oh!

OCA-git-bot commented Jul 8, 2026

Uh oh!

Uh oh!

OCA-git-bot commented Jul 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

18 participants

	self.env._("Unit not found: '%(iu)s'", iu=iu)
	self.env._("Unit not found: '%s'", iu)

Uh oh!

Uh oh!

Conversation

victoralmau commented Jun 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

HaraldPanten commented Jun 18, 2026

Uh oh!

OCA-git-bot commented Jun 18, 2026

Uh oh!

HaraldPanten commented Jun 18, 2026

Uh oh!

juancarlosonate-tecnativa left a comment

Choose a reason for hiding this comment

Uh oh!

pedrobaeza left a comment

Choose a reason for hiding this comment

Uh oh!

pedrobaeza Jul 8, 2026

Choose a reason for hiding this comment

Uh oh!

pedrobaeza Jul 8, 2026

Choose a reason for hiding this comment

Uh oh!

OCA-git-bot commented Jul 8, 2026

Uh oh!

OCA-git-bot commented Jul 8, 2026

Uh oh!

Uh oh!

OCA-git-bot commented Jul 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

18 participants

victoralmau commented Jun 18, 2026 •

edited

Loading