[IMP] l10n_es_aeat_mod303_vat_prorate: add special vat prorate

[Tisho99]

Adaptación de l10n_es_aeat_mod303_vat_prorate para trabajar con la opción especial de prorrateo de iva agregada en el siguiente PR: #3626, a razón la la siguiente issue: #3605

• Agregado un nuevo método para calcular el monto prorrateado basado en la suma de los movimientos prorrateados en lugar de una regla de proporción. Este nuevo método sólo se utiliza en el prorrateo especial. El antiguo método se ha mantenido en el prorrateo general debido a su estado de desarrollo más maduro. Una mejora futura podría ser la sustitución del método antiguo por el nuevo; sin embargo, la posibilidad de editar el porcentaje prorrateado del IVA directamente en el formulario 303 no está disponible en el nuevo método, y se necesita el consenso de la comunidad.

• Añadidos tests unitarios.

T-6134

[OCA-git-bot]

Hi @pedrobaeza, @victoralmau, some modules you are maintaining are being modified, check this out!

[Tisho99]

@HaraldPanten @ValentinVinagre

[HaraldPanten]

@etobella @pedrobaeza

WDYT?

[etobella]

Aprovechando que estáis haciendo esto, podéis cambiar esto:

https://github.com/OCA/l10n-spain/blame/17.0/l10n_es_aeat_mod303_vat_prorate/models/mod303.py#L39-L44

La prorrata debería estar entre 0 y 100 (0 incluido). Hay empresas que tiene una prorrata del 0% pero igualmente declaran el 303 (empresas sanitarias, en las que solo prestan servicios médicos y, por lo tanto no tiene nunca facturas con IVA

[ValentinVinagre]

Aprovechando que estáis haciendo esto, podéis cambiar esto:

https://github.com/OCA/l10n-spain/blame/17.0/l10n_es_aeat_mod303_vat_prorate/models/mod303.py#L39-L44

La prorrata debería estar entre 0 y 100 (0 incluido). Hay empresas que tiene una prorrata del 0% pero igualmente declaran el 303 (empresas sanitarias, en las que solo prestan servicios médicos y, por lo tanto no tiene nunca facturas con IVA

2 preguntas respecto eso:

• En caso del 0 a hacienda ¿no le dará resquemor que se le mande con un 0?
• En caso del 100% entiendo que también se tendrían que realizar los apuntes de impuestos a 0?

merci

[etobella]

A ver, imaginemonos una empres aque factura 1.000.000 de euros sin IVA y, por lo que fuera, tiene una factura de 100€ con IVA. Obviamente tiene que declarar el 303 y su prorrata es del 0. Deberian aceptarlo

El caso de 100 son las empresas más habituales. Por lo que entiendo ya lo revisamos en el sistema, por lo que no debería ser un problema.

[ValentinVinagre]

A ver, imaginemonos una empres aque factura 1.000.000 de euros sin IVA y, por lo que fuera, tiene una factura de 100€ con IVA. Obviamente tiene que declarar el 303 y su prorrata es del 0. Deberian aceptarlo

El caso de 100 son las empresas más habituales. Por lo que entiendo ya lo revisamos en el sistema, por lo que no debería ser un problema.

Correcto, perdona que me he colado 😅

[Tisho99]

Hola @HaraldPanten

He arreglado los errores que comentamos ayer y el PR está listo para revisión

[Tisho99]

Adapted to the upgrades of vat prorate.

Ready for review

[HaraldPanten]

@etobella Qué tal lo ves ahora? Te cuadraría?

Gracias.

[etobella]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it! Prepared branch 17.0-ocabot-merge-pr-3656-by-etobella-bump-nobump, awaiting test results.

[OCA-git-bot]

@etobella your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-3656-by-etobella-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[etobella]

He abortado por mal uso del comando. Lo pongo como major

/ocabot merge major

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it! Prepared branch 17.0-ocabot-merge-pr-3656-by-etobella-bump-major, awaiting test results.

[OCA-git-bot]

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