[17.0][IMP] l10n_es_cnae: Translate industries into Spanish

[manuelregidor]

Este PR genera correctamente las traducciones de las industrias específicas de España al castellano.

T-6028

[manuelregidor]

@HaraldPanten @ValentinVinagre

[pedrobaeza]

No debería ser solo ca.po?

[manuelregidor]

@pedrobaeza Fue la primera opción que barajamos, pero las industrias se crean sin ID externo, por lo que desde el fichero .po no se podían referencia y no cogía las traducciones. La opción de generar las industrias específicas de España con external ID también se contempló, pero como el módulo l10n_eu_nace no crea IDs externos, decidimos no crearlos tampoco para las industrias que se crean con este módulo.

[pedrobaeza]

Vale, si yo lo he dicho al ver el listado de archivos modificados, que ver un ca_ES.po, y creo que debería ser ca.po a secas

[manuelregidor]

@pedrobaeza Modificado. Gracias!

[pedrobaeza]

Una duda, esto está poniendo la traducción a catalán de recursos de fuera de este módulo?

También veo que el nuevo método está sin cobertura de tests.

[HaraldPanten]

Una duda, esto está poniendo la traducción a catalán de recursos de fuera de este módulo?

También veo que el nuevo método está sin cobertura de tests.

Correcto. Es un caso algo especial y se están traduciendo conceptos que no son de este módulo propiamente dicho sino de su dependencia. Pero tiene su explicación:

El wizard del módulo l10n_eu_nace descarga las claves oficiales a nivel europeo y el idioma que corresponde (la traducción). El problema es que el catalán (a día de hoy) no es idioma oficial en la UE, por lo que los códigos comunes no pueden ser traducidos directamente en ese módulo.

Al no poder hacerlo ahí, creemos que la mejor forma de hacerlo es directamente desde l10n_es_cnae, ya que es en España donde tenemos varios idiomas que además están soportados en Odoo y que no son oficiales a nivel europeo actualmente (euskera, gallego y catalán).

Si el día de mañana los idiomas son definitivamente oficiales, lo correcto sería que añadiéramos las traducciones de la misma forma que se añade la traducción al castellano, francés, alemán... y que lo hiciera el módulo l10n_eu_nace.

Hemos pensado que la opción planteada en este PR es la mejor forma de darle solución. Los módulos de la localización española también es útil que den solución a estos casos de uso.

Estamos dispuestos a mejorar lo que sea necesario, en caso de que haya una forma mejor.

[pedrobaeza]

Vale, entonces creo que eso debería ir en un módulo aparte l10n_ca_cnae, no aquí, ya que si no, se está añadiendo a este módulo cosas que mucha gente no podrán mantener por no ser catalano-parlante. Si solo fuera la parte de catalán de las claves que se añaden aquí, podría ser tolerable, pero lo otro ya supone mucha más carga, incluido el código extra.

[manuelregidor]

@pedrobaeza De acuerdo, quito la parte del catalán del módulo l10n_es_cnae y la pongo en el módulo l10n_ca_cnae. Amplio tests también.

[manuelregidor]

@pedrobaeza @HaraldPanten Realicé el cambio sugerido y se ha abierto un PR dependiente de este con el módulo l10n_ca_cnae (https://github.com/OCA/l10n-spain/pull/3631)

[OCA-git-bot]

This PR looks fantastic, let's merge it! Prepared branch 17.0-ocabot-merge-pr-3628-by-pedrobaeza-bump-minor, awaiting test results.

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[OCA-git-bot]

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