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

Declaración 303 cantidades a compensar V12 #1624

Closed
anajuaristi opened this issue Mar 1, 2021 · 7 comments
Closed

Declaración 303 cantidades a compensar V12 #1624

anajuaristi opened this issue Mar 1, 2021 · 7 comments

Comments

@anajuaristi
Copy link
Contributor

Cuando en una declaración de un periodo en el 303, se obtiene una cantidad negativa, a compensar, al intentar calcular el siguiente periodo da este error
"La cuota a compensar debe ser indicada como una cantidad positiva"

Para solucionarlo, he sacado a la vista formulario de la declaración, el grupo de campos invisible cuando es borrador y ahí he visto que independientemente del resultado anterior, la casilla: potential_cuota_compensar va con el valor cero.

Asignando el importe en positivo de la declaración anterior en esa casilla hace bien el cálculo. Creo que habría que meter un valor default en ella. Ahora bien, no sé esto qué implicaciones tiene y si se están cubriendo todos los casos. O si en realidad no hay que asignarlo en esa casilla y sí en otra... Antes de subir ningún parche, me gustaría que me comenteis cómo lo veis.

Gracias!
Ana

@pedrobaeza
Copy link
Member

Lo comentado por chat, que el código es correcto ahora mismo, y no se me ocurre ninguna condición para hacer saltarlo en un uso normal del modelo. Hacen falta los pasos para reproducir el problema en un runbot.

@anajuaristi
Copy link
Contributor Author

Aparte de en nuestro entorno, también lo hemos probado en runbot.
En principio, con conseguir que la declaración del periodo anterior te dé negativo, ya se reproduce.
Te paso entorno. Hemos hecho con febrero en negativo y el que salta el warning es marzo
http://3467238-12-0-521c25.runbot1-3.odoo-community.org/

@pedrobaeza
Copy link
Member

Pues ésa es la cuestión, que se hace el valor absoluto para cambiar el negativo:

mod303.cuota_compensar = abs(prev_report.resultado_liquidacion)

Lo tendré que mirar más en detenimiento a ver si pasa un estado intermedio.

@ManelNight
Copy link

Buenas tardes,

En una v11 nos pasa lo mismo. El mes de enero se genero correctamente pero el de febrero genera el error que indica @anajuaristi

Saludos,
Manel

pedrobaeza added a commit that referenced this issue Mar 5, 2021
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes #1624
pedrobaeza added a commit that referenced this issue Mar 5, 2021
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes #1624
pedrobaeza added a commit that referenced this issue Mar 5, 2021
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes #1624
@pedrobaeza
Copy link
Member

Solucionado en los commits referenciados por GitHub. Aunque normalmente ya no doy soporte a la 11.0, no costaba mucho hacer el cherry-pick y lo he dejado hecho, y así @RodrigoBM no tiene que hacerlo 😉

@RodrigoBM
Copy link
Contributor

@pedrobaeza que alegría me das, muchas gracias. ❤️

@ManelNight
Copy link

Gracias @pedrobaeza, confirmo que esta solucionado para la v11.

Saludos

ferran-S73 pushed a commit to Studio73/l10n-spain that referenced this issue Mar 18, 2021
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes OCA#1624
ValentinVinagre pushed a commit to sygel-technology/l10n-spain that referenced this issue Dec 23, 2021
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes OCA#1624
kikosanchez pushed a commit to Comunitea/l10n-spain that referenced this issue May 25, 2022
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes OCA#1624
FernandoRomera pushed a commit to MallorcaSoft/l10n-spain that referenced this issue Dec 21, 2022
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes OCA#1624
pedrobaeza added a commit to Tecnativa/l10n-spain that referenced this issue Mar 21, 2024
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes OCA#1624
pedrobaeza added a commit to Tecnativa/l10n-spain that referenced this issue Mar 22, 2024
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

Fixes OCA#1624
Roodin pushed a commit to Comunitea/l10n-spain that referenced this issue Oct 24, 2024
If not, the constraint is evaluated each time, and can lead to
inconsistent states.

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

No branches or pull requests

4 participants