[16.0][MIG] l10n_es_intrastat_report: Migration to 16.0

[RabbitJon-S73]

_gather_invoices --> Suprimida la herencia ya que por defecto ya se comprueba el país de origen.
Las funciones _update_computation_line_vals y _prepare_invoice_domain creo que se podrían borrar:

• _prepare_invoice_domain: Añade el domain ("move_type", "in", ("in_invoice", "out_refund") / ("move_type", "in", ("out_invoice", "in_refund") pero de base esta llegando ("in_invoice", "in_refund") / ("out_invoice", "out_refund"), por lo que las dos restricciones no le veo mucho sentido.
• _set_defaults: Eliminado puesto que el informe ahora viene dado por el tipo de intrastat en la posición fiscal

Se ha cambiado el botón de generate_xml en intrasta_product a una action que llama a un informe sobre el modelo intrastat.product.declaration. El problema es que estamos trabajando con l10n.es.intrastat.product.declaration y este informe ya no se genera una vez instalado el módulo. La opción que he tomado para no duplicar código es heredar con el mismo nombre de modelo la declaración.

Los test y otros aspectos del módulo no están terminados, me esperaré a tener un poco de feedback sobre si los cambios son o no correctos para continuar.

[pedrobaeza]

Entiendo que eso no vale, porque la otra tabla ya existirá. Habrá que hacer un INSERT INTO.

[RabbitJon-S73]

@pedrobaeza @victoralmau ya está el script para migrar la base de datos

[victoralmau]

El script de migración entiendo que estará probado con algún ejemplo de datos en v15 y transformarlos para mostrarlos correctamente en v16.

He realizado alguna prueba funcionalidad en runboat para generar alguna declaración y revisar un poco su funcionamiento y parece ser todo correcto, por lo que por mi parte está OK.

¡Buen trabajo!

TT43234

[RabbitJon-S73]

Esto ya está probado por nuestra parte con clientes que venían de otras versiones. En principio la migración es correcta. No obstante, con un cliente que venia de la 12.0 hemos tenido problemas en el paso por la 14.0 ya que esta migracion renombra el campo en la tabla de intrastat_product_declaration pero no en l10n_es_intrastat_product_declaration y puede dar error. También el campo src_dest_country_code añadido en OCA/intrastat-extrastat#163 puede no estar presente en la tabla y fallar la migración. Esta query puede servir

UPDATE l10n_es_intrastat_product_declaration_line
SET src_dest_country_code = CASE WHEN rc.code = 'GB' THEN 'XI' ELSE rc.code END
FROM res_country AS rc WHERE rc.id = src_dest_country_id;

@pedrobaeza @victoralmau

[victoralmau]

Gracias por el feedback, siguiendo tus comentarios he añadido los cambios que indicas #3218, puedes revisarlo y si con eso quedaría totalmente afinado se podría fusionar.

[RabbitJon-S73]

Buenas @victoralmau @pedrobaeza podemos ir cerrando esto?

[pedrobaeza]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-3111-by-pedrobaeza-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 0dd8b58. Thanks a lot for contributing to OCA. ❤️

[SoniaViciana]

Este informe ¿realmente se ha probado en algún entorno? Me gustaría entender el procedimiento de mergeo. Sin meterme en profundidad...:

1. Están duplicados los códigos de Transacción:

2. Las regiones de intrastat hay que crearlas a mano.

3. No se refleja el incoterm en el fichero csv.

4. Salta un error al pulsar aquí en las facturas:

Saludos,

[victoralmau]

Este informe ¿realmente se ha probado en algún entorno? Me gustaría entender el procedimiento de mergeo. Sin meterme en profundidad...:

1. Están duplicados los códigos de Transacción:

Gracias por indicarlo, se está resolviendo en #3300

2. Las regiones de intrastat hay que crearlas a mano.

Esto ya era así en v15, por lo que en v16 no ha cambiado.

3. No se refleja el incoterm en el fichero csv.

A priori se debería mostrar el código incoterm https://github.com/OCA/l10n-spain/blob/16.0/l10n_es_intrastat_report/models/l10n_es_intrastat_product_declaration.py#L135 de la línea de declaración, pero será necesaria la corrección #3301 para que se defina la información correspondiente en las líneas de la declaración que se generan.

4. Salta un error al pulsar aquí en las facturas:

Se ha resuelto en OCA/intrastat-extrastat#237.

Saludos,

[SoniaViciana]

Muchas gracias por las aportaciones y la rápidez @victoralmau, en breve haremos nuevas pruebas y os comentamos.

Saludos,