[16.0][ADD] l10n_es_vat_book_prorate: Cálculo cuota deducible en el libro de IVA con la Prorrata del IVA

[EmilioPascual]

Cuando están instalados el módulo de la prorrata del IVA l10n_es_vat_prorate y el módulo del Libro de IVA l10n_es_vat_book la cuota deducible no se calcula correctamente. Ver issue: #4060

@rafaelbn @ArantxaSudon @pedrobaeza @etobella @Shide ¿podéis revisarlo, por favor?

@moduon MT-5880

[ArantxaSudon]

Revisión Funcional Hecha

Probado poniendo una prorrata en la compañía del 80% y haciendo facturas con distintos tipos impositivos de IVA y el LIVA sale perfecto

Muchas gracias @EmilioPascual

[rafaelbn]

Yes!!!! 👍🏼 ❤️ Thank you Emilio!

And the result. Actually is wrong!

Please @omar7r could you review it? 🙏🏼

[rafaelbn]

@etobella , ya sé que vosotros estáis en el SII pero creo que esto le puede interesar mucho a todos los que usan prorrata 😄

[omar7r]

Probado funcionalmente con el resultado esperado.

Gracias

[OCA-git-bot]

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

[rafaelbn]

Hola @OCA/local-spain-maintainers , por favor alguno puede mezclar esto?

Tiene todas las validaciones necesarios y el tiempo adecuado.

Sólo falta hacer merge

Gracias
Saludos,

[pedrobaeza]

Faltaría como mínimo un script de migración para copiar el valor de tax_amount en deductible_amount en los registros ya existentes.

Insisto en revisar el enfoque hecho con el SII para ver si se puede evitar crear un glue module. ¿No se podría llamar a

l10n-spain/l10n_es_vat_prorate/models/account_move.py

Line 14 in d0eef08

def _process_aeat_tax_fee_info(self, res, tax, sign):

?

[EmilioPascual]

Faltaría como mínimo un script de migración para copiar el valor de tax_amount en deductible_amount en los registros ya existentes.

Insisto en revisar el enfoque hecho con el SII para ver si se puede evitar crear un glue module. ¿No se podría llamar a

l10n-spain/l10n_es_vat_prorate/models/account_move.py

Line 14 in d0eef08

def _process_aeat_tax_fee_info(self, res, tax, sign):

?

Se podría hacer una pequeña refactorización para poder utilizar los métodos _process_aeat_tax_base_info y _process_aeat_tax_fee_info de l10n_es_aeat y así salvar esa parte. Pero lo que no veo como hacer sin una gran refactorización del módulo es como filtrar los apuntes por el campo vat_prorate, no estoy viendo una manera factible y que tenga sentido de extraer un método a l10n_es_aeat tal y como está construido el módulo del libro de IVA.

En cuanto al script de migración tienes razón, me pongo a ello.

[EmilioPascual]

@pedrobaeza ya está el script de migración ¿Puedes volver a revisarlo incluido mi comentario anterior? Gracias

[pedrobaeza]

Buenas, Emilio, no entiendo por qué deberías filtrar los apuntes por el campo vat_prorate. Al final, para todos los apuntes, sean de prorrata o no, tú llamas al método que obtiene la cantidad deducible, y lo colocas en el campo correspondiente. El código que obtiene ese importe ya pondrá el importe completo o prorrateado según sus condiciones.

[EmilioPascual]

En el módulo l10n_es_vat_book se utilizan los modelo l10n.es.vat.book.line y l10n.es.vat.book.line.tax donde se tiene el calculo de la base, el total, ... y donde se debería incluir la cantidad deducible.

Para crear esos registros, en https://github.com/OCA/l10n-spain/blob/17.0/l10n_es_vat_book/models/l10n_es_vat_book.py#L471-L483 se filtran los apuntes quedando excluidos los apuntes que tienen el check de la prorrata, por lo que no se tendrán en cuenta al crear los registros con los respectivos cálculos.

[rafaelbn]

Buenos días @OCA/local-spain-maintainers :

Por favor , tenemos 3 validaciones, 1 de un PSC.

¿Hace falta algo más para poder hacer merge?

No sé qué más podemos hacer para agilizar esto.

Muchas gracias
Saludos,
Rafa

[etobella]

@rafaelbn Creo que lo que intentas hacer es un "@pedrobaeza Puedes volver a revisar con los comentarios de @EmilioPascual 🙏 "

[pedrobaeza]

Tal como os comenté, no hace falta un glue module para esto, si no utilizar los métodos estándar de l10n_es_aeat. Aquí está el cambio necesario para ello:

#4311

Revisadlo por favor y podemos proceder al cierre de este PR y fusión del otro.