Skip to content

[19.0][MIG] l10n_es_aeat_mod130: Migration to 19.0 - #4945

Merged
OCA-git-bot merged 40 commits into
OCA:19.0from
moduon:19.0-mig-l10n_es_aeat_mod130
May 6, 2026
Merged

[19.0][MIG] l10n_es_aeat_mod130: Migration to 19.0#4945
OCA-git-bot merged 40 commits into
OCA:19.0from
moduon:19.0-mig-l10n_es_aeat_mod130

Conversation

@EmilioPascual

Copy link
Copy Markdown
Contributor

MT-14057 @moduon

Pedro M. Baeza and others added 30 commits April 9, 2026 08:03
* Clases a CapWords
* PEP8
* Icono del modulo
- Remove redundant fields
- Expenses of difficult justification
- Monetary type in fields
- Adjust BOE export format
- Adjust views
- Test BOE export
- Other fine tuning
Currently translated at 51.4% (73 of 142 strings)

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

Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod130
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod130/ca/
- Standard procedure
- Switch to computed fields
- AEAT base adaptation
- Tests adaptation
Currently translated at 11.7% (16 of 136 strings)

Translation: l10n-spain-14.0/l10n-spain-14.0-l10n_es_aeat_mod130
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-l10n_es_aeat_mod130/sl/
Currently translated at 100.0% (135 of 135 strings)

Translation: l10n-spain-15.0/l10n-spain-15.0-l10n_es_aeat_mod130
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-15-0/l10n-spain-15-0-l10n_es_aeat_mod130/es/
@EmilioPascual EmilioPascual changed the title [MIG] l10n_es_aeat_mod130: Migration to 19.0 [19.0][MIG] l10n_es_aeat_mod130: Migration to 19.0 Apr 9, 2026
@EmilioPascual
EmilioPascual force-pushed the 19.0-mig-l10n_es_aeat_mod130 branch from ac3606b to 380951e Compare April 9, 2026 08:34

@ArantxaSudon ArantxaSudon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisión funcional. Se recogen los ingresos y gastos del periodo y se calculan correctamente los resultados

Con Resultados negativos

Image

Con resultados positivos:

Image

Gracias @EmilioPascual @moduon MT-14057

@HaraldPanten

Copy link
Copy Markdown
Contributor

/ocabot migration l10n_es_aeat_mod130

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone May 5, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request May 5, 2026
40 tasks

@fcvalgar fcvalgar left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Buen trabajo con la migración del módulo @EmilioPascual .

He revisado funcionalmente el módulo l10n_es_aeat_mod130 y he realizado las siguientes pruebas:

Test 1: OK - Creación de una nueva declaración del modelo 130 con compañía, ejercicio, periodo, diario y cuenta de contrapartida.
Test 2: OK - Cálculo básico con ingresos y gastos, verificando que el módulo calcula ingresos computables, gastos reales, gastos de difícil justificación, rendimiento neto, 20% del rendimiento y resultado de la declaración.
Test 3: OK - Cálculo de gastos de difícil justificación, comprobando que se calculan como el 5% de la diferencia positiva entre ingresos computables y gastos reales.
Test 4: OK - Cálculo con pagos fraccionados previos, verificando que en periodos posteriores se tienen en cuenta los importes positivos de declaraciones anteriores del mismo ejercicio.
Test 5: OK - Tipo de declaración “A ingresar”, comprobando que una declaración con resultado positivo queda correctamente clasificada como a ingresar.
Test 6: OK - Tipo de declaración negativa, comprobando que una declaración sin importe a ingresar queda correctamente clasificada como negativa.
Test 7: OK - Deducción por préstamo de vivienda habitual, verificando que la casilla [16] solo calcula importe cuando está marcada la opción de préstamo y existe importe positivo en [14]; si [14] es 0, la casilla queda correctamente en 0.
Test 8: OK - Bloqueo de actividad agrícola, ganadera, forestal o pesquera, comprobando que al seleccionar un tipo de actividad no soportado y recalcular, el sistema muestra el mensaje de error correspondiente.
Test 9: OK - Bloqueo de deducción por artículo 80 bis, comprobando que al activar esta deducción y recalcular, el sistema muestra el mensaje de error correspondiente.
Test 10: OK - Declaración complementaria sin casilla [18], verificando que el sistema bloquea la confirmación si se marca la declaración como complementaria sin informar el resultado de declaraciones anteriores.
Test 11: OK - Exportación BOE, comprobando que una declaración calculada puede generar el fichero de exportación con la configuración del modelo 130.
Test 12: OK - Seguridad multiempresa, verificando que el usuario solo puede consultar declaraciones de las compañías a las que tiene acceso.

Todo parece funcionar correctamente en las pruebas realizadas.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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). 🤖

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot
OCA-git-bot merged commit 20e9144 into OCA:19.0 May 6, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.