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_mod303: Migration to 16.0 #2693

Merged
merged 147 commits into from
Dec 27, 2022

Conversation

FernandoRomera
Copy link
Contributor

No description provided.

Pedro M. Baeza and others added 30 commits December 21, 2022 16:22
[MIG] l10n_es_aeat_mod303: Migración a v7 del módulo con las siguientes mejoras:
- Convenciones de la comunidad.
- PEP8 en la mayoría de código.
- Icono para el módulo.
- Adaptación de vistas a las posibilidades de la v7.
- Restringidas las cuentas bancarias a las de la compañía.
…más código.

[IMP] l10n_es_aeat: Añadido wizard para ver el resultado de la exportación sin necesidad de tener el módulo document.
[IMP] l10n_es_aeat: Puesta transición de estados sin workflow.
[IMP] l10n_es_aeat_mod303: Adaptado al nuevo mecanismo. Se elimina workflow anterior mediante un script de migración.
[MIG] Eliminados módulos no migrados a v7
…zado

Presentación del Modelo AEAT 303
================================

* Incluye un motor genérico para obtener los totales de códigos de impuestos
  de manera parametrizada y con posibilidad de versionado por fechas
* Incluye un motor de parametrización de las exportaciones a formato BOE
  con posibilidad de versionado por fechas.
* Se incluyen las parametrizaciones de ambos motores para el 303.
…dacion y corregido formulario de mod303, sustituido campo casilla_46 repetido por campo casilla_69.
…ón cuando se hace la declaración

* Generador del asiento de regularización con cargo a un proveedor "Agencia
  Estatal de Administración Tributaria" creado al efecto.

Configuración
=============

Para activar la creación del asiento de regularización en un modelo, hay que
poner en el modelo correspondiente el campo allow_posting a True, y establecer
en la configuración de impuestos los conceptos que se regularizarán con el
flag "to_regularize".
======================
AEAT - Prorrata de IVA
======================

Módulo para gestionar la prorrata del IVA en las declaraciones de la AEAT (por
el momento, en el modelo 303), según el artículo 92 de la Ley 37/1992, de 28 de
diciembre, del Impuesto sobre el Valor Añadido.

Uso
===

Realizando el modelo 303, se puede seleccionar si se desea aplicar prorrata de
IVA.

Prorrata general
----------------

En el caso de seleccionar "Prorrata general", aparece un nuevo campo llamado
"Porcentaje de prorrata de IVA" que puede ser rellenado a mano.

Además, cuando se realice el asiento de regularización, la parte proporcional
de la cuota no deducida se compensará como gasto.

Se ha incluido un calculador del porcentaje de prorrata, utilizable en 2 casos:

* Al comienzo de un nuevo ejercicio, para determinar el porcentaje provisional
  a aplicar hasta el último periodo de ese ejercicio.
* En la última declaración del ejercicio, para determinar el porcentaje
  definitivo de prorrata, y sobre el que se compensará la diferencia.

Problemas conocidos / Hoja de ruta
==================================

* La prorrata especial de IVA no está contemplada aún.
…ixes 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.
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.
pedrobaeza and others added 6 commits December 21, 2022 16:22
Following odoo/odoo#95967, the mapping is done
for mapping the corresponding fields in the 303 model.

If someone sells electricity at that rate, there's no mapping yet, as we
don't know how the AEAT is going to reformat the model.

TT38004
When doing mathematical operations over several rounded amounts, you can
get a non rounded result. Example:

>>> tax_lines.mapped("amount")
[1022.18, 125899.42, 5399.41, 7354.03, 30.5, -3104.6, -181.45, 0.27, 18.03, 0.0, -0.34]
>>> sum(tax_lines.mapped("amount"))
136437.44999999995

Leading to possible wrong report result ("Solicitud de devolución"
instead of "Resultado cero").

In this commit, we round all the amounts + use float_compare for the
result to avoid the problem.

TT40670
@FernandoRomera
Copy link
Contributor Author

#2526

@pedrobaeza
Copy link
Member

/ocabot migration l10n_es_aeat_mod303

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

pedrobaeza commented Dec 22, 2022

La vista no es correcta:

Selección_020

También en el subtotal hay solape.

¿No lo has revisado en local?

@FernandoRomera
Copy link
Contributor Author

Perdonad, se ve que subí una revisión anterior. Ahora lo compruebo y reviso los otros (creo que me debió pasar lo mismo).

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

Queda un pequeño glitch visual con una línea extra:

Selección_002

No es crítico, pero a ver si podemos evitarlo.

@OCA-git-bot
Copy link
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). 🤖

@FernandoRomera
Copy link
Contributor Author

@pedrobaeza
Corregido:

image

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

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

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

@FernandoRomera FernandoRomera deleted the 16.0-mig-l10n_es_aeat_mod303 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.