[19.0][MIG] l10n_es_aeat_mod390: Migration to 19.0 by pablo-cort-s73 · Pull Request #4727 · OCA/l10n-spain

pablo-cort-s73 · 2026-01-14T08:54:57Z

Migración standard a v19.0

acysos · 2026-01-17T09:19:44Z

/ocabot migration l10n_es_aeat_mod390

Reyes4711-S73

LGTM

ArantxaSudon

Revisión Funcional

Generado el modelo 390, hechos cambios en todos los periodos y correctamente recogidos en dicho modelo

Gracias @pablo-cort-s73

MT-14050 @moduon

pablo-cort-s73 · 2026-03-26T10:07:27Z

Rebase a 19.0 i eliminado el commit de dependicia a #4680

OCA-git-bot · 2026-03-26T10:19:07Z

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Reyes4711-S73 · 2026-03-26T13:13:40Z

@OCA/local-spain-maintainers Por favor, podéis revisar el PR?

pedrobaeza · 2026-03-26T19:21:53Z

                    # Si salió a devolver, casilla 98 = casilla 71 del último periodo
                    # del año si fue a devolver
                    casilla_98 = abs(report_303_last_period.resultado_liquidacion)
+                    casilla_662 = 0.0


Por qué se pone esto a 0 aquí? Si es un parche extra, tendría que venir en un commit separado con su propia explicación.

Disculpa la tardanza al contestar @pedrobaeza , pero lo estuve revisando todo.

De hecho, tenía un error al actualizar la sintaxis de los dominios a Odoo 19, sin querer añadí el dominio de result_type = C en el search de applied_this_year al copiarlo desde la estructura de compensation_this_year.

Por otra parte, en referencia al poner a 0, he añadido un nuevo commit [FIX], ya que cuando el último periodo del 303 sale a devolver, la casilla [662] debe ser 0, ya que el contribuyente está reclamando la devolución de todo el saldo acumulado del año y no queda nada pendiente de compensar para ejercicios siguientes.

Ahora ya realiza todos los cálculos correctos para la casilla 662.

Creo que tendría más sentido incluirlo dentro del método que ya existe para calcular la casilla 662

pedrobaeza

Perdón, botón equivocado. Marco así hasta aclarar mi pregunta.

================================ Presentación del Modelo AEAT 390 ================================ Modelo 390 de la AEAT. Declaración-resumen anual del Impuesto sobre el Valor Añadido. Instalación =========== Este módulo requiere del módulo *account_tax_balance* que se encuentra en https://github.com/OCA/account-financial-reporting. Uso === Para crear un modelo, por ejemplo de un trimestre del año: 1. Ir a *Contabilidad > Declaraciones AEAT > Modelo 390*. 2. Pulsar en el botón "Crear" 3. Seleccionar el ejercicio fiscal. 4. Seleccionar el tipo de declaración. 5. Rellenar el teléfono y teléfono móvil, necesarios para la exportacion BOE 6. Guardar y pulsar en el botón "Calcular" 7. Rellenar (si es necesario) aquellos campos que Odoo no calcula automáticamente. 8. Cuando los valores sean los correctos, pulsar en el botón "Confirmar" 9. Podemos exportar en formato BOE para presentarlo telemáticamente en el portal de la AEAT Problemas conocidos / Hoja de ruta ================================== * La declaración sólo se puede realizar para personas jurídicas. * No se han implementado todas las casillas de opciones de la empresa, como por ejemplo si la empresa está en concurso de acreedores o si pertenece al registro de devolución mensual. * No se calculan operaciones intragrupo. * No se contempla el régimen de criterio de caja. * No se contempla el régimen especial de bienes usados, objetos de arte, antigüedades y objetos de colección. * No se contempla el régimen especial de agencias de viaje. * No se contempla el régimen especial de la agricultura, ganadería y pesca. * No se contempla la prorrata general de IVA. * No se contempla el régimen simplificado. * No se contempla el régimen de deducción diferenciado. * No se contempla la inversión de sujeto pasivo nacional. * No se ha incluido el cálculo para los bienes de inversión. * No se ha incluido el cálculo para otros supuestos de inversión de sujeto pasivo. * No se tienen en cuenta tributaciones territoriales. * Obtener las casillas 95, 97 y 98 de las declaraciones del 303.

This way, we don't initially see all the subconfigs for each model and we can concentrate on main export configurations. This is got through `active` field default behaviour. Archived config are still directly searchable through the subconfig selector in the configuration line. All models export configs have been adapted to this new approach.

Currently translated at 100.0% (137 of 137 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod390 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod390/es/

* Include chatter (mail.thread) * Include activities (mail.activity.mixin) * Track AEAT report state change * Include new admin user in AEAT group by default

- Añadido Diseño de registro dr390e2019v100 para declaración de 2019

…itable They weren't by mistake. Fixes OCA#1256

…n tests This way, we can reuse it in other modules that don't inherit AEAT report like SII. We also clean a non needed reading of the tax description field. Tests were still using description method for populating data, so we switch to new system and perform needed changes in dependent tests. s

EmilioPascual · 2026-04-08T07:31:14Z

¡Hola @EmilioPascual! Veo la sugerencia de cambiar la lista por una tupla. Por curiosidad y para tenerlo en cuenta para futuros PRs: ¿hay algún motivo técnico específico por el que prefieras tuplas aquí?

Lo comento porque, según la documentación, el operador 'in' del Domain acepta listas sin problema (y las suelo usar por inercia por si luego necesito añadir valores dinámicamente). No sé si es por un tema de rendimiento, inmutabilidad, pero dímelo y así lo tengo en cuenta para futuros cambios. Los commits pendientes los reviso esta mañana ¡Gracias por la revisión!

En python las tuplas son ligeramente más rápidas en cuanto a iteración y consumo de memoria

Updated according https://sede.agenciatributaria.gob.es/static_files/Sede/Disenyo_registro/DR_300_399/archivos_25/dr390e2025.xlsx MT-13085 @moduon

The box 28 is "IVA devengado", so it must read from account 477 MT-13476 @moduon

@moduon MT-13508

Box 99 proper name is "Operaciones en régimen general". TT60503

pablo-cort-s73 · 2026-04-10T06:23:15Z

Cambios hechos y agregados los commits de la v18.0! @EmilioPascual
cc @pedrobaeza

EmilioPascual · 2026-04-10T06:31:47Z

Cambios hechos y agregados los commits de la v18.0! @EmilioPascual cc @pedrobaeza

Gracias @pablo-cort-s73.

Sigo pensando que para poner la casilla 662 cuando es a devolver en el último trimestre tednría que hacerse en el método de ya calcula esa casilla, no veo nada para que no se pueda hacer ahí.

Hay un nuevo commit b65466b de hace 2 días.

pablo-cort-s73 · 2026-04-10T06:43:39Z

@EmilioPascual el commit esta añadido, y tienes razón con lo de la casilla 662, ahora lo ajusto, gracias!

HaraldPanten · 2026-04-10T06:49:38Z

@pablo-cort-s73 En cuanto al commit mencionado (b65466b), puedes mirar de limpiar lo de 17.0 en el mensaje? Se nos pasó.

@pedrobaeza FYI

EmilioPascual · 2026-04-10T07:10:59Z

@EmilioPascual el commit esta añadido, y tienes razón con lo de la casilla 662, ahora lo ajusto, gracias!

Tienes razón con el commit, no lo vi 🙏

… refund When the last 303 period is a refund (result types D, V or X), the taxpayer is reclaiming the full accumulated VAT balance for the year. Therefore, field [662] (compensation pending for future years) must be set to 0, as there is nothing left to carry forward.

pablo-cort-s73 · 2026-04-10T07:56:25Z

Cambios hechos @EmilioPascual @HaraldPanten
Gracias!

HaraldPanten

A la espera del OK de Emilio y Pedro, por mi parte adelante.

EmilioPascual

¡Fenomenal!. Gracias @pablo-cort-s73

pedrobaeza

Gracias a todos.

Excelente ejemplo de colaboración inter-empresas.

/ocabot merge nobump

OCA-git-bot · 2026-04-10T09:31:51Z

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

HaraldPanten · 2026-04-10T09:38:03Z

Gracias a todos.

Excelente ejemplo de colaboración inter-empresas.

/ocabot merge nobump

Para presentar el caso en los Spanish OCA Days es excelente. Buen ejemplo para la charla de iniciación 👏

OCA-git-bot · 2026-04-10T09:41:07Z

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot · 2026-04-10T09:42:18Z

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

RabbitJon-S73 approved these changes Jan 15, 2026

View reviewed changes

OCA-git-bot added this to the 19.0 milestone Jan 17, 2026

OCA-git-bot mentioned this pull request Jan 17, 2026

Migration to version 19.0 #4402

Open

40 tasks

Reyes4711-S73 approved these changes Jan 20, 2026

View reviewed changes

OCA-git-bot added the approved label Jan 20, 2026

ArantxaSudon approved these changes Mar 3, 2026

View reviewed changes

pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_aeat_mod390 branch 2 times, most recently from c26ced5 to 24a03bd Compare March 12, 2026 16:24

pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_aeat_mod390 branch from 24a03bd to b304b4b Compare March 26, 2026 09:44

OCA-git-bot added the ready to merge label Mar 26, 2026

pedrobaeza approved these changes Mar 26, 2026

View reviewed changes

pedrobaeza requested changes Mar 26, 2026

View reviewed changes

OCA-git-bot removed approved ready to merge labels Mar 26, 2026

pedrobaeza and others added 12 commits March 31, 2026 16:19

[IMP] l10n_es_aeat_mod390: Add tests

0e7afc8

[MIG] l10n_es_aeat_mod390: Migration to 10.0

87c167b

[MIG] l10n_es_aeat_mod390: Migration to 11.0

3ef100f

[IMP] l10n_es_aeat_mod390: 2018 BOE format + investment & ISP taxes

08e8b0a

[MIG] l10n_es_aeat_mod390: Migration to 12.0

d3b636a

Translated using Weblate (Spanish)

637e233

Currently translated at 100.0% (137 of 137 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod390 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod390/es/

[IMP] l10n_es_aeat: Add chatter + activities to AEAT models

5c23da0

* Include chatter (mail.thread) * Include activities (mail.activity.mixin) * Track AEAT report state change * Include new admin user in AEAT group by default

l10n_es_aeat_mod390:

74cbdfc

- Añadido Diseño de registro dr390e2019v100 para declaración de 2019

[FIX] l10n_es_aeat_mod390: Make 1st representative date and notary ed…

61eb4d8

…itable They weren't by mistake. Fixes OCA#1256

pablo-cort-s73 and others added 6 commits April 10, 2026 07:58

[MIG] l10n_es_aeat_mod390: Migration to 19.0

37f91d9

[IMP] l10n_es_aeat_mod390: 2025 BOE export format

df5d7f1

Updated according https://sede.agenciatributaria.gob.es/static_files/Sede/Disenyo_registro/DR_300_399/archivos_25/dr390e2025.xlsx MT-13085 @moduon

[IMP] l10n_es_aeat_mod390: Test new export BOE config

2fca868

[FIX] l10n_es_aeat_mod390: Box 28 restricted to account 477

6d8569b

The box 28 is "IVA devengado", so it must read from account 477 MT-13476 @moduon

[IMP] l10n_es_aeat_mod390: Box 523

5445434

@moduon MT-13508

[FIX] l10n_es_aeat_mod390: Proper box name

32427fd

Box 99 proper name is "Operaciones en régimen general". TT60503

pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_aeat_mod390 branch from 0eb89af to e5532d6 Compare April 10, 2026 06:14

[IMP] l10n_es_aeat_mod390: remove s_iva0_ic,s_iva0_e obsolete tax maps

4472e36

pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_aeat_mod390 branch from e5532d6 to 85fd4f8 Compare April 10, 2026 07:54

HaraldPanten approved these changes Apr 10, 2026

View reviewed changes

EmilioPascual approved these changes Apr 10, 2026

View reviewed changes

pedrobaeza approved these changes Apr 10, 2026

View reviewed changes

OCA-git-bot added the approved label Apr 10, 2026

OCA-git-bot added the bot is merging ⏳ label Apr 10, 2026

OCA-git-bot added the ready to merge label Apr 10, 2026

OCA-git-bot merged commit b0e8d6e into OCA:19.0 Apr 10, 2026
7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Apr 10, 2026

Uh oh!

Uh oh!

Conversation

pablo-cort-s73 commented Jan 14, 2026

Uh oh!

acysos commented Jan 17, 2026

Uh oh!

Reyes4711-S73 left a comment

Choose a reason for hiding this comment

Uh oh!

ArantxaSudon left a comment

Choose a reason for hiding this comment

Uh oh!

pablo-cort-s73 commented Mar 26, 2026

Uh oh!

OCA-git-bot commented Mar 26, 2026

Uh oh!

Reyes4711-S73 commented Mar 26, 2026

Uh oh!

pedrobaeza Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

pablo-cort-s73 Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

EmilioPascual Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

pedrobaeza left a comment

Choose a reason for hiding this comment

Uh oh!

EmilioPascual commented Apr 8, 2026

Uh oh!

pablo-cort-s73 commented Apr 10, 2026

Uh oh!

EmilioPascual commented Apr 10, 2026

Uh oh!

pablo-cort-s73 commented Apr 10, 2026

Uh oh!

HaraldPanten commented Apr 10, 2026

Uh oh!

EmilioPascual commented Apr 10, 2026

Uh oh!

pablo-cort-s73 commented Apr 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

HaraldPanten left a comment

Choose a reason for hiding this comment

Uh oh!

EmilioPascual 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!

OCA-git-bot commented Apr 10, 2026

Uh oh!

HaraldPanten commented Apr 10, 2026

Uh oh!

OCA-git-bot commented Apr 10, 2026

Uh oh!

Uh oh!

OCA-git-bot commented Apr 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

pablo-cort-s73 commented Apr 10, 2026 •

edited

Loading