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

[FIX] [16.0] l10n_es_vat_prorate: Recompute VAT prorate bills and refunds in a period #4014

Open
wants to merge 2 commits into
base: 16.0
Choose a base branch
from

Conversation

Shide
Copy link
Contributor

@Shide Shide commented Feb 14, 2025

Permite recomputar la prórrata del IVA en las Facturas y Reembolsos de proveedores dentro de un periodo de prórrata.
Las facturas o reembolsos que no se hayan podido pasar a borrador (para hacer el recómputo), las mostrará en una lista tras finalizar el proceso.

https://www.loom.com/share/ae5f6ae531ee4fbd9182a342a9bd67d6?sid=9e8143ca-d468-49a2-9812-72d05dae0393

MT-8885 @moduon @rafaelbn @fcvalgar @yajo @EmilioPascual @ArantxaSudon @pedrobaeza revisad si queréis por favor 😄

@OCA-git-bot
Copy link
Contributor

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

@Shide Shide force-pushed the 16.0-recompute_vat_prorate-l10n_es_vat_prorate branch from 625310d to f10a2f6 Compare February 14, 2025 09:46
@Shide Shide force-pushed the 16.0-recompute_vat_prorate-l10n_es_vat_prorate branch from f10a2f6 to 03adb0b Compare February 14, 2025 09:47
@pedrobaeza pedrobaeza added this to the 16.0 milestone Feb 14, 2025
@pedrobaeza pedrobaeza requested a review from etobella February 14, 2025 19:36
Copy link
Contributor

@ArantxaSudon ArantxaSudon left a comment

Choose a reason for hiding this comment

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

Revisión Funcional

Creadas varias facturas con una prorrata. Al cambiar la prorrata de ese periodo y darle actualizar, el cálculo ha variado. En el 303 también ha cambiado al darle a recalcular

Gracias @Shide

Copy link
Contributor

@EmilioPascual EmilioPascual left a comment

Choose a reason for hiding this comment

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

Un pequeño cambio opcional.

Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

Listo! 👍🏼

Gracias 😄 ❤️

@rafaelbn
Copy link
Member

@Shide esto no es un FIX es un IMP (Improvement). Se debe mezclar como minor o major

[IMP] [16.0] l10n_es_vat_prorate: Recompute VAT prorate bills and refunds in a period #4014

Para tu información @etobella esto ya lo tenemos en PROD.

Resuelve el problema de que metes facturas en 2024 en 2025 (primera , segunda semana) y no has configurado la prorrata del 2025 porque hasta que no calculas la nueva, generalmente cuando haces el 303 de 4T y el 390 que te sale la nueva.

Para mi, falta atender el comentario de @EmilioPascual y está para merge. Gracias

@etobella
Copy link
Member

Creo que lo que estais haciendo es un poco peligros si ya has cerrado periodos... soy partidario de verificar las fechas de cierre de impuestos para evitar que un usuario accidentalmente intente recalcular un periodo presentado.

@Shide
Copy link
Contributor Author

Shide commented Feb 24, 2025

@etobella Si tienes las fechas de bloqueo puestas, cuando pasas un asiento a borrador te salta un error

@etobella
Copy link
Member

etobella commented Feb 24, 2025

Lo sé, y no puedo modificar ningun asiento en este caso pues... Por eso lo comento
Lo mejor seria conseguir que modificara solo aquello que puede.

@rafaelbn
Copy link
Member

Lo mejor seria conseguir que modificara solo aquello que puede.

Hola @etobella !

La situación ideal es que la persona que lo está haciendo el sistema le avise si está intentando cambiar algo que no debe y por lo tanto, desbloquee la contabilidad (que queda registro en res.company, quién y cuando ha cambiado las fechas de bloqueo) resuelva el problema de la prorrata y vuelva a bloquear.

Piensa que esto se da siempre en el mismo contexto y por lo tanto la lógica y la historia del usuario cuando lo quiere arreglar es esta que planteamos:

  1. Voy a re-calcular la prorrata (tengo permisos para ello)
  2. Me da un error porque la contabilidad está bloqueada (quizá la ha bloqueado otra persona)
  3. Sigo el procedimiento interno para bloquear y desbloquear que puede ser pedir autorización a CFO, etc... y desbloqueo diciendo el por qué.
  4. Una vez desbloqueado lo que quiero cambiar, lo re-calculamos.
  5. Vuelvo a seguir el procedimiento para bloquear de nuevo.

Si el sistema no avisa entonces no se tiene la certeza de lo que ha hecho.

Ya nos dices

¡Gracias!

@etobella
Copy link
Member

El único problema de lo expuesto es que Odoo no permite tirar hacia atrás los bloqueos de fechas contables (hasta v18)

https://github.com/odoo/odoo/blob/17.0/addons/account_lock/models/res_company.py

Y a partir de v18 no es con todos los campos pero pasa lo mismo (https://github.com/odoo/odoo/blob/18.0/addons/account/models/company.py#L464)

Teniendo en cuenta que estamos en v16 en este PR, no sé como van a seguir este flujo 🤔

@Shide Shide force-pushed the 16.0-recompute_vat_prorate-l10n_es_vat_prorate branch from 03adb0b to 874e12e Compare February 28, 2025 10:23
@Shide
Copy link
Contributor Author

Shide commented Feb 28, 2025

@etobella He puesto otro campo que dice si se puede recomputar (si / no / parcial) y oculta y muestra botones de diferentes colores (verde o amarillo) indicando que el reprorateado será parcial o completo.

Con esto creo que funcionará bien con los bloqueos de fechas en futuras versiones.

Como solo debe ejecutarse a principios de año (cuando no se han bloqueado fechas de impuestos y se permite la modificación de asientos), me imagino que en futuras versiones funcionará bien.

Además, ahora es compatible con modificar solo lo que puede (reprorateado parcial) y coge la fecha acorde a lo que puede modificar.

También decir que la alternativa actual es picarte todas las facturas a mano a principios de año y hacer prácticamente el mismo proceso que hará este PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants