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

[FW][FIX] account_payment_group: constraint move name (seq) #345

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

fw-bot-adhoc
Copy link
Contributor

Cuando tenemos un move asociado al payment group queremos que el move tenga el mismo nombre que el del payment group. Esto para que en cualquier reporte aparezca este nombre de referencia.

En verion 15 hay una constraint nueva _constrains_date_sequence que trae un problema. cuando vamos a validar un account.move revisa el nombre/secuencia que le fue asignado con un regex para encontrar algo parecido a un año y lo compara con el año de la fecha del move y si son diferentes años no permite continuar. Ejemplo: Tenemos un pago de proveedor que se llama OP - 2022 - 00012345 y lo validamos y generamos los account.move en el año 2023. El resultado es que nos salta la excpeción indicando que la secuencia que queremos asignar al move no es valida y debe corresponder al año.

Para fines del account.move no queremos que esto aplique. No importa que nombre utilice el payment.group queremos que al validarse se traslade el nombre a los move generados. La modificacion de este metodo logra evitar esa comparacion de fechas

Forward-Port-Of: #344

Cuando tenemos un move asociado al payment group queremos que el move tenga el mismo nombre que el del payment
group. Esto para que en cualquier reporte aparezca este nombre de referencia.

En verion 15 hay una constraint nueva _constrains_date_sequence que trae un problema. cuando vamos a validar un
account.move revisa el nombre/secuencia que le fue asignado con un regex para encontrar algo parecido a un
año y lo compara con el año de la fecha del move y si son diferentes años no permite continuar. Ejemplo:
Tenemos un pago de proveedor que se llama OP - 2022 - 00012345 y lo validamos y generamos los account.move en el
año 2023. El resultado es que nos salta la excpeción indicando que la secuencia que queremos asignar al move no
es valida y debe corresponder al año.

Para fines del account.move no queremos que esto aplique. No importa que nombre utilice el payment.group queremos
que al validarse se traslade el nombre a los move generados. La modificacion de este metodo logra evitar esa
comparacion de fechas

X-original-commit: 71e5bc4
@roboadhoc
Copy link
Contributor

@fw-bot-adhoc
Copy link
Contributor Author

@zaoral cherrypicking of pull request #344 failed.

stdout:

Auto-merging account_payment_group/models/account_move.py
CONFLICT (content): Merge conflict in account_payment_group/models/account_move.py

stderr:

14:25:47.022862 git.c:439               trace: built-in: git cherry-pick 330cc903489991eb5b8c6357c205f1844ef2d756
error: could not apply 330cc903... [FIX] account_payment_group: constraint move name (seq)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'
----------
status:

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

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.

None yet

3 participants