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] account_payment_group: constraint move name (seq) #344

Closed
wants to merge 1 commit into from

Conversation

zaoral
Copy link
Contributor

@zaoral zaoral commented Jan 4, 2023

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

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
@roboadhoc
Copy link
Contributor

@zaoral
Copy link
Contributor Author

zaoral commented Jan 4, 2023

@roboadhoc r+

roboadhoc pushed a commit to ingadhoc/odoo-public-administration that referenced this pull request Jan 4, 2023
This reverts commit 0eee2ba.

closes #470

Related: ingadhoc/account-payment#344
Signed-off-by: Katherine Zaoral <[email protected]>
roboadhoc pushed a commit that referenced this pull request Jan 4, 2023
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

closes #344

Related: ingadhoc/odoo-public-administration#470
Signed-off-by: Katherine Zaoral <[email protected]>
@roboadhoc roboadhoc temporarily deployed to merge January 4, 2023 14:25 Inactive
@roboadhoc roboadhoc closed this Jan 4, 2023
@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

7 similar comments
@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

@fw-bot-adhoc fw-bot-adhoc deleted the 15.0-h-58360-kz branch January 18, 2023 14:37
@fw-bot-adhoc
Copy link
Contributor

@zaoral this pull request has forward-port PRs awaiting action (not merged or closed):
#345

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

Successfully merging this pull request may close these issues.

3 participants