[ADD] l10n_es_ine_code: New module

[moval-agroingenieria]

• Module to obtain the National Statistics Institute codes for any city, province, or autonomous community of Spain.

[pedrobaeza]

Gracias por la contribución. Veo que el módulo queda algo cojo, al no haber ningún enlace con otro objeto del sistema, por ejemplo entre res.city y su código INE, o que tampoco se pueda escoger este código en res.partner.

[pedrobaeza]

Debería llamarse ca.po.

[pedrobaeza]

Sería conveniente evitar estas líneas intermedias vacías para mejor lectura de los "bloques de código".

[pedrobaeza]

Las imágenes de empresa hay que eliminarlas en las contribuciones a OCA.

[pedrobaeza]

Habría que diseñar una carga diferida de este archivo por dos razones:

• Tiempo de instalación inicial.
• Tiempo de actualización cada vez que se hace un update all.

Como pronto, una forma de mejorarlo es pasarlo a CSV, que tiene mejores tiempos de carga, pero como comento, lo más ideal sería que se cargue una vez instalado, al estilo de l10n_es_partner o l10n_es_toponyms.

[moval-agroingenieria]

Vale, gracias por las sugerencias y aclaraciones.
· Lo cambiaremos a CSV y diseñaremos la carga diferida.
· Se enlazará con la ciudad aunque no parece una información relevante para el usuario. Este modelo se había pensado para ser consultado por otros módulos que crean archivos normalizados de cobro en los que es obligatorio el código INE de la ciudad.
· Borramos el banner y arreglamos los problemas de cálidad de código.

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[pedrobaeza]

Hola. Nunca respondisteis a esto...

[moval-agroingenieria]

Buenas.

Sí, perdona, está pendiente. Vamos a darle prioridad para enviarlo antes de los 30 días.

[moval-agroingenieria]

Hola @pedrobaeza,

Gracias por el feedback en el PR. Tenemos la carga diferida implementada (CSV + post_init_hook), pero necesitamos aclarar algunos puntos antes de continuar:

1. Enlace con res.city

• ¿Qué esperas exactamente del enlace entre res.city y código INE?
• ¿Un campo ine_code_id en res.city que apunte a res.ine.code?
• ¿O campos Many2one en res.ine.code (city_id) que apunten a res.city?
• ¿O ambos (relación bidireccional)?

2. Campos Many2one en res.ine.code

• Mencionas enlazar el código de municipio con res.city, provincia con state, y CCAA con... ¿qué modelo exactamente?
• ¿Quieres que res.ine.code tenga campos state_id y city_id apuntando a estos registros?

3. Mapeo de códigos

• ¿Cómo mapeo los códigos INE de provincia (1-52) con los res.country.state de España? ¿Existe ya este mapeo en algún módulo de l10n-spain o alguna funcionalidad parecida?

Si puedes darnos un ejemplo concreto de cómo esperas que funcione la integración, sería de gran ayuda.

Gracias!

[pedrobaeza]

La idea sería en este código INE tener un city_id que apunte a la ciudad, y de ahí, puedes también obtener por related provincia si quisieras. Luego, hay que ver si se puede enganchar las ciudad de base_location_geonames_import con éstas. Al establecer la ciudad en el partner, se podría auto-establecer el código INE según esto.