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

[8.0][FIX] l10n_es_aeat_mod347: datos erróneos en clave B cuando un p… #1022

Merged
merged 1 commit into from
Feb 21, 2019

Conversation

enriquemartin
Copy link
Contributor

@enriquemartin enriquemartin commented Feb 21, 2019

…artner tiene facturas en clave A pero solo movimientos de caja en clave B

Ejemplo:

  • 2 facturas de proveedor cuyo suma es mayor que el límite --> clave A
  • 1 factura de cliente --> clave B
  • 1 factura rectificativa de cliente --> clave B
  • 1 movimiento de caja con importe superior al límite --> clave B
  • La "suma" del importe de ambas facturas de la clave B es 0 y por tanto en ese registro no es necesario detallarlas.

Con el código actual al llamar al método _partner_record_b_create después de haber pasado por _partner_record_a_create, la variable vals contiene la clave invoice_record_ids con las facturas de cliente.
En el método _partner_record_b_create al no entrar por el if porque el importe no supera el límite, esa clave no se actualiza. Al entrar en la parte en la que se detallan los movimientos de caja, se añaden nuevas claves al diccionario vals y se crea el registro (l10n.es.aeat.mod347.partner_record). El problema es que se arrastran las facturas que se han incluido en el registro de la clave a porque siguen estando en la variable vals.

Haciendo un copy de vals en cada uno de los métodos solucionamos el problema. En realidad en el método _partner_record_b_create no sería necesario, pero por consistencia.

@pedrobaeza
Copy link
Member

Para reducir el diff, en lugar de renombrar la variable, haz vals = vals.copy() y ya está.

@pedrobaeza pedrobaeza added this to the 8.0 milestone Feb 21, 2019
…artner tiene facturas en clave A pero solo movimientos de caja en clave B
@pedrobaeza pedrobaeza merged commit ff32727 into OCA:8.0 Feb 21, 2019
@enriquemartin enriquemartin deleted the 8.0_fix_l10n_es_aeat_mod347 branch September 28, 2022 13:31
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.

None yet

2 participants