OCA / l10n-spain

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

Migrar módulo l10n_es_aeat Odoo v17 #3382

Closed ramiadavid closed 9 months ago

ramiadavid commented 9 months ago

Como supongo que ya sabréis y si no os lo digo yo en la v17 han eliminado los modelos account.tax.template y account.accont.template, ahora se crean las cuentas y los impuestos leyendo directamente de los CSV cuando cargas el plan contable, pero estas plantillas no se guardan en la BBDD.

Esto supone un problema para la migración de este modulo (y todos los que dependen del el) ya que en los mapeos de impuestos se utilizaban estos registros para que el mapeo fuera válido para todas las compañías.

Yo he estando mirando de migrar este modulo creando yo unos modelos (aeat.tax y aeat.account) obteniendo los datos de los mismos CSV de la localización española (en realidad solo genero dos campos nombre para mostrarlo en el mapeo y xmlid para poder vincularlo con el impuesto/cuenta de la compañía), y con muy pocas modificaciones los demás módulos vuelven a funcionar.

No se que propuestas tenéis para este asunto, ya que es un cambio que supongo que habrá que consensuar.

Otra opción seria crear un mapeo para cada compañía y utilizar directamente los impuestos/cuentas de cada compañía pero esto implica mas cambios.

Ya me decís que opináis, ya que hasta que no se defina este asunto, no podemos avanzar con la migración de todos los módulos de la aeat.

pedrobaeza commented 9 months ago

En cuanto supe de esa eliminación, y conociendo que los XML-IDs siguen existiendo, pensé en sustituir cada m2o/m2m a objetos template por textos con los XML-IDs (campo texto para los m2o, y o2m para los m2m, por extensibilidad), y cambiar el poco código implicado para hacer un .ref(<xml_id>) en lugar de coger directamente el recordset. Pero el resto seguiría básicamente igual.

Con eso, los cambios no son muy grandes. ¿Qué te parece? Es lo mismo básicamente que tú comentas, ¿no?

Si quieres ponerte con ello, te voy revisando. Si no, yo me pondré en un mes o así.

pedrobaeza commented 9 months ago

De todas formas, cierro, porque esta conversación debería estar en #3298. Seguimos por ahí.