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

l10n_es_account_bank_statement_import_n43. No se pueden importar extracto en moneda distinta a EUR #217

Closed Roodin closed 7 years ago

Roodin commented 8 years ago

La información de la divisa de la cuenta bancaria a importar está puesta de forma fija a EUR en el propio código y no se está extrayendo del código de divisa que figura en el archivo N43.

En el módulo account_bank_statement_import se añadió en junio una comprobación de divisa que impide realizar ahora la importación del extracto de una cuenta en otra divisa,

pedrobaeza commented 8 years ago

Si existe esa información de divisa en el N43, entonces se puede coger, pero creo recordar que no existía. Lo que se puede hacer entonces es coger la divisa del diario. ¿Puedes proponer el código que en uno u otro caso lo resuelva?

Gracias.

Roodin commented 8 years ago

Sí está la información, pero en una codificación ISO de tres caracteres numéricos. Tengo una solución hecha pero solo para solucionar dólares y libras, que me urgía, tengo que crear un diccionario para mapear las otras monedas que constan al menos en el anexo de La Caixa y lo propongo.

Dólar australiano 036 Dólar canadiense 124 Corona Danesa 208 Yen japonés 392 Dólar neozelandés 554 Corona noruega 578 Corona sueca 752 Franco suizo 756 Libra esterlina 826 Dólar USA 840 Euro 978

pedrobaeza commented 8 years ago

Vaya! Por eso no me sonaba a mí, porque es una codificación variable. Y sabes si todos los bancos coinciden? Yo creo que lo más sensato puede ser importarlo en la moneda del diario del banco, no crees?

Roodin commented 8 years ago

Es una codificación estándar (código numérico que tenga establecido en la tabla del estándar ISO 4217) lo que pasa es que esa ISO 4217 [1] contiene tanto el código alfabético que usa Odoo como el numérico que usa la norma 43, de ahí que no sé hasta qué punto interesaría hacer algo más general para incluir este código numérico en las monedas, bien dentro del módulo de extractos o más global. ¿Cómo lo veis?

[1]http://www.iso.org/iso/es/home/standards/currency_codes.htm

pedrobaeza commented 8 years ago

Gracias por la referencia. Queda claro, aunque la elección del formato del N43 no sea el más adecuado para Odoo en este caso. Si se mete el código, lo suyo sería en un módulo genérico _base_currencyiso4217 o similar para independizarlo del N43, pero sigo pensando: ¿no es mejor quitarnos de líos e ignorar esa moneda y poner la que lleve el diario?

Roodin commented 8 years ago

Supongo que sí, lo único es tener esa confirmación de la moneda correcta y que estaríamos pasando por alto una comprobación que se hace más tarde en account_bank_statement_import, que es el motivo de que ahora falle, pero tampoco veo porque esto podría causar problemas, no creo que desde una cuenta en USD por ejemplo, se pueda generar un extracto en EUR, aunque no estoy seguro.

El 21 de septiembre de 2015, 19:45, Pedro M. Baeza <notifications@github.com

escribió:

Gracias por la referencia. Queda claro, aunque la elección del formato del N43 no sea el más adecuado para Odoo en este caso. Si se mete el código, lo suyo sería en un módulo genérico _base_currencyiso4217 o similar para independizarlo del N43, pero sigo pensando: ¿no es mejor quitarnos de líos e ignorar esa moneda y poner la que lleve el diario?

— Reply to this email directly or view it on GitHub https://github.com/OCA/l10n-spain/issues/217#issuecomment-142056271.

acysos commented 8 years ago

Hola,

Has probado a importar con QIF en vez de usar N43 para las cuentas de moneda extranjera?

Saludos

fgarciatecbrain commented 8 years ago

Hola,

Tal como dice pedrobaeza es mejor tomar la moneda del díario. para hacer esto se ha de modificar el fichero "account_bank_statement_import_n43.py" (línea 289)

y cambiar

return 'EUR', False, [vals_bank_statement]

por

return self.journal_id.currency.name, False, [vals_bank_statement]

Probado en versión 8.

Saludos,

pedrobaeza commented 7 years ago

Como esto está ya solucionado en el PR relacionado, cierro.