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

l10n_es_aeat_349: corrige error en el 349 cuando se generan apuntes contables #918

Merged
merged 1 commit into from
Sep 12, 2018

Conversation

JordiBForgeFlow
Copy link
Sponsor Member

Corrige error en el 349 cuando se generan apuntes contables asociados a una factura que no apuntan directamente a una línea de factura.

…ontables

asociados a una factura que no apuntan directamente a una línea de factura.
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

👍

@pedrobaeza
Copy link
Member

Esto es lo mismo que #913 y la solución ya dijimos cuál era ahí.

@pedrobaeza pedrobaeza closed this Sep 12, 2018
@JordiBForgeFlow
Copy link
Sponsor Member Author

@pedrobaeza En este caso el fix es para otra cosa. En Eficent usamos anglo saxon accounting (es muy mal nombre para poner a lo que en verdad se llama procedimiento de inventario perpetuo por desdoblamoiento completo/incompleto. Ver https://odoospain.odoo.com/blog/noticias-odoospain-1/post/contabilizacion-de-compras-y-ventas-con-inventario-permanente-y-desdoblamiento-completo-10

Mira https://github.com/odoo/odoo/blob/3656999f5e51658e0611af100c1507abc7647d94/addons/purchase/models/account_invoice.py#L198

@pedrobaeza
Copy link
Member

Vale, pues vaya chapuza de sobreescritura que hace ese módulo...

@pedrobaeza pedrobaeza merged commit 21e0ee7 into OCA:11.0 Sep 12, 2018
@LoisRForgeFlow LoisRForgeFlow deleted the 11.0-l10n_es_aeat_349-fix branch September 14, 2018 12:03
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