OCA / l10n-spain

Odoo Spain Localization
https://www.aeodoo.org/estado-localizacion
GNU Affero General Public License v3.0
274 stars 517 forks source link

[10.0] l10n_es_aeat_sii: Cuota y base eviados siempre en la divisa de la factura #778

Closed omar7r closed 6 years ago

omar7r commented 6 years ago

Hola,

Espero estar equivocado, pero acabo de revisar el código y se están enviando al SII los importes de base e impuestos en libras por ejemplo si la factura de compra es libras y esto es incorrecto, en la 8.0 funciona bien pero de forma nativa en la 8.0 hay en las líneas de los impuestos de las facturas dos campos de base y dos de cuota que contienen los valores tanto en la divisa de la factura como en la divisa de la compañía y ahora en la 10.0 no hay los campos que dan estos importes en la divisa de la compañía por lo que, me imagino que habría que incluirlos en el módulo account_invoice_currency y usar estos en el SII. ¿Opiniones?

Me imagino que esto también afectará a la 9.0.

Un saludo

pedrobaeza commented 6 years ago

Sí, el problema puede ser el mismo en la v9 si es así.

Efectivamente, ahora se lee la tabla account.invoice.tax para obtener esos datos, y en dicha tabla tanto la base como la cuota aparecen en la moneda de la factura.

Es algo que habría que gestionar en el propio módulo, no en account_invoice_currency, ya que no se depende del mismo actualmente. En este módulo se podrían añadir campos calculados base_cc y amount_cc que hagan la conversión y sustituirlos en los métodos que obtienen los datos.

omar7r commented 6 years ago

Ok, si lo ves mejor dentro del módulo los añado ahí, lo decía porque es algo muy general que no se si afecta a algún otro módulo de la AEAT, ¿los campos calculados con store o sin?

pedrobaeza commented 6 years ago

Bueno, yo creo que en ningún otro caso hace falta ese desglose... ¿tienes alguno en mente?

No sé, el problema es añadir una dependencia a mitad de la vida del módulo, que haría fallar a aquellos que tienen el módulo SII instalado pero no el otro. Pero sí, lo de ser genérico no está mal.

Dado que el propio campo base es no store, no puedes hacer store la correspondencia en moneda de la compañía.

omar7r commented 6 years ago

No, me acordé de las distintas versiones del libro de iva que hubo en las versiones anteriores pero en la 10.0 ya ví que se hacía por apuntes, lo añado en el l10n_es_aeat_sii y si en algún momento vemos que tal se pueden mover a l10n_es_aeat, que sí es dependencia.

pedrobaeza commented 6 years ago

👍

omar7r commented 6 years ago

PR propuesto