Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][MIG] l10n_es_aeat_mod115: Migration to 16.0 #2695

Merged
merged 40 commits into from
Dec 27, 2022

Conversation

FernandoRomera
Copy link
Contributor

No description provided.

agaldona and others added 28 commits December 22, 2022 10:49
* Eliminar periodo específico y corregir exportación
* Cálculo de importes de impuestos por cadena en lugar de recordset
* Gestionar grupos de compañías
* Evitar problemas en campos related no readonly. Fixes OCA#290

Cuando se tienen campos related, aunque el valor no se cambie,
Odoo lanza una escritura en el objeto relacionado cada vez que se
guarda el objeto principal. En el caso de la moneda asociada a la
compañía, hace que aquellos usuarios que no tengan permiso para
modificar la compañía (la mayoría), falle en la creación del modelo.

* Mejor herencia de impuestos

El anterior método utilizado, al tener una clase y un many2one genérico,
no permitía poner los datos en múltiples modelos hijos, ya que se repetían
los IDs, y cogía los datos de un modelo en otros.

Con este nuevo sistema, se sigue utilizando una tabla, pero con doble
índice: el modelo y el ID, parecido a un campo reference, pero gestionado
por código propio en el método de cálculo.

El 303 está adaptado a este nuevo sistema, y se incluye un script de
migración que conserve los datos.
Currently translated at 94.3% (33 of 35 strings)

Translation: l10n-spain-11.0/l10n-spain-11.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-11-0/l10n-spain-11-0-l10n_es_aeat_mod115/ca/
* Standard procedure
* README by fragments
* Improve code a bit
* Expand tests for negative result
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% (76 of 76 strings)

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

Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod115/ca/
* Include chatter (mail.thread)
* Include activities (mail.activity.mixin)
* Track AEAT report state change
* Include new admin user in AEAT group by default
Currently translated at 84.1% (90 of 107 strings)

Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod115/es/
…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
Currently translated at 85.0% (91 of 107 strings)

Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod115/es/
@FernandoRomera
Copy link
Contributor Author

#2526

Copy link

@MRomeera MRomeera left a comment

Choose a reason for hiding this comment

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

Ok

@pedrobaeza
Copy link
Member

/ocabot migration l10n_es_aeat_mod115

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Dec 25, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 25, 2022
54 tasks
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Probado funcionalmente en runboat y código revisado.

Haz por favor el mismo squashing que he mencionado en el otro pull request.

acysos and others added 12 commits December 27, 2022 09:15
[FIX] l10n_es_aeat_mod115: minor fix

[FIX] l10n_es_aeat_mod115: minor fix

[FIX] l10n_es_aeat_mod115: minor fix
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-13.0/l10n-spain-13.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-13-0/l10n-spain-13-0-l10n_es_aeat_mod115/
Currently translated at 100.0% (98 of 98 strings)

Translation: l10n-spain-13.0/l10n-spain-13.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-13-0/l10n-spain-13-0-l10n_es_aeat_mod115/es/
Currently translated at 100.0% (98 of 98 strings)

Translation: l10n-spain-13.0/l10n-spain-13.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-13-0/l10n-spain-13-0-l10n_es_aeat_mod115/ca/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-14.0/l10n-spain-14.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-l10n_es_aeat_mod115/
Co-Authored-By: Jorge Hernández <jhernandez@coninpe.es>
Co-Authored-By: Raúl Fernández <raul.fernandez@qubiq.es>
Currently translated at 100.0% (101 of 101 strings)

Translation: l10n-spain-15.0/l10n-spain-15.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-15-0/l10n-spain-15-0-l10n_es_aeat_mod115/ca/
Currently translated at 100.0% (101 of 101 strings)

Translation: l10n-spain-15.0/l10n-spain-15.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-15-0/l10n-spain-15-0-l10n_es_aeat_mod115/ca/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-15.0/l10n-spain-15.0-l10n_es_aeat_mod115
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-15-0/l10n-spain-15-0-l10n_es_aeat_mod115/
@FernandoRomera
Copy link
Contributor Author

@pedrobaeza
Realizado squashing.

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-2695-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 893280b into OCA:16.0 Dec 27, 2022
@OCA-git-bot
Copy link
Contributor

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

@FernandoRomera FernandoRomera deleted the 16.0-mig-l10n_es_aeat_mod115 branch December 28, 2022 16:41
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.